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

VERIFIED FIXED in Camino1.5

Status

Camino Graveyard
Preferences
--
trivial
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: Paul Borokhov (lensovet), Assigned: froodian (Ian Leue))

Tracking

({verified1.8.1})

unspecified
Camino1.5
PowerPC
Mac OS X
verified1.8.1

Details

Attachments

(4 attachments, 7 obsolete attachments)

(Reporter)

Description

11 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

11 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

11 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

11 years ago
Depends on: 278820
(Assignee)

Comment 3

11 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

11 years ago
Created attachment 227914 [details]
New Appearance.nib
Attachment #227914 - Flags: review?(alqahira)
(Assignee)

Comment 5

11 years ago
Created attachment 227915 [details]
Screenshot of the colors & links tab
Assignee: nobody → stridey
Status: NEW → ASSIGNED
(Assignee)

Comment 6

11 years ago
Created attachment 227916 [details]
Screenshot of links tab
(Assignee)

Comment 7

11 years ago
Created attachment 227917 [details]
Screenshot of advanced sheet
(Assignee)

Comment 8

11 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

11 years ago
Created attachment 227931 [details]
Minor IRC tweaks

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

11 years ago
Created attachment 227933 [details]
Sets min width/height
Attachment #227931 - Attachment is obsolete: true
Attachment #227933 - Flags: review?(alqahira)
Attachment #227931 - Flags: review?(alqahira)
(Assignee)

Comment 11

11 years ago
Created attachment 227936 [details]
Not b0rken

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

11 years ago
Created attachment 227938 [details]
Makes dropdown menu longer

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

11 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

11 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

11 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

11 years ago
Created attachment 228247 [details]
Screenshot of Colors & Links side

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

11 years ago
Created attachment 228248 [details]
Screenshot of Links side
Attachment #227916 - Attachment is obsolete: true
(Assignee)

Comment 21

11 years ago
Created attachment 228249 [details]
Screenshot of advanced sheet
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

11 years ago
Whiteboard: [needs checkin]

Comment 23

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

Updated

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

Updated

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

Updated

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