Closed
Bug 398549
Opened 17 years ago
Closed 17 years ago
Find a great name for the "Extra" tab in Cert Manager
Categories
(Core :: Security: PSM, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: KaiE, Assigned: KaiE)
References
Details
Attachments
(1 file)
3.78 KB,
patch
|
rrelyea
:
review+
sayrer
:
approval1.9+
|
Details | Diff | Splinter Review |
The 5th tab in Cert Manager is currently named "Extra". It has been mentioned before that name isn't really perfect. It has also been proposed to use "Orphans", which wasn't accepted either. Today I came up with a new idea: "Unsorted" What do you think? Better ideas? Or should we keep "Extra"?
Assignee | ||
Comment 2•17 years ago
|
||
I think "Other" is too similar to our "Other's people" tab. When you walk people through the UI over a phone, the tabs should be easier to distinguish.
Comment 3•17 years ago
|
||
FWIW, "Uncategorized" matches the full sentence that accompanies the tab: You have certificates on file that do not fit in any of the other categories. http://mxr.mozilla.org/mozilla/source/security/manager/locales/en-US/chrome/pippki/certManager.dtd#53 Likewise "Unclassified", "Unrecognized", "Unknown" - don't really know if one of those is better than the others. "Others" isn't a bad idea, and Kai's concern could be readily addressed by changing "Other People's" to just "People". This would give it nice symmetry with "Servers" and "Authorities", giving us: .-------------------------------------------------------------. | Certificate Manager | +-------------------------------------------------------------+ | Your Certificates | People | Servers | Authorities | Others | ... Though if we're tinkering, I wonder if, frequency-wise, it might be more appropriate to go: .-------------------------------------------------------------. | Certificate Manager | +-------------------------------------------------------------+ | Your Certificates | Servers | Authorities | People | Others |
(In reply to comment #3) These mockups look very good. I could go with either set.
Assignee | ||
Comment 5•17 years ago
|
||
I like the proposal to reame "Other People's" to "People". If we renamed tabs and change the ordering at the same time would this confuse users? I think I'd prefer to keep the ordering that users are currently used to. Whatever we do, I'd prefer if we kept People-and-Servers next to each other, as those are very similar classes of certs. The "frequency" that users use the tabs depends on the application. In Thunderbird our users might use the "people" tab a lot, and never look at the "server" tab.
Assignee | ||
Comment 6•17 years ago
|
||
(In reply to comment #3) > FWIW, "Uncategorized" matches the full sentence that accompanies the tab: You > have certificates on file that do not fit in any of the other categories. I feel that "uncategorized" is a complicated term, while "unsorted" feels like a simple term. I you translate that to german, you end up with "Unkategorisiert", which feels strange, even though the term "Kategorie" exists. On the other hand the term "unsortiert" probably makes sense to everyone. But I'm hearing you'd prefer "Others" anyway. > Likewise "Unclassified", "Unrecognized", "Unknown" - don't really know if one > of those is better than the others. If a cert ends up being displayed in the "Extra" tab, it just means one thing: Our logic to sort the cert failed to identify it. I'll attach a patch: - rename "Other People's" to "People" - rename "Extra" to "Others"
Assignee | ||
Comment 7•17 years ago
|
||
Please see bug 387480 comment 78. For consistency, we should adjust the sentence in the "servers" tab. I'm adding this change to this patch, too.
Assignee | ||
Comment 8•17 years ago
|
||
Attachment #283997 -
Flags: review?(rrelyea)
Comment 9•17 years ago
|
||
Comment on attachment 283997 [details] [diff] [review] Patch v1 r+ the code is correct as long as ui is happy with the names. bob
Attachment #283997 -
Flags: review?(rrelyea) → review+
Assignee | ||
Comment 10•17 years ago
|
||
(In reply to comment #9) > r+ the code is correct as long as ui is happy with the names. I think both Johnathan and Bob L are happy, because I picked the first proposal from comment 3.
Assignee | ||
Comment 11•17 years ago
|
||
Comment on attachment 283997 [details] [diff] [review] Patch v1 Requesting approval for this patch that changes strings and string-IDs only
Attachment #283997 -
Flags: approval1.9?
Updated•17 years ago
|
Attachment #283997 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 12•17 years ago
|
||
checked in, marking fixed.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 13•17 years ago
|
||
verified fixed with Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9a9pre) Gecko/2007101605 Minefield/3.0a9pre ID:2007101605
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 14•17 years ago
|
||
Reopening bug. All patches that got checked in to trunk yesterday are being backed out, because it's unclear which patch has caused a performance regression.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 15•17 years ago
|
||
checked in again, marking fixed.
Status: REOPENED → RESOLVED
Closed: 17 years ago → 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•