Rethink "exit reader" icon

VERIFIED FIXED in Firefox 24

Status

()

Firefox for Android
Theme and Visual Design
VERIFIED FIXED
5 years ago
2 years ago

People

(Reporter: ibarlow, Assigned: Margaret)

Tracking

unspecified
Firefox 24
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: ui-hackathon, reader)

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
It might make sense to revisit the "book with an X on it" treatment for pages in Reader Mode. Especially since long pressing the icon will now add the page to the user's reading list (see https://bugzilla.mozilla.org/show_bug.cgi?id=784387#c29)

The bug should be a simple image swap, assigning to myself so I don't forget.
(Reporter)

Updated

5 years ago
Summary: Rething "exit reader" icon → Rethink "exit reader" icon
(Reporter)

Updated

5 years ago
Whiteboard: ui-hackathon
(Reporter)

Updated

5 years ago
Whiteboard: ui-hackathon → ui-hackathon, reader
(Reporter)

Comment 1

5 years ago
Created attachment 749274 [details]
Mockup of new icon

Instead of a button that communicates "exit", let's treat this icon more like a toggle where it is grey when it is off, and orange when it is on.
(Reporter)

Comment 2

5 years ago
New icon resources for mdpi/hdpi/xhdpi http://cl.ly/2M050L2E3G1b
(Assignee)

Comment 3

5 years ago
Created attachment 749466 [details] [diff] [review]
patch
Assignee: ibarlow → margaret.leibovic
Attachment #749466 - Flags: review?(sriram)
Attachment #749466 - Flags: review?(sriram) → review+
https://hg.mozilla.org/mozilla-central/rev/fa1dc340708b
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24

Updated

5 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.