Change url_bar_entry_pressed outline color to action orange

RESOLVED FIXED in Firefox 40

Status

()

Firefox for Android
Theme and Visual Design
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mcomella, Assigned: anirudh24seven, Mentored)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 40
All
Android
Points:
---

Firefox Tracking Flags

(firefox40 fixed)

Details

(Whiteboard: [good first bug])

Attachments

(4 attachments, 2 obsolete attachments)

Per the new color palette, assuming the name does not change.

i.e. this icon [1]. Unknown if this also affects tablet.

NI :antlam for new assets.

[1]: https://mxr.mozilla.org/mozilla-central/source/mobile/android/base/resources/drawable-xhdpi/url_bar_entry_pressed.9.png
Flags: needinfo?(alam)
Created attachment 8568223 [details]
urlbar_active_9patch_update.zip

Updated orange to #FF9500
Flags: needinfo?(alam)
Created attachment 8586463 [details]
urlbar_active.tar.gz

Compressed assets.
Attachment #8568223 - Attachment is obsolete: true
Your task: replace the assets in mobile/android/base/resources/drawable-*dpi/url_bar_entry_pressed.9.png with the resources in the attached tar ball. Then, make sure the url bar, when pressed, looks the same before and after the patch is applied with the exception that the color of the url bar will change to a slightly different orange hue.

To start, set up a build environment - you can see the instructions here: https://wiki.mozilla.org/Mobile/Fennec/Android

Then, you'll need to create a patch to upload - see
https://wiki.mozilla.org/Mobile/Fennec/Android#Creating_commits_and_submitting_patches

If you need any help, you can reply to this bug, or feel free to message me on IRC - my nick is "mcomella" and you can find me in #mobile. If you need IRC setup instructions, see https://wiki.mozilla.org/IRC

Thanks and happy coding! ^_^
Whiteboard: [good first bug]
Mentor: michael.l.comella, mhaigh
(Assignee)

Comment 4

3 years ago
Created attachment 8597018 [details] [diff] [review]
Change url_bar_entry_pressed outline color to action orange
(Assignee)

Comment 5

3 years ago
Created attachment 8597020 [details] [diff] [review]
Change url_bar_entry_pressed outline color to action orange
(Assignee)

Updated

3 years ago
Attachment #8597018 - Attachment is obsolete: true
(Assignee)

Comment 6

3 years ago
I have replaced the following configurations:

- hdpi
- mdpi
- xhdpi
- xxhdpi


However, I think there are other configurations for the same file for which no assets have been provided:

- large-hdpi-v11
- large-mdpi-v11
- large-xhdpi-v11
- large-xhdpi-v11

The existing assets for these configurations seem different. Therefore, I think we need new assets for them too.
Flags: needinfo?(michael.l.comella)
(Assignee)

Comment 7

3 years ago
Created attachment 8597043 [details]
Before Screenshot

Screenshot of the Address Bar border on a Nexus 5 before this bug fix.
(Assignee)

Comment 8

3 years ago
Created attachment 8597044 [details]
After Screenshot

Screenshot of the Address Bar border on a Nexus 5 after this bug fix.
Hey, Anirudh!

Sorry for the delay - I'll do my best to make sure it doesn't happen again. Nice digging on the tablet files. However, it looks like the tablet assets have already been updated (perhaps back when we did the new tablet refresh in bug 1014156?) so no worries there - I'll review your patch.

If you're curious how I checked, I opened up the resource locally and used the gcolor2 application to check the color of the border in the tablet asset.
Flags: needinfo?(michael.l.comella)
Assignee: nobody → anirudh24seven
I made a push to our try test servers (see above).

Once it goes green, feel free to add the checkin-needed keyword [1]. Let me know if you need help reading the results. Note that all patches that use "checkin-needed" must also have an associated green try run.

[1]: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch#Getting_the_patch_checked_into_the_tree
Comment on attachment 8597020 [details] [diff] [review]
Change url_bar_entry_pressed outline color to action orange

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

Looks good to me!
Attachment #8597020 - Flags: review+
(Assignee)

Updated

3 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/94ca3cad5caa
Whiteboard: [good first bug] → [good first bug][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/94ca3cad5caa
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Whiteboard: [good first bug][fixed-in-fx-team] → [good first bug]
Target Milestone: --- → Firefox 40
You need to log in before you can comment on or make changes to this bug.