Closed
Bug 894473
Opened 11 years ago
Closed 4 years ago
Support using LWT accent color as background color in more of the UI
Categories
(Firefox for Android Graveyard :: Theme and Visual Design, defect, P3)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: wesj, Unassigned)
References
Details
Attachments
(9 files)
New about:home doesn't show themes over the top of it anymore. Maybe we have to do that because we're showing lists now? but it would be nice if we still have the nice big image...
Reporter | ||
Updated•11 years ago
|
Blocks: new-about-home
Comment 1•11 years ago
|
||
There are no designs for this yet. Ideally, changing the layouts from standard android to GeckoXYZView would show the personas.
Updated•11 years ago
|
OS: Linux → Android
Hardware: x86_64 → All
Updated•11 years ago
|
Summary: Support personas on the new about:home → Support personas / lightweight themes on the new about:home
Comment 2•11 years ago
|
||
What will users see if they have any installed persona's and update to 26?
Comment 3•11 years ago
|
||
(In reply to Aaron Train [:aaronmt] from comment #2)
> What will users see if they have any installed persona's and update to 26?
I guess they'll see the theme applied to the toolbar only?
Comment 4•11 years ago
|
||
Yes. The theme will be on the toolbar only.
Comment 5•10 years ago
|
||
I'm not sure how better theme intergration would look with our title strip, but we used to do this [1] with our old about:home, and I think sriram made some progress using the theme background in the browser menu.
Any thoughts from UX?
[1] http://cl.ly/image/3j3L091C391E
Flags: needinfo?(alam)
Comment 6•10 years ago
|
||
Ah, this is interesting! Are we looking at just this page and just the tool bar? I'm just wondering where else these themes surface themselves.
Flags: needinfo?(alam)
Comment 7•10 years ago
|
||
I feel like this should be a WONTFIX. Leaving the call to antlam.
Flags: needinfo?(alam)
Comment 8•10 years ago
|
||
Will leave NI on for this one as I mull it over a bit more but I'm leaning towards a WONTFIX too.
The only + side I see is having a more "personalized" about:home? but I'm not sure that's best achieved by just exposing the theme's image in the toolbar on about:home.
Comment 9•10 years ago
|
||
One thing I wouldn't mind trying to do is use the "accent color" provided in a LWT to change the color of our UI in many more places. We have plans for using LWTs in our onboarding process soon, and adding the accent color to more of the UI would go a long way. Using the color is a lot easier than using a bitmap too.
Updated•10 years ago
|
Blocks: lwt-improvements
Updated•10 years ago
|
Summary: Support personas / lightweight themes on the new about:home → Support using LWT accent color as background color in more of the UI
Comment 10•10 years ago
|
||
We could try using the color (or a lighter version) as the background color:
* the "tab strip" in the HomePager
* the pages themselves
* the URLBar textview
We could try using a darker version of the color as the background color:
* Tabs tray
It seems the tabs tray is already tinted with the predominant color, just not too heavily and many themes seem to use black, which does not change the grey background in any extreme way. See [1] for the associated code.
[1]: https://mxr.mozilla.org/mozilla-central/source/mobile/android/base/LightweightTheme.java?rev=0dc0ff9bb484#425
For comparison's sake...
Another idea: tinting the system status bar with the predominant theme color (the color change to the URL bar at the moment can look a bit strange, depending on the theme).
Playing around a bit.
Assignee: nobody → michael.l.comella
Status: NEW → ASSIGNED
After trying a few values, I tried one that's a bit darker. I'll attach followup screenshots with issues, however.
(In reply to Michael Comella (:mcomella) from comment #15)
> After trying a few values, I tried one that's a bit darker.
I meant that I found an alpha value that's a bit more extreme that was my favorite of those tried.
The font here is too dark - we can steal the tab name font color and maybe lighten the blue for the link (I think all themes should get darker as they filter over the default background color).
The tabs tray background is not the same as the space to the right of the toolbar and the lack of transition there looks pretty crappy. I think we should use the same technique there as we do for non-private browsing (but keep the toolbar the darker solid color).
Note that the issue in comment 17 can be swept under the rug if we don't make the background color as extreme. For the record, I used 88 for the alpha value in the screenshots, but an alpha value of 55 or lower would also work.
This is as extreme as we can get without dark colors (e.g. black) causing the gray sub-text to be difficult to read.
If we were more thorough here, we have the, "Is this theme dark?" information already so we can change the font colors too (at the price of code and theming complexity).
Comment 21•10 years ago
|
||
Nice experiments here Michael! With the new toolbar's stroke around the URL bar, it looks like we can offer a bit more in terms of what a LWT entails.
(In reply to Mark Finkle (:mfinkle) from comment #9)
> One thing I wouldn't mind trying to do is use the "accent color" provided in
> a LWT to change the color of our UI in many more places.
I'd agree. When I first submitted a theme, I expected the "accent" color to do more.
Although, I'm not so sure about using it for something as dominant as a "background color", even just as a tint. I wonder how it would look if we used the accent color for maybe changing the color of the buttons or the underline/indicator? (would we have to prep the assets of each icon of a different color?)
Flags: needinfo?(alam)
Keep in mind I still have the background color change.
This is a tint indicator that is hard to see - what do you think, Anthony? Keep in mind this color also needs to mesh well with the tabs tray background if it's also the selector in the tabs tray and highlight on the thumbnails.
Attachment #8506509 -
Flags: feedback?(alam)
Note that the tabs tray selector is an image resource, which we can set to the color of our choosing using color filters (but perhaps the base asset should be white, rather than orange and we color with that). As for the thumbnail highlight, I'm not sure - I was unable to find where this color was set, but presumably we can use a similar technique to either one here.
Updated•10 years ago
|
Priority: -- → P3
Comment 25•10 years ago
|
||
Comment on attachment 8506509 [details]
(v1) tinted home pager indicator light
Yeah that's kind of weird, but it does show the idea off. (+ for that!)
I'm not sure the change in background is necessary, maybe just the underline? That way, users could use bolder colors (e.g. orange underline is great, but if the background was orange that might not be so great).
I wonder how/if this affects some of the ideas we had around themes when we were in London (for Tablets).
Maybe we should hold off on this and see how it affects tablet's first.
Thoughts?
Flags: needinfo?(michael.l.comella)
Attachment #8506509 -
Flags: feedback?(alam) → feedback+
(In reply to Anthony Lam (:antlam) from comment #25)
> Maybe we should hold off on this and see how it affects tablet's first.
I would agree, at least until bug 1085771 lands.
Depends on: 1085771
Flags: needinfo?(michael.l.comella)
Not a priority so I'm going to unassign. Note that I lost the patches when my hard drive died. Also, changing the tab indicator light for PB mode may be desirable (like bug 1177611).
Assignee: michael.l.comella → nobody
Status: ASSIGNED → NEW
Comment 28•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Updated•4 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
•