Skip to content

Passer à une icône dans la barre d’URL + UI améliorée + liens API/icônes#2

Open
Raph563 wants to merge 2 commits intomainfrom
codex/creer-userscript-pour-raccourcisseurs-d-url-3qi3nm
Open

Passer à une icône dans la barre d’URL + UI améliorée + liens API/icônes#2
Raph563 wants to merge 2 commits intomainfrom
codex/creer-userscript-pour-raccourcisseurs-d-url-3qi3nm

Conversation

@Raph563
Copy link
Owner

@Raph563 Raph563 commented Feb 14, 2026

Motivation

  • Fournir un vrai petit icône dans la barre d’URL (page action) pour raccourcir l’URL de l’onglet actif au lieu d’un simple UserScript / bouton toolbar.
  • Améliorer l’ergonomie visuelle du popup et de la page de paramètres pour rendre l’usage et la configuration des tokens/API plus simples.
  • Donner des ressources directes pour récupérer les clés API et des sources d’icônes développeur afin que l’icône par défaut puisse être remplacée facilement.

Description

  • Remplacement du comportement toolbar par une page_action avec gestion dans firefox-extension/manifest.json et un nouveau firefox-extension/background.js pour afficher l’icône sur les onglets http(s); ajout des icônes SVG locales dans firefox-extension/icons/.
  • Refonte du popup UI avec firefox-extension/popup/popup.html|popup.css|popup.js pour un design modernisé, pré-remplissage de l’URL active, sélection du service, raccourcissement, copie et affichage de l’URL finale, et ouverture des liens d’aide via browser.tabs.create.
  • Amélioration de la page d’options firefox-extension/options/* pour gérer les tokens (Bitly, TinyURL, Rebrandly), afficher l’historique enrichi avec date/heure et lister les sites d’icônes recommandés.
  • Documentation mise à jour dans README.md avec instructions d’installation (about:debugging), liens directs vers les APIs (Bitly, TinyURL, Rebrandly, is.gd, v.gd) et ressources d’icônes (Material Icons, Heroicons, Font Awesome, Tabler Icons).
  • Conserver et améliorer l’alternative UserScript mozilla-url-shortener.user.js pour usage in-page, tout en précisant ses limites par rapport à une WebExtension.

Testing

  • Vérification de la syntaxe JavaScript avec node --check firefox-extension/background.js, node --check firefox-extension/popup/popup.js et node --check firefox-extension/options/options.js : toutes les vérifications sont passées avec succès.
  • Validation JSON du manifeste avec python -m json.tool firefox-extension/manifest.json : réussite.
  • Génération d’un aperçu statique du popup via Playwright et sauvegarde de la capture sous artifacts/firefox-urlbar-popup-v2.png : réussite.

Codex Task

@chatgpt-codex-connector
Copy link

💡 Codex Review

const response = await apiRequest({ method: 'GET', url: item.shortUrl, headers: { 'Cache-Control': 'no-cache' } });

P1 Badge Allow expansion requests to short-link domains

expandLink calls GM_xmlhttpRequest on item.shortUrl, but the userscript metadata only whitelists API hosts (api-ssl.bitly.com, api.tinyurl.com, api.rebrandly.com, etc.). For links returned by Bitly/TinyURL/Rebrandly (bit.ly, tinyurl.com, rebrand.ly, or branded domains), the request is blocked by @connect, so “Afficher URL” fails even though shortening succeeded.


.hidden { display: none !important; }

P2 Badge Namespace hidden class to avoid page-wide CSS collisions

The injected stylesheet defines a global .hidden { display: none !important; } rule in a userscript that runs on arbitrary websites, so any site element using a hidden class can be force-hidden by this script. That can break host-page UI unrelated to the shortener widget; this selector should be scoped to the widget’s own classes instead.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant