question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Expose more properties for theme install

See original GitHub issue

For themes, we only expose a fraction of properties compared to what we used to in the old frontend.

Example: https://addons.mozilla.org/en-US/firefox/addon/pixel-bears/ , see property data-browsertheme of the install button.

For QA: please test for regressions related to all theme functionality. This includes theme detail pages, previewing, and installation in both AMO and the Discovery Pane.

Old frontend:

"accentcolor": "#"
"author": "Star_Shine"
"category": "Other"
"description": ""
"detailURL": "https://addons.mozilla.org/en-US/firefox/addon/pixel-bears/"
"footer": "https://addons.cdn.mozilla.net/user-media/addons/357219/2.jpg?1326075809"
"footerURL": "https://addons.cdn.mozilla.net/user-media/addons/357219/2.jpg?1326075809"
"header": "https://addons.cdn.mozilla.net/user-media/addons/357219/Myhellokitty.jpg?1326075809"
"headerURL": "https://addons.cdn.mozilla.net/user-media/addons/357219/Myhellokitty.jpg?1326075809"
"iconURL": "https://addons.cdn.mozilla.net/user-media/addons/357219/preview_small.jpg?1326075809"
"id": "357219"
"name": "Pixel bears"
"previewURL": "https://addons.cdn.mozilla.net/user-media/addons/357219/preview_large.jpg?1326075809"
"textcolor": "#ffffff"
"updateURL": "https://versioncheck.addons.mozilla.org/en-US/themes/update-check/357219"
"version": "1.0"

New frontend:

"accentcolor": "#"
"author": "Star_Shine"
"description": null
"footerURL": "https://addons.cdn.mozilla.net/user-media/addons/357219/2.jpg?1326075809"
"headerURL": "https://addons.cdn.mozilla.net/user-media/addons/357219/Myhellokitty.jpg?1326075809"
"id": "357219"
"name": "Pixel bears"
"textcolor": "#ffffff"

As far as I can see, this data is used by Firefox, for example to set the favicon in the Add-ons Manager and to set the updateURL, which is important for static themes. Optimally, we would expose the same properties as in the old frontend.

Marking P2 on @jvillalobos’ request.

Issue Analytics

  • State:closed
  • Created 6 years ago
  • Comments:11 (6 by maintainers)

github_iconTop GitHub Comments

2reactions
wagnerandcommented, Nov 8, 2017

This might have been fixed. I could not reproduce any scenario where installing a theme does not add an icon to the add-on manager.

@kumar303 I can still reproduce.

STR using example: https://addons.mozilla.org/en-US/firefox/addon/xmas-02-paillette/

  1. Install using the old frontend: screen shot 2017-11-08 at 11 33 46
  2. Go to the Add-ons Manager screen shot 2017-11-08 at 11 35 00

vs.

  1. Install using the new frontend: screen shot 2017-11-08 at 11 35 37
  2. Go to the Add-ons Manager screen shot 2017-11-08 at 11 35 54

@ValentinaPC

we are talking about static themes, yes?

For now, we don’t. Eventually, it should be supported when static themes are ready too.

what icon do you talk about? this one:

Yes, the icon in the Add-ons Manager; also see above screenshots.

1reaction
kumar303commented, Nov 8, 2017

I thought we had a bug on file for the disappearing icon behavior but I can’t find it.

found it https://github.com/mozilla/addons-frontend/issues/3014 😃

Read more comments on GitHub >

github_iconTop Results From Across the Web

Exposing Properties to the Properties Window - Visual ...
This walkthrough exposes the public properties of an object to the Properties ... For more information, see Install the Visual Studio SDK.
Read more >
Theming and Theme Switching with React and styled- ...
A theme is a structured group of properties that we'll use in the application. You may have more properties and/or different ways to ......
Read more >
Theme Configuration - Tailwind CSS
The theme object contains keys for screens , colors , and spacing , as well as a key for each customizable core plugin....
Read more >
Defining a theme with an .info.yml file
To create a Drupal 8 or later theme you need to first create a THEMENAME.info.yml file that provides meta-data about your theme to...
Read more >
Settings
Exposing settings makes your theme more customizable so it can better express a merchant's brand. It also can make your theme more flexible...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found