Closed
Bug 1207545
Opened 9 years ago
Closed 9 years ago
Inconsistency between the steps from tracking protection UITour
Categories
(Firefox :: Tours, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: phorea, Assigned: agibson)
Details
(Whiteboard: [kb=1856193])
Attachments
(1 file)
(deleted),
image/png
|
Details |
Reproduced with Firefox 42 beta 1, latest Aurora 43.0a2 2015-09-23 and latest Nightly 44.0a1 2015-09-22 under Win 7 64-bit, Ubuntu 14.04 and Mac OS X 10.9.5. STR: 1. Open a private window and select "See how this works" button to start the tracking protection tour - https://www.mozilla.org/en-US/firefox/44.0a1/tracking-protection/start/ 2. Notice the styling of the first step and then select Next to go to the second one ER: All the three steps from the tour have the same styling AR: Step 2 is different from steps 1 and 3: - different font used (eg for Win: Helvetica (bold) instead of Lucinda Grande (bold)) - the Next button is slightly smaller - Next text has a different (or smaller) font - X button to close the tooltip has a different style and background icon
Assignee | ||
Comment 1•9 years ago
|
||
Thanks for filing this, The info panel on step 2 of the TP tour isn't in fact a real XUL panel. We can't target them on content in the web page itself, so for step 2 we recreated one using HTML/CSS. This is why they look a bit different. I can work on this and try and make it as close as possible. I'm comforted by the fact this was down as blocking Bug 1207419, which shows we must be at least close :)
No longer blocks: 1207419
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → agibson
Status: NEW → ASSIGNED
Whiteboard: [kb=1856193]
Assignee | ||
Comment 2•9 years ago
|
||
Matt, a few styling questions for the info panel on step 2: 1.) I'm currently using `font: -moz-document;` for the info panel. Should I be using something else to make sure it matches the real XUL panels? `font: message-box` perhaps? 2.) Can you help point me to the CSS for the actual panels, so I can try and match things as close as possible. 3.) I'm also trying to locate the image(s) used for the beveled (x) close button. Thanks!
Flags: needinfo?(MattN+bmo)
Comment 3•9 years ago
|
||
Removing these bugs as blockers for the release of the Tracking Protection UITour for the Fall. Any UITour position-related bugs should be tied to bug 1207419.
No longer blocks: 1178805
Updated•9 years ago
|
Comment 4•9 years ago
|
||
As discussed today: https://mxr.mozilla.org/mozilla-central/source/browser/themes/shared/UITour.inc.css https://mxr.mozilla.org/mozilla-central/search?string=UITourTooltip&find=\.css&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central plus using the devtools on the browser are useful. Images are from: https://mxr.mozilla.org/mozilla-central/find?string=icons%2Fclose.png&tree=mozilla-central&hint=
Flags: needinfo?(MattN+bmo)
Assignee | ||
Comment 5•9 years ago
|
||
Closing this, as was fixed in Bug 1209429. We added separate button styles for Windows, OSX and Linux for the step 2 door-hanger, as per Comment 4. It may not be pixel-perfectly identical, but we agreed it wasn't worth spending too much time on.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•