Closed
Bug 760187
Opened 11 years ago
Closed 11 years ago
Phone URL bar should have softer texture
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox15 verified, firefox16 verified, firefox17 verified)
VERIFIED
FIXED
Firefox 16
People
(Reporter: sriram, Assigned: sriram)
References
Details
Attachments
(1 file)
197.44 KB,
patch
|
mfinkle
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
As a part of bug 758804, a more softer texture was added to the backgrounds. The phone UI should have the resources replaced for the url-bar to feel in place.
Assignee | ||
Comment 1•11 years ago
|
||
This patch changes the URL bar resources to match the new texture. Few resources need change by ibarlow, as they have weird black edges. I'll push the patch with those changes.
Assignee: nobody → sriram
Attachment #628918 -
Flags: review?(mark.finkle)
Comment 2•11 years ago
|
||
Comment on attachment 628918 [details] [diff] [review] Patch I am OK with replacing the images, but let's try to make the textures as small as apossible
Attachment #628918 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Pushed after replacing the resources: One extra line changed in browser_toolbar to accommodate the perfect size of the plus button. http://hg.mozilla.org/integration/mozilla-inbound/rev/b7fb3d52c843
Updated•11 years ago
|
Target Milestone: --- → Firefox 16
Comment 4•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b7fb3d52c843
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 628918 [details] [diff] [review] Patch [Approval Request Comment] Bug caused by (feature/regressing bug #): - User impact if declined: There will be a visual clutter on phones due to difference in texture between the url-bar and about:home page. Testing completed (on m-c, etc.): 06/05 Risk to taking this patch (and alternatives if risky): None String or UUID changes made by this patch: -
Attachment #628918 -
Flags: approval-mozilla-aurora?
Comment 6•11 years ago
|
||
Comment on attachment 628918 [details] [diff] [review] Patch no risk, approved.
Attachment #628918 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 7•11 years ago
|
||
Pushed to aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/3b77c47e1c58
Updated•11 years ago
|
status-firefox15:
--- → fixed
Updated•11 years ago
|
Status: RESOLVED → VERIFIED
status-firefox16:
--- → verified
status-firefox17:
--- → verified
OS: Mac OS X → Android
Hardware: x86 → ARM
Updated•2 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•