fix(a11y): restore keyboard focus after addCurrentDomain

[Copilot]

Disabling the "Add" button at the start of addCurrentDomain immediately drops browser focus to the top of the DOM, breaking keyboard navigation and screen readers.

Changes

• popup.js — addCurrentDomain finally block: after re-enabling the button, programmatically move focus to #status, which always holds a meaningful message (success or error) at that point

} finally {
  isAddingDomain = false;
  if (addButton) addButton.disabled = false;
  const statusElement = document.getElementById("status");
  if (statusElement) {
    if (!statusElement.hasAttribute("tabindex")) {
      statusElement.setAttribute("tabindex", "-1");
    }
    statusElement.focus();
  }
}

Placing the logic in finally ensures all exit paths — success, duplicate, no-domain, storage error — restore focus consistently. Pattern mirrors existing focus management in clearDomains.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Corriger la perte de focus d'accessibilité après un ajout de domaine</issue_title>
<issue_description>## Contexte
Dans la fenêtre de la popup, l'utilisateur a la possibilité d'ajouter le nom de domaine de l'onglet actif dans la liste blanche.

Problème

Lorsque la fonction addCurrentDomain se termine avec succès dans popup.js, elle peut déclencher des modifications d'interface qui rendent le bouton "Ajouter" inactif (disabled) ou masqué. Lorsqu'un élément actif (avec le focus) devient disabled, le navigateur perd le repère de focus clavier, renvoyant l'utilisateur en haut du DOM et brisant la navigation pour les lecteurs d'écran.

Scénario de Reproduction

1. Ouvrir la popup sur un site non répertorié.
2. Utiliser la touche Tab pour naviguer jusqu'au bouton "Ajouter le domaine courant".
3. Appuyer sur Entrée ou Espace.
4. Essayer de tabuler à nouveau : le flux de navigation est cassé.

Comportement Attendu

Après une action réussie désactivant l'élément courant, le focus doit être programmatiquement transféré vers un élément logique suivant, comme le statut de confirmation ou le bouton de suppression de la liste.

Pistes Techniques

Fichier popup.js, fonction addCurrentDomain.
À la fin du succès de l'ajout, déplacer le focus :

const statusElement = document.getElementById("status");
if (statusElement) {
  statusElement.setAttribute("tabindex", "-1");
  statusElement.focus();
}

Environnement

Interface Popup UI, Accessibilité (a11y).</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Corriger la perte de focus d'accessibilité après un ajout de domaine #85

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR addresses an accessibility regression in the popup UI where disabling the “Add” button during addCurrentDomain can cause keyboard focus to be lost, disrupting keyboard navigation and screen reader flow.

Changes:

• Restores a predictable focus target after addCurrentDomain completes by focusing the #status element in the finally block.
• Ensures #status is programmatically focusable by adding tabindex="-1" when missing.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.