Last Comment Bug 335637 - "Backup" and "Backup all" should be "Back Up" in the Certificate action menu
: "Backup" and "Backup all" should be "Back Up" in the Certificate action menu
Status: RESOLVED FIXED
[good first bug]
: fixed1.8.1.1, polish
Product: Camino Graveyard
Classification: Graveyard
Component: Security (show other bugs)
: unspecified
: PowerPC Mac OS X
-- minor (vote)
: Camino1.5
Assigned To: froodian (Ian Leue)
:
:
Mentors:
Depends on:
Blocks: 325880
  Show dependency treegraph
 
Reported: 2006-04-26 23:18 PDT by Smokey Ardisson (offline for a while; not following bugs - do not email)
Modified: 2006-12-11 08:12 PST (History)
5 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
change "Backup" to "Back Up" in .nib file (1.50 KB, patch)
2006-07-02 01:57 PDT, Isriya Paireepairit
froodian: review-
Details | Diff | Splinter Review
Reupload CertificatesWindow.nib (11.47 KB, application/zip)
2006-07-02 08:06 PDT, Isriya Paireepairit
froodian: review-
Details
Reupload CertificatesWindow.nib again (11.74 KB, application/zip)
2006-07-17 21:29 PDT, Isriya Paireepairit
froodian: review-
Details
New CertificatesWindow.nib (11.72 KB, application/zip)
2006-12-01 00:23 PST, froodian (Ian Leue)
no flags Details
New CertificatesWindow.nib (11.71 KB, application/zip)
2006-12-01 20:26 PST, froodian (Ian Leue)
alqahira: review+
stuart.morgan+bugzilla: superreview-
Details
New CertificatesWindow.nib (11.71 KB, application/zip)
2006-12-05 17:48 PST, froodian (Ian Leue)
no flags Details
Actually 20 px (11.72 KB, application/zip)
2006-12-05 18:07 PST, froodian (Ian Leue)
stuart.morgan+bugzilla: superreview+
Details
Screenshot of 247613 (31.85 KB, image/png)
2006-12-05 18:07 PST, froodian (Ian Leue)
no flags Details

Description User image Smokey Ardisson (offline for a while; not following bugs - do not email) 2006-04-26 23:18:22 PDT
From bug 335393:

Someone can file yet another bug about changing "Backup" to "Back Up"

This looks like just a nib change to CertificatesWindow.nib
Comment 1 User image Mike Pinkerton (not reading bugmail) 2006-04-27 05:27:19 PDT
http://dictionary.reference.com/search?q=backup

i don't think we need to change this. To me "Back Up" means to go backwards, like reverse.
Comment 2 User image Chris Lawson (gone) 2006-04-27 06:07:30 PDT
Keep scrolling on that page, Mike, and you'll see this:

backup

<operating system> ("back up" when used as a verb)

Two words as a verb. None of the definitions above that are for verbs; they're all for nouns or adjectives. You can have "backup software" to make a "backup", but that software "backs up" your data. (Note you don't say it "backups" your data.)

cl
Comment 3 User image Mike Pinkerton (not reading bugmail) 2006-04-27 07:35:16 PDT
you win ;) i concede.
Comment 4 User image Chris Stewart 2006-06-21 14:44:15 PDT
Is this being worked on?  I'm new to contributing and something this minor would be a good place to start.  Can I run with this if it hasn't been assigned?

- Chris
Comment 5 User image Chris Lawson (gone) 2006-06-21 15:46:05 PDT
(In reply to comment #4)
> Is this being worked on?  I'm new to contributing and something this minor
> would be a good place to start.  Can I run with this if it hasn't been
> assigned?

Go right ahead, and welcome to the gang!

cl
Comment 6 User image Chris Stewart 2006-06-21 15:50:26 PDT
Thank you.  It will take me some time to get my environment configured, etc, etc, so please be patient with me. :)

- chris
Comment 7 User image Isriya Paireepairit 2006-07-02 01:57:07 PDT
Created attachment 227873 [details] [diff] [review]
change "Backup" to "Back Up" in .nib file
Comment 8 User image froodian (Ian Leue) 2006-07-02 07:36:06 PDT
Comment on attachment 227873 [details] [diff] [review]
change "Backup" to "Back Up" in .nib file

I'm glad you're interested in helping out, but we don't accept diffs for nib files.  For more info, check out http://wiki.caminobrowser.org/Development:Reviewing (you want to zip the actual nib and attach it here) :)
Comment 9 User image Isriya Paireepairit 2006-07-02 08:06:56 PDT
Created attachment 227881 [details]
Reupload CertificatesWindow.nib

Ok, I don't know about this before. Reupload.
Comment 10 User image Smokey Ardisson (offline for a while; not following bugs - do not email) 2006-07-02 22:21:48 PDT
Håkan, are we doing the right thing tab-chain wise in this nib (Security prefPane, Certificates button)?  

Seems to me this is just like the Bookmarks Manager, so the views should be getting a focus ring (which would lessen confusion as to why tabbing once--with a Category that has some right-pane content--doesn't focus the first item but requires a down-arrow--again, just like the BM).
Comment 11 User image Håkan Waara 2006-07-03 02:42:11 PDT
This nib worked fine for me, focus-wise. The first listbox is focused (with a blue focus ring), and the listboxes (+  the action button with FKA on) are in the focus chain.
Comment 12 User image froodian (Ian Leue) 2006-07-03 08:02:13 PDT
Comment on attachment 227881 [details]
Reupload CertificatesWindow.nib

Hi Isriya,

This looks good, but it occurs to me that while we're touching this nib, we should make the window comply with our margins standards in http://wiki.caminobrowser.org/Development:Editing_Nibs

Specifically, the action menu button should be 17 px below the view it's under, and the bottom of the window should  be 20 px below that.  Also, the initialFirstResponder should be set to the entire extended split view, and the extended split view and the action menu should each have a nextKeyView of eachother.
Comment 13 User image Isriya Paireepairit 2006-07-03 08:13:01 PDT
Actually, I didn't touch anything in the NIB, just change the string and recompile. mmm... I might accidentially hit it and move. 

Anyway, I'll resubmit this. Thanks for review.
Comment 14 User image Isriya Paireepairit 2006-07-17 21:29:21 PDT
Created attachment 229613 [details]
Reupload CertificatesWindow.nib again

Update NIB file to match Camino Development Guideline.
Comment 15 User image froodian (Ian Leue) 2006-07-18 12:32:40 PDT
Comment on attachment 229613 [details]
Reupload CertificatesWindow.nib again

In this case I would leave initialFirstResponder unset (since that way it does the Right Thing in focusing Category first, instead of focusing the actions menu button).

Also, the actions menu button should only be 17 px below the bottom of of the ExtendedOutlineView (Name, details, expires, etc).

Looking better.
Comment 16 User image froodian (Ian Leue) 2006-12-01 00:23:31 PST
Created attachment 247161 [details]
New CertificatesWindow.nib
Comment 17 User image froodian (Ian Leue) 2006-12-01 20:26:19 PST
Created attachment 247247 [details]
New CertificatesWindow.nib

Increases min width and does a 3px bounding-box adjustment, per IRC.
Comment 18 User image Smokey Ardisson (offline for a while; not following bugs - do not email) 2006-12-01 20:32:11 PST
Comment on attachment 247247 [details]
New CertificatesWindow.nib

Nice; r=ardissone

Requesting rs on these cosmetic nib-only changes (per policy).
Comment 19 User image Stuart Morgan 2006-12-01 20:48:54 PST
Comment on attachment 247247 [details]
New CertificatesWindow.nib

(In reply to comment #12)
> Specifically, the action menu button should be 17 px below the view it's under,
> and the bottom of the window should  be 20 px below that.

No; this creates a lot of weird dead space at the bottom, and every example of similar controls I can think of looks the way it was before.
Comment 20 User image Smokey Ardisson (offline for a while; not following bugs - do not email) 2006-12-03 11:35:47 PST
(In reply to comment #19)
> (From update of attachment 247247 [details] [edit])
> (In reply to comment #12)
> > Specifically, the action menu button should be 17 px below the view it's under,
> > and the bottom of the window should  be 20 px below that.
> 
> No; this creates a lot of weird dead space at the bottom, and every example of
> similar controls I can think of looks the way it was before.

I see nothing in the HIG that supports having a miniscule bottom margin instead of the standard Aqua margins; moreover, it looks out of place in comparison with our cookie sheets, where the button is well-placed.
Comment 21 User image Stuart Morgan 2006-12-03 13:38:45 PST
(In reply to comment #20)
> I see nothing in the HIG that supports having a miniscule bottom margin instead
> of the standard Aqua margins;

The 'G' is for Guidelines; I don't think we should slavishly following margin guidelines that are there solely for aesthetics even when it makes the overall visual appearance worse and more unbalanced.

Note that we also use the current cert spacing in the Bookmark Manager.

> moreover, it looks out of place in comparison with our cookie sheets, where the
> button is well-placed.

We also have an aqua button there, making the space much less empty. If the cert viewer were a sheet with a Done button, then I would agree.
Comment 22 User image froodian (Ian Leue) 2006-12-03 13:42:43 PST
What about putting the button back close to the table view, but leaving the 20 px between it and the edge of the window?  Although the bookmark manager has dimensions that reflect our currect certificate window, the status bar buffers the edge of the window.  I really think that the 20px around the edge is important.
Comment 23 User image froodian (Ian Leue) 2006-12-05 17:48:47 PST
Created attachment 247610 [details]
New CertificatesWindow.nib

Does comment 22
Comment 24 User image froodian (Ian Leue) 2006-12-05 18:07:21 PST
Created attachment 247613 [details]
Actually 20 px

Sorry, *this* does comment 22
Comment 25 User image froodian (Ian Leue) 2006-12-05 18:07:47 PST
Created attachment 247614 [details]
Screenshot of 247613
Comment 26 User image Stuart Morgan 2006-12-11 08:03:08 PST
Comment on attachment 247613 [details]
Actually 20 px

sr=smorgan
Comment 27 User image froodian (Ian Leue) 2006-12-11 08:12:53 PST
Checked in on 1.8branch and trunk.

Note You need to log in before you can comment on or make changes to this bug.