Closed Bug 957281 Opened 6 years ago Closed 5 years ago

Help button should be positioned at the lower end of the preferences window

Categories

(Toolkit :: Preferences, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla31

People

(Reporter: stefanh, Assigned: stefanh)

Details

Attachments

(1 file, 1 obsolete file)

We should position the help button at the lower end of the prefwindow when 'browser.preferences.instantApply' is set to true.

HIG:
"In dialogs (including preferences windows) and drawers, the Help button can be located in either the lower-left or lower-right corner. In a dialog that includes OK and Cancel buttons (or other buttons used to dismiss the dialog), the Help button should be in the lower-left corner, vertically aligned with the buttons. In a dialog that does not include OK and Cancel buttons, such as a preferences window, the Help button should be in the lower-right corner. For example, the Mail Fonts & Colors preference displays a Help button in the lower-right corner."
Attached patch Move the button (obsolete) — Splinter Review
Assignee: nobody → stefanh
Status: NEW → ASSIGNED
Attachment #8356743 - Flags: review?(mano)
Attachment #8356743 - Attachment description: 957281.1.diff → Move the button
I'll try to get this done this week.
Comment on attachment 8356743 [details] [diff] [review]
Move the button

I prefer hiding the spacer over moving nodes around.

document.getAnonymousElementByAttribute(this "anonid", "spacer").hidden = true

should work.
Attachment #8356743 - Flags: review?(mano)
(In reply to Mano from comment #3)
> document.getAnonymousElementByAttribute(this "anonid", "spacer").hidden =
> true
> 
> should work.

Yeah, but 'pack="end"' only applies to windows (see the ifdef starting at 576). I can of course set it when I hide the spacer - what do you think?
Flags: needinfo?(mano)
Yes, that's better. When we introduce platform difference in xbl widgets like this, it's almost always better to have the difference in the form of an attribute rather than in the tree structure.
Flags: needinfo?(mano)
Attached patch 957281.2.diffSplinter Review
I had a discussion with Neil about this and it's of course so that the pack attribute will only affect the layout of the buttons when the spacer is hidden. pack="end" was included in the ifdef in bug 309140, and the patch there also unhided the spacers.

So, I decided to revert the pack attribute changes made for 8 1/2 years ago :-)
Attachment #8356743 - Attachment is obsolete: true
Attachment #8393648 - Flags: review?(mano)
https://hg.mozilla.org/mozilla-central/rev/6d276863a7e3
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
QA Whiteboard: [good first verify]
You need to log in before you can comment on or make changes to this bug.