Closed
Bug 1194388
Opened 11 years ago
Closed 11 years ago
Navigation toolbar and URL bar style change on tab focus
Categories
(Firefox for iOS :: Browser, defect)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fxios | + | --- |
People
(Reporter: aaronmt, Assigned: sleroux)
Details
(Keywords: reproducible)
Attachments
(1 file)
|
47 bytes,
text/x-github-pull-request
|
bnicholson
:
review+
bmunar
:
review+
dhenein
:
ui-review+
|
Details | Review |
Currently the navigation toolbar, on tab focus, will flicker as it looks like it's style or transparency etc changes.
Visit http://mozilla.org
Tap the tab switcher button
Visit the same tab again
See the nav-toolbar swap from a solid grey to it's translucent flavour afterwards
Updated•11 years ago
|
Component: General → Browser
Updated•11 years ago
|
Summary: Navigation toolbar style change on tab focus → Navigation toolbar and URL bar style change on tab focus
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → sleroux
| Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8648235 -
Flags: review?(bnicholson)
| Assignee | ||
Updated•11 years ago
|
Attachment #8648235 -
Flags: review?(bmunar)
| Assignee | ||
Comment 2•11 years ago
|
||
Adding the B-boys for review and Darrin for UX feedback
| Assignee | ||
Updated•11 years ago
|
Attachment #8648235 -
Flags: ui-review?(dhenein)
Comment 3•11 years ago
|
||
B-boyss!!
Comment 4•11 years ago
|
||
Comment on attachment 8648235 [details] [review]
https://github.com/mozilla/firefox-ios/pull/935
Seems simple enough to me.
Attachment #8648235 -
Flags: review?(bnicholson) → review+
Updated•11 years ago
|
Attachment #8648235 -
Flags: review?(bmunar) → review+
Updated•11 years ago
|
Status: NEW → ASSIGNED
Comment 6•11 years ago
|
||
Comment on attachment 8648235 [details] [review]
https://github.com/mozilla/firefox-ios/pull/935
LGTM. Still see the flicker but I suppose for most pages, white is better than black.
Attachment #8648235 -
Flags: ui-review?(dhenein) → ui-review+
| Assignee | ||
Comment 7•11 years ago
|
||
Merged
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•