Closed
Bug 1055341
Opened 10 years ago
Closed 10 years ago
Sync preferences pane has orphaned misplaced help/[?] image/link
Categories
(Firefox :: Settings UI, defect)
Firefox
Settings UI
Tracking
()
VERIFIED
FIXED
Iteration:
39.1 - 9 Mar
People
(Reporter: markh, Unassigned)
References
Details
(Whiteboard: [fixed by bug 1136670])
Attachments
(1 file)
(deleted),
image/png
|
Details |
See attached image - there is a stray [?] image way to the right and bottom of the pane. It ends up opening https://support.mozilla.org/en-US/kb/how-do-i-set-up-firefox-sync?redirectlocale=en-US&as=u&redirectslug=How+to+sync+Firefox+settings+between+computers&utm_source=inproduct
Updated•10 years ago
|
Flags: firefox-backlog+
Comment 1•10 years ago
|
||
From in-content preference triage today: Removing this as a blocker for shipping in-content prefs. We should fix it, but it is not going to block us from shipping.
Updated•10 years ago
|
Updated•10 years ago
|
Summary: Sync preferences pane has orphaned misplaced [?] image/link → Sync preferences pane has orphaned misplaced help/[?] image/link
Comment 2•10 years ago
|
||
Marking as good first bug candidate. I wasn't able to figure out the problem after poking at it with the devtools a bit (my XUL is rusty). Jared, have any idea how this should be solved?
Flags: needinfo?(jaws)
Whiteboard: [gfbc]
Comment 3•10 years ago
|
||
When Mark used the words "stray", I believe he was describing how far away the Help button is from the rest of the UI. Unfortunately, there isn't an easy solution to this problem, as the Sync pane uses a <xul:deck> to display the relevant sync-preferences view. <xul:deck>'s are displayed at the size of their largest pane, and in the case of a user who doesn't have Sync configured, this leaves a lot of whitespace at the bottom.
It is possible that we could ditch the <xul:deck> and just go with setting .hidden=true on the <xul:vbox> children of #weavePrefsDeck. Blair, what do you think about doing that? If we go this route, we can add steps to make this a good-first-bug.
Flags: needinfo?(jaws) → needinfo?(bmcbride)
Comment 4•10 years ago
|
||
Yeah, I'd figured out what he meant but I wasn't able to debug why it happened (didn't know the behavior of <deck>.
I think replacing it with a a JS solution sounds good. Alternatively, we can make multiple copies of the help button (I don't like this solution though) for each vbox.
Comment 5•10 years ago
|
||
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #3)
> It is possible that we could ditch the <xul:deck> and just go with setting
> .hidden=true on the <xul:vbox> children of #weavePrefsDeck. Blair, what do
> you think about doing that? If we go this route, we can add steps to make
> this a good-first-bug.
Yes please. We need to move away from using a deck regardless of this bug - it forces a design/flow that doesn't fit with in-content prefs.
Flags: needinfo?(bmcbride)
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [gfbc] → [fixed by bug 1136670]
Updated•10 years ago
|
Iteration: --- → 39.1 - 9 Mar
Flags: qe-verify?
Updated•10 years ago
|
Flags: qe-verify? → qe-verify+
QA Contact: camelia.badau
Comment 6•10 years ago
|
||
Verified fixed on Windows 7 64bit, Ubuntu 13.10 32bit and Mac OSX 10.9.5 using latest Nightly 39.0a1 (buildID: 20150309030224).
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•