Closed
Bug 1064867
Opened 10 years ago
Closed 10 years ago
Light-weight theme for 'Firefox Privacy Coach'
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(fennec+)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fennec | + | --- |
People
(Reporter: krudnitski, Assigned: Margaret)
References
Details
Attachments
(1 file, 1 obsolete file)
47.98 KB,
image/png
|
Details |
Anthony - can you design a light-weight theme (using current theme parameters) for Firefox Confidential? Extending beyond a light-weight theme will be logged in another bug.
Updated•10 years ago
|
tracking-fennec: ? → 33+
Updated•10 years ago
|
tracking-fennec: 33+ → +
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(alam)
Comment 1•10 years ago
|
||
Thanks for the NI on this - will take a look at something simple for 33.
Comment 2•10 years ago
|
||
Just to clarify, are we talking about a simple theme just like these ones? (i.e. we only need an image + a "footer image") https://addons.mozilla.org/en-US/firefox/user/Antlam/themes
Flags: needinfo?(margaret.leibovic)
Assignee | ||
Comment 3•10 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #2) > Just to clarify, are we talking about a simple theme just like these ones? > (i.e. we only need an image + a "footer image") > > https://addons.mozilla.org/en-US/firefox/user/Antlam/themes Yup. I don't think we even need a footer image for Fennec, unless that's required to upload one to AMO.
Flags: needinfo?(margaret.leibovic)
Comment 4•10 years ago
|
||
Updating the assets on my "13fox" theme to play better on mobile. Try it out on mobile to see what you guys think. The color palette (as some of you may recognize) comes from the full on, heavy weight, UI overhaul, "blackfox" theme I drew up originally. https://addons.mozilla.org/en-US/firefox/addon/13fox/ Would this be enough for the scope of this bug? I feel like this is a good start.
Flags: needinfo?(alam)
Updated•10 years ago
|
Summary: Light-weight theme for 'Firefox Confidential' → Light-weight theme for 'Firefox Privacy Coach'
Comment 5•10 years ago
|
||
It seems the 13fox theme doesn't play perfectly well with some top add-ons like ABP: The down-button for a sub menu isn't visible anymore, see screenshot attached (it might need zooming in to see). What can we do about this?
Flags: needinfo?(alam)
Comment 6•10 years ago
|
||
(In reply to Frederik Braun [:freddyb] from comment #5) > Created attachment 8497400 [details] > Selection_032.png > > It seems the 13fox theme doesn't play perfectly well with some top add-ons > like ABP: The down-button for a sub menu isn't visible anymore, see > screenshot attached (it might need zooming in to see). > > What can we do about this? Hmm.. I see what you're saying. We could try a grey that's less dark but from what I know we try to consider dark themes when we design for this type of function. Is this desktop that you're testing this on?
Flags: needinfo?(alam)
Comment 7•10 years ago
|
||
Yes, this is Nightly on desktop, ubuntu 14.04 linux 64bit (most recent version).
Comment 8•10 years ago
|
||
It looks like the theme adds a stroke around the FF icons (albeit a bit thick), I wonder if there's a way to force it on the Add-on icons so they pop?
Comment 9•10 years ago
|
||
(In reply to Robin Andersen [:tecgirl] from comment #8) > It looks like the theme adds a stroke around the FF icons (albeit a bit > thick), I wonder if there's a way to force it on the Add-on icons so they > pop? It's not really worth comparing Desktop theme add-ons and Fennec themes. Desktop add-ons can change icons, colors and many other things. Not possible in Fennec. Lightweight themes in Fennec are simply: 1. 3000px x 200px header image: displayed top/right aligned in the browser toolbar and tabs tray 2. 3000px x 100px footer image: unused 3. accent/background color: used to determine if the theme image is mostly dark or mostly light 4. text color: unused Fennec uses #3 to determine which set of light/dark images should be used in the application. It's not perfect.
Assignee | ||
Comment 10•10 years ago
|
||
I'll look into the technical bit of how to get the add-on to install a lightweight theme.
Assignee: nobody → margaret.leibovic
Assignee | ||
Comment 11•10 years ago
|
||
It's not really obvious that this is a lightweight theme, but maybe that's a good thing? The curve isn't visible at all, I wonder if that would confuse users.
Assignee | ||
Comment 12•10 years ago
|
||
Comment on attachment 8497400 [details]
Selection_032.png
This bug is just about Firefox for android, so this desktop screenshot doesn't affect things.
Attachment #8497400 -
Attachment is obsolete: true
Assignee | ||
Comment 13•10 years ago
|
||
The theme is included in the add-on here: https://github.com/leibovic/privacy-coach/blob/master/bootstrap.js#L65 I tried using local files, but LightweightThemeManager doesn't allow chrome:// URLs, so I would need to figure out how to add resource:// or file:// URLs in my add-on. mfinkle, do you think it's worth pursuing that, or is it fine to just stick with https URLs?
Flags: needinfo?(mark.finkle)
Comment 14•10 years ago
|
||
https://developer.mozilla.org/en-US/docs/Chrome_Registration#resource Might be as simple as resource privacy-coach content/ in your chrome.manifest. Anything we can do to improve a user's first experience with these seems worth a few minutes of investigation, IMO.
Assignee | ||
Comment 15•10 years ago
|
||
(In reply to Richard Newman [:rnewman] from comment #14) > https://developer.mozilla.org/en-US/docs/Chrome_Registration#resource > > Might be as simple as > > resource privacy-coach content/ > > in your chrome.manifest. > > Anything we can do to improve a user's first experience with these seems > worth a few minutes of investigation, IMO. Ah, yeah, that's not hard to do at all :) Unfortunately, though, it looks like we're not equipped to make lightweight themes from resource:// URIs: W/GeckoBitmapUtils(27074): decodeUrl: malformed URL resource://privacycoach/header.png I also tried using resolveGeckoURI to resolve a chrome URI, but of course LightweightThemeManager also doesn't like jar: URIs. So, given this is an add-on we intend to make compatible with Fx33, I don't think we have very many options here.
Comment 16•10 years ago
|
||
(In reply to :Margaret Leibovic from comment #15) > Unfortunately, though, it looks like we're not equipped to make lightweight > themes from resource:// URIs: > > W/GeckoBitmapUtils(27074): decodeUrl: malformed URL > resource://privacycoach/header.png > > I also tried using resolveGeckoURI to resolve a chrome URI, but of course > LightweightThemeManager also doesn't like jar: URIs. Worth filing bugs so we can work on some low priority fixes. > So, given this is an add-on we intend to make compatible with Fx33, I don't > think we have very many options here. We can host the Lightweight theme on AMO (or use an existing one, like you're doing) and use the just use the HTTPS link in the add-on.
Flags: needinfo?(mark.finkle)
Assignee | ||
Comment 17•10 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #16) > (In reply to :Margaret Leibovic from comment #15) > > > Unfortunately, though, it looks like we're not equipped to make lightweight > > themes from resource:// URIs: > > > > W/GeckoBitmapUtils(27074): decodeUrl: malformed URL > > resource://privacycoach/header.png > > > > I also tried using resolveGeckoURI to resolve a chrome URI, but of course > > LightweightThemeManager also doesn't like jar: URIs. > > Worth filing bugs so we can work on some low priority fixes. We would probably need to change LightweightThemeManager to allow jar: URIs, since I don't think our Java code would know what to do with a Gecko resource:// URI (why we use this resolveGeckoURI helper method before passing things to Java). > > So, given this is an add-on we intend to make compatible with Fx33, I don't > > think we have very many options here. > > We can host the Lightweight theme on AMO (or use an existing one, like > you're doing) and use the just use the HTTPS link in the add-on. Yes, I was implying that we don't have options beyond what I'm currently doing. You're going to need network to download the add-on, so I think it's fine to require some network to download the theme. Marking this bug as fixed since the add-on supports a theme. We can change the theme there easily if we want to use something else instead of 13fox.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 18•10 years ago
|
||
Anthony, can we get a description or change title to reflect some sort of connection to Privacy Coach or privacy for the 13fox theme? The theme will show up as its own Add-on and may confuse users if they don't read the title on the landing page.
Flags: needinfo?(alam)
Comment 19•10 years ago
|
||
(In reply to Robin Andersen [:tecgirl] from comment #18) > Anthony, can we get a description or change title to reflect some sort of > connection to Privacy Coach or privacy for the 13fox theme? The theme will > show up as its own Add-on and may confuse users if they don't read the title > on the landing page. Ah, gotcha. Just changed it. Not sure if Marketing or anyone would like to fill in a description as well for this but I left it blank for now.
Flags: needinfo?(alam)
Updated•3 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
•