Change phrasing of "Advanced" button tip in Appearance/Fonts

VERIFIED FIXED in Camino1.5

Status

defect
--
trivial
VERIFIED FIXED
13 years ago
13 years ago

People

(Reporter: lensovetp, Assigned: froodian)

Tracking

({verified1.8.1})

unspecified
Camino1.5
PowerPC
macOS

Details

Attachments

(4 attachments, 7 obsolete attachments)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1a3) Gecko/20060614 Camino/1.0+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1a3) Gecko/20060614 Camino/1.0+

As part of the checkin for bug 277278, we introduced a new "Advanced" button "tip". Currently it reads « In Advanced you can set the minimum font size or other font types for this region. »
I propose we change it to « To set the minimum font size and other font types for this region, click Advanced... ». I'm not really crazy about the current version, in part because it uses the word "you", which I think is a little weird for a UI. Also, it says "or", though I don't get why - you can change Both the font size And the font types in the sheet. There's also a comma missing after Advanced. The new version would also go above the button, rather than below it.

Reproducible: Always
(Assignee)

Comment 1

13 years ago
We use "you" all throughout the prefpanes, so I don't think that's bad, but I do agree that the text could be less awkward.  Perhaps just 

"Set the minimum font size and other font types for this region."  

This would keep parity with the tip text for webfeatures.prefpane's "Add sites that should be allowed to show pop-up windows to a list."

The text should still be below the button though, since this is what we do with all our other tip texts.  By the way, bug 277278 didn't introduce the tip, it just capitalized the word "Advanced."
Blocks: 325880
Status: UNCONFIRMED → NEW
Ever confirmed: true
Target Milestone: --- → Camino1.1
Summary: Change phrasing of "Advanced" button tip in Apperance/Fonts → Change phrasing of "Advanced" button tip in Appearance/Fonts
(Reporter)

Comment 2

13 years ago
ah, ok :). I see the reference in webfeatures, and then this sounds OK. I'm not exceptionally crazy about it just saying "Set...", but consistency works, and I don't like the idea of saying "click this" as well. and compromise is good, so I say your version works.

p.s. as for you using "you" elsewhere, we only use it in Keychain access, and that wording needs to get changed too. Have it say «Edit your keychain items» (or maybe, Edit your keychain items with the Keychain Access app). And then make the "some websites won't let you save..." go on a separate line.
(Assignee)

Updated

13 years ago
Depends on: 278820
(Assignee)

Comment 3

13 years ago
When this is done we should be sure to resize the window correctly (595px) and make "Page Colors" consistent with "Link appearance"
(Assignee)

Comment 4

13 years ago
Posted file New Appearance.nib (obsolete) —
Attachment #227914 - Flags: review?(alqahira)
(Assignee)

Comment 5

13 years ago
Posted image Screenshot of the colors & links tab (obsolete) —
Assignee: nobody → stridey
Status: NEW → ASSIGNED
(Assignee)

Comment 6

13 years ago
Posted image Screenshot of links tab (obsolete) —
(Assignee)

Comment 7

13 years ago
Posted image Screenshot of advanced sheet (obsolete) —
(Assignee)

Comment 8

13 years ago
One question:  How should we do the px at the top of the prefpane?  I just left it the way it was, but do we want it 20px from the top of the "this is inside the tabview" area, or 20px from the top of the tabs? (sorry for the bugspam)
(Assignee)

Comment 9

13 years ago
Posted file Minor IRC tweaks (obsolete) —
This is more or less like the screenshots.  Just a couple px here and there.
Attachment #227914 - Attachment is obsolete: true
Attachment #227931 - Flags: review?(alqahira)
Attachment #227914 - Flags: review?(alqahira)
(Assignee)

Comment 10

13 years ago
Posted file Sets min width/height (obsolete) —
Attachment #227931 - Attachment is obsolete: true
Attachment #227933 - Flags: review?(alqahira)
Attachment #227931 - Flags: review?(alqahira)
(Assignee)

Comment 11

13 years ago
Posted file Not b0rken (obsolete) —
Sorry, that last nib was a dud (saved showing the wrong tab).
Attachment #227933 - Attachment is obsolete: true
Attachment #227936 - Flags: review?(alqahira)
Attachment #227933 - Flags: review?(alqahira)
(Assignee)

Comment 12

13 years ago
Hi reed.
Attachment #227936 - Attachment is obsolete: true
Attachment #227938 - Flags: review?(alqahira)
Attachment #227936 - Flags: review?(alqahira)
(In reply to comment #12)
> Hi reed.

Hi! How are you doing? :)
Comment on attachment 227938 [details]
Makes dropdown menu longer

r=me

Hopefully this is the end of this party :-)
Attachment #227938 - Flags: superreview?(mikepinkerton)
Attachment #227938 - Flags: review?(alqahira)
Attachment #227938 - Flags: review+
(Reporter)

Comment 15

13 years ago
(In reply to comment #8)
> One question:  How should we do the px at the top of the prefpane?  I just left
> it the way it was, but do we want it 20px from the top of the "this is inside
> the tabview" area, or 20px from the top of the tabs? (sorry for the bugspam)
> 

actually, 10px from the bottom of the tabs. see http://developer.apple.com/documentation/UserExperience/Conceptual/OSXHIGuidelines/art/ct_tabpaneinset.gif and http://developer.apple.com/documentation/UserExperience/Conceptual/OSXHIGuidelines/XHIGControls/chapter_18_section_7.html#//apple_ref/doc/uid/TP30000359-DontLinkElementID_524
(Assignee)

Comment 16

13 years ago
I was referring to the space between the tabs and the top of the window, not the tabs and the top of the inset, but thanks.  It's dealt with (per IRC). :)
what are we fixing here? i'm so confused.
(Assignee)

Comment 18

13 years ago
We're:

- Changing "In Advanced you can set the minimum font size or
other font types for this region." to "Set the minimum font size and other font types for this region."
- Correcting margins and spacing to match all the other prefpanes' new standard
- Making "Link appearance" "Link Appearance" to match "Page Colors"

In other words, just cosmetic polish as part of the prefpane cleanup.
(Assignee)

Comment 19

13 years ago
I realized that the screenshots included are a few revisions out of date (even though there have only been minor changes since then).  Uploading screenshots of the final versions for sr-ing pleasure. ;)
Attachment #227915 - Attachment is obsolete: true
(Assignee)

Comment 20

13 years ago
Attachment #227916 - Attachment is obsolete: true
(Assignee)

Comment 21

13 years ago
Attachment #227917 - Attachment is obsolete: true
Comment on attachment 227938 [details]
Makes dropdown menu longer

rs=pink
Attachment #227938 - Flags: superreview?(mikepinkerton) → superreview+
(Assignee)

Updated

13 years ago
Whiteboard: [needs checkin]

Comment 23

13 years ago
Fixed trunk and branch.
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [needs checkin]
(Assignee)

Updated

13 years ago
No longer depends on: 278820
(Assignee)

Updated

13 years ago
Status: RESOLVED → VERIFIED
Keywords: verified1.8.1
(Assignee)

Updated

13 years ago
Keywords: fixed1.8.1
You need to log in before you can comment on or make changes to this bug.