Wrong styling for the onboarding tour footer links

VERIFIED FIXED in Firefox 41

Status

()

Firefox
New Tab Page
VERIFIED FIXED
3 years ago
2 years ago

People

(Reporter: cornel_ionce, Assigned: hoosteeno)

Tracking

41 Branch
Firefox 42
Points:
---

Firefox Tracking Flags

(firefox41 verified, firefox42 verified)

Details

Attachments

(3 attachments)

Affected builds:
- Firefox 40.0a2 Aurora, build ID: 20150528004000
- Firefox 41.0a1 Nightly, build ID: 20150527135446
Affected platforms: Windows 7 64-bit, Mac OS X 10.9.5 and Ubuntu 14.04 32-bit.

STR:
1. Open Firefox using a clean profile;
2. Open a newtab page;
3. Visit the "More about new Tab" or "About your privacy" links from the page tour.

Results:
The styling is wrong after the links are visited, the links become black.

Screenshot: http://i.imgur.com/mNHCPcM.png
Blocks: 1138818
Ed, this issue should be resolved by bug 1172987. Would it make sense to uplift those fixes to Aurora to fix this?
Flags: needinfo?(edilee)

Comment 2

3 years ago
The black color doesn't seem to be fixed on Nightly or Aurora by bug 1172987. That bug has been uplifted.
Flags: needinfo?(edilee)
Hi, I'd like to give this bug a shot. Quick questions:

1) What is the _expected_ color of visited links? Same as un-visited links?
2) In my freshly built copy of nightly, the link to "Privacy Notice" seems to be browser-default blue, rather than #4A90E2 (the blue color of other un-visited links in the intro). When visited, it is browser-default purple. It's also underlined. Shall I apply the same link styles to inline links throughout?
(In reply to Justin Crawford [:hoosteeno] [:jcrawford] from comment #3)
> Hi, I'd like to give this bug a shot. Quick questions:
> 
> 1) What is the _expected_ color of visited links? Same as un-visited links?

The spec for the UX here is on page 12 of this doc: https://bug1150228.bugzilla.mozilla.org/attachment.cgi?id=8592387

> 2) In my freshly built copy of nightly, the link to "Privacy Notice" seems
> to be browser-default blue, rather than #4A90E2 (the blue color of other
> un-visited links in the intro). When visited, it is browser-default purple.
> It's also underlined. Shall I apply the same link styles to inline links
> throughout?

The links this bug is talking about is the footer links "More about new Tab" or "About your privacy".

The spec says the colour should be #171F26 for visited links which is what it is now...

Aaron, is this correct or should the colours of the footer links be different?
Flags: needinfo?(athornburgh)
Created attachment 8634165 [details]
Screen Shot 2015-07-14 at 3.49.49 PM.png


> > 2) In my freshly built copy of nightly, the link to "Privacy Notice" seems
> > to be browser-default blue, rather than #4A90E2 (the blue color of other
> > un-visited links in the intro). When visited, it is browser-default purple.
> > It's also underlined. Shall I apply the same link styles to inline links
> > throughout?
> 
> The links this bug is talking about is the footer links "More about new Tab"
> or "About your privacy".

Got it. Shall I open another bug for the link styling issues in the paragraph text (see attached)? Or, since the footer may not be an issue, we could repurpose this bug.

Comment 6

2 years ago
Marina, the Style Guide is incorrect...

The STATIC color of the footer links is #4890E2 (blue, not green), and the ROLLOVER color is #000000 (black).

My apologies, not sure how that happened.
Flags: needinfo?(athornburgh)
> Got it. Shall I open another bug for the link styling issues in the
> paragraph text (see attached)?

Yes please do open another bug for this separate issue and go based on Aaron's comment 6 for this bug.
Created attachment 8635564 [details] [diff] [review]
bug1169241.patch

This is my first Firefox & hg patch submission ever. Please let me know if I have made obvious newbie mistakes.
Attachment #8635564 - Flags: feedback?
Comment on attachment 8635564 [details] [diff] [review]
bug1169241.patch

Review of attachment 8635564 [details] [diff] [review]:
-----------------------------------------------------------------

W00t Congrats on your first patch :)
Attachment #8635564 - Flags: feedback? → review+
Thanks!
Keywords: checkin-needed
Hi, this failed to apply, 

Fetching... done
Parsing... done
adding 1169241 to series file
renamed 1169241 -> bug1169241.patch
applying bug1169241.patch
patching file browser/base/content/newtab/newTab.css
Hunk #1 FAILED at 758
1 out of 1 hunks FAILED -- saving rejects to file browser/base/content/newtab/newTab.css.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working directory
errors during apply, please fix and refresh bug1169241.patch

could you take a look, maybe needs a rebase? Thanks!
Flags: needinfo?(hoosteeno)

Updated

2 years ago
Keywords: checkin-needed
Created attachment 8636770 [details] [diff] [review]
Add hover style to footer links in newtab intro
Assignee: nobody → hoosteeno
Status: NEW → ASSIGNED
Comment on attachment 8636770 [details] [diff] [review]
Add hover style to footer links in newtab intro

The change required was different as a result of intervening changes, so I have a new patch for review.
Flags: needinfo?(hoosteeno)
Attachment #8636770 - Flags: review?(msamuel)

Updated

2 years ago
Attachment #8636770 - Flags: review?(msamuel) → review+
Thanks!
Keywords: checkin-needed
that worked perfect Justin, thanks! and welcome to mozilla!

Comment 16

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/7ecd5948ed30
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/7ecd5948ed30
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox42: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
I found same issue depends on this bug on Firefox Nightly -- windows 8.1(32bit)

Build ID 	20150529030205
User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:41.0) Gecko/20100101 Firefox/41.0


It's okay on Latest Developer Edition & Latest Nightly

Developer Edition

Build ID 	20150805004014
User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:41.0) Gecko/20100101 Firefox/41.0

Latest Nightly

Build ID 	20150805030208
User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:42.0) Gecko/20100101 Firefox/42.0

[bugday-20150805]

Updated

2 years ago
status-firefox41: affected → fixed
OS: All → Windows 8.1
Version: Trunk → 41 Branch

Updated

2 years ago
OS: Windows 8.1 → All
It's also found on

Firefox nightly

Build ID 	20150528030206
User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:41.0) Gecko/20100101 Firefox/41.0

[bugday-20150805]
This is now indeed fixed now using latest builds.
Status: RESOLVED → VERIFIED
status-firefox41: fixed → verified
status-firefox42: fixed → verified
You need to log in before you can comment on or make changes to this bug.