Reader Mode: Give visited links a different colour

VERIFIED FIXED in Firefox 51

Status

()

Toolkit
Reader Mode
P2
enhancement
VERIFIED FIXED
2 years ago
4 months ago

People

(Reporter: Gerhard Großmann, Assigned: rakhisharma)

Tracking

(Blocks: 1 bug)

38 Branch
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 verified)

Details

(Whiteboard: [about-reader-ui])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0
Build ID: 20150511103818

Steps to reproduce:

For some websites FF offers a reader mode since Version 38.0.5.


Actual results:

However the CSS for this mode lacks to differentiate visited and not yet visited links.


Expected results:

See http://joelcalifa.com/blog/revisiting-visited/ why distinctive colours help the user to navigate.

Updated

2 years ago
Component: Untriaged → Reader Mode
Product: Firefox → Toolkit

Updated

10 months ago
Severity: normal → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P2

Updated

10 months ago
Whiteboard: [about-reader-ui]

Updated

10 months ago
Blocks: 1286221

Updated

10 months ago
Assignee: nobody → Rakhish1994
Comment hidden (mozreview-request)
(Assignee)

Comment 2

9 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=38e15653f104

Comment 3

9 months ago
mozreview-review
Comment on attachment 8782389 [details]
Bug 1170955 - Reader Mode: Give visited links a different color. .

https://reviewboard.mozilla.org/r/72574/#review70210

::: toolkit/themes/shared/aboutReaderContent.css:88
(Diff revision 1)
>    text-decoration: underline;
>    font-weight: normal;
>  }
>  
>  a,
> -a:visited,
> +a:link,

I think we should just remove the :visited line, and probably make sure all 3 remaining ones include ":link". Please do the same for the rule above (the one with text-decoration: underline and font-weight: normal). Right now anchors that are not links will also get underlines and get colored, which is not right.

::: toolkit/themes/shared/aboutReaderContent.css:94
(Diff revision 1)
> +a:visited {
> +  color: #609;
> +}

This color does not work well with the dark theme, maybe let's use #c2e which is a bit lighter, so there's more contrast? It still has >3:1 contrast with the light and sepia theme as well, so I think that works.
Attachment #8782389 - Flags: review?(gijskruitbosch+bugs)
Comment hidden (mozreview-request)
(Assignee)

Comment 5

9 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=672fdd25cee7

Comment 6

9 months ago
mozreview-review
Comment on attachment 8782389 [details]
Bug 1170955 - Reader Mode: Give visited links a different color. .

https://reviewboard.mozilla.org/r/72574/#review70226

Looks good to me, thanks!
Attachment #8782389 - Flags: review?(gijskruitbosch+bugs) → review+

Comment 7

9 months ago
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/33584a021edd
Reader Mode: Give visited links a different color. r=Gijs.

Comment 8

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/33584a021edd
Status: NEW → RESOLVED
Last Resolved: 9 months ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51

Updated

7 months ago
Duplicate of this bug: 1240035
QA Whiteboard: [good first verify]
The Reader Mode gives visited links a different colour.
Status: RESOLVED → VERIFIED
QA Whiteboard: [good first verify] → [good first verify][bugday-2017-01-06]
status-firefox51: fixed → verified
You need to log in before you can comment on or make changes to this bug.