docs: Add Firefox Add-on support to homepage showcase#2208
docs: Add Firefox Add-on support to homepage showcase#2208aklinker1 merged 16 commits intowxt-dev:mainfrom
Conversation
Allow extension entries to include a firefoxSlug alongside the Chrome ID, rendering a "Firefox" badge that links to addons.mozilla.org. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
✅ Deploy Preview for creative-fairy-df92c4 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
I support the idea, since I also publish on Firefox Addons, but having Firefox URL optional while Chrome Extension URL required is still biased toward Chrome Extensions. What if you only publish the extension(s) on Firefox? You can't showcase it, then. Due to smaller user base of Firefox, it actually serves as a great real testing ground with real users before finally deploying it to Chrome. So some people, including me, may publish it there first. |
|
I'll make that update. Also going to scrap the extensions that are already there and add the firefox slug if I can find it. |
|
Only used "K" and not "m" to give the number some weight, when it was 2,000k, it seemed more impressive than 2m. Also, this change helped the stats sit on a single line, making the display cleaner. |
|
@aklinker1 what do you think about firefox addition? |
|
Reviewing this soon! Firefox is my daily driver, so I'm down to add this for sure. |
Using the same logic, we can push it further by not doing any unit conversion at all, and let the raw count speak for itself (e.g., 200.000 users). This is currently the case for both Firefox Addons and Chrome Web Store. I'd suggest keeping it that way. Also, this is an entirely a separate change unrelated to the main topic of this PR. I think it's better to propose this in a different PR |
|
If we want to shorten the numbers, we should use Maybe we combine the firefox and chrome stats into a single number and just add links to each of the stores? That would simplify the UI a bit and give us more room to show the full number. Edit: Chrome reports a weekly user count vs Firefox's daily user count. The current PR ignores that, which is fine. It would also be fine to ignore the units when adding the two numbers together. |
There was a problem hiding this comment.
If you merge with main, the failing checks will pass and your PR will be deployed to preview.
Only real concern is around simplifying the data fetching by combining the firefox and chrome requests into a single graphql call. Then the whole UI could be simplified by combining the results into a single list of standardized objects before rendering them.
We should probably also add support for Edge, but I need to add that to the graphql API first as well. But ideally, this PR adds support for any number of extension stores in a generic way.
Combine Chrome and Firefox data fetching into one GraphQL request using the unified Extension type from queue.wxt.dev (v0.4.4). This addresses PR review feedback: - Delete useFirefoxAddonDetails.ts (separate Mozilla REST API calls) - Update useListExtensionDetails.ts to query both chromeExtensions and firefoxAddons via a shared ExtensionData fragment - Combine user counts into a single number per extension card - Remove link from extension name; store links (Chrome, Firefox) now appear only in the footer row - Use full locale-formatted numbers instead of custom K notation - Generic stores array supports future store additions (e.g. Edge) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Re-add link on extension name (defaults to Chrome, falls back to Firefox) - Fix Firefox data lookup by extracting slug from storeUrl since the GraphQL API returns numeric IDs, not slugs Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
@InvictusNavarchus & @aklinker1. I took your comments into consideration and pushed up the simplified new version. Let me know what you both think. |
- Remove unused chromeToFirefox map - Inline sort score instead of sortKey + strip pattern Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
@aklinker1 can we add the Firefox slug field to the Extension type query? |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2208 +/- ##
==========================================
+ Coverage 79.35% 79.54% +0.18%
==========================================
Files 130 130
Lines 3798 3798
Branches 854 854
==========================================
+ Hits 3014 3021 +7
+ Misses 699 693 -6
+ Partials 85 84 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@wxt-dev/analytics
@wxt-dev/auto-icons
@wxt-dev/browser
@wxt-dev/i18n
@wxt-dev/is-background
@wxt-dev/module-react
@wxt-dev/module-solid
@wxt-dev/module-svelte
@wxt-dev/module-vue
@wxt-dev/runner
@wxt-dev/storage
@wxt-dev/unocss
@wxt-dev/webextension-polyfill
wxt
commit: |
|
Nice, much simpler, looks great too!
Sure, I can add that! |
|
Is this what you wanted? wxt-dev/queue@3ce60f3#diff-70f27b1705e6c4424067f93d1d897f8f48711869f34688493bf0d16b1e8d0f1e It's deployed |
|
The slug option will clean up some logic here. If that is done first.
But we don’t need to wait that change can be another PR.
…On Sun, Apr 12, 2026 at 09:16 Aaron ***@***.***> wrote:
*aklinker1* left a comment (wxt-dev/wxt#2208)
<#2208 (comment)>
Do you want to use in this PR for something, or can I merge?
—
Reply to this email directly, view it on GitHub
<#2208 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAO3DJFE67XIDKAB2QBDWI34VOJJDAVCNFSM6AAAAACW4M44FSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DEMZRGU3TSNRTHA>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Use the new `slug` field from the GraphQL API to key Firefox results instead of parsing it from storeUrl with a regex. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The slug field is on FirefoxAddon type only, not the shared Extension type. Move it out of the ExtensionData fragment and query it inline on firefoxAddons. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Sweet, gonna merge, then move the list to a YML file outside the vue component lol. It's gotten quite large. |
|
Having problems with github actions... one sec... |
|
Thanks for helping make WXT better! |
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Aaron <aaronklinker1@gmail.com>
Summary
firefoxSlugto extension entries (Chrome-only, Firefox-only, or both)ExtensiontypeScreenshot