Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export static theme add-on .xpi from current theme settings #494

Merged
merged 2 commits into from Oct 9, 2018

Conversation

lmorchard
Copy link
Contributor

  • Enable Export button in AppHeader

  • New export.js module with client-side support to assemble a .zip/.xpi
    static theme add-on

  • Add stub component for ExportThemeDialog

  • Add Redux store support for theme export progress and result

  • Extract shared convertToBrowserTheme code from add-on for producing
    real theme from Redux data

  • Add jszip and redux-promise dependencies

  • Turn off some eslint rules that seem annoying

Issue #93

@lmorchard
Copy link
Contributor Author

lmorchard commented Oct 5, 2018

To use this, you should be able to hit "Export" in the header and then see links to theme.zip & theme.xpi in the browser preview area. They're the same file, but I found .zip easier to download and open to look at than .xpi. But, you can use the .xpi in about:debugging to load up the theme

capture

This could definitely use some UX improvements. I'm continuing to tinker to see if I can just have a download automatically start, rather than displaying the links.

@lmorchard
Copy link
Contributor Author

Oh yeah, and we could do with adding some way to let the user set the name, version, etc for the add-on on export.

- Enable Export button in AppHeader

- New export.js module with client-side support to assemble a .zip/.xpi
  static theme add-on

- Add stub component for ExportThemeDialog

- Add Redux store support for theme export progress and result

- Extract shared convertToBrowserTheme code from add-on for producing
  real theme from Redux data

- Add jszip and redux-promise dependencies

- Turn off some eslint rules that seem annoying

- Webpack config tweak for dynamic import bundle name

- Add babel dynamic import plugin

Issue mozilla#93
@meandavejustice
Copy link
Contributor

This works for me, really cool feature.

@meandavejustice meandavejustice merged commit 2b5cdf5 into mozilla:master Oct 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants