fix: restore keyboard focus after clearing whitelist

[Copilot]

• Add document.getElementById("add-domain").focus() after await renderList() in clearDomains in popup.js
• Validate syntax with node --check
• Run code review (no blocking issues)
• Run CodeQL check (0 alerts)
• Improve focus logic: guard against disabled #add-domain, fall back to #clear-domains, then #status region

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Perte de focus après le vidage complet de la liste des exceptions</issue_title>
<issue_description>## Contexte
L'interface utilisateur de la popup permet de vider l'intégralité de la liste blanche via le bouton "Vider la liste" (clear-domains), défini dans popup.js.

Problème

Après avoir cliqué sur "Vider la liste" et validé la boîte de dialogue de confirmation, le DOM est mis à jour et la liste est vidée, mais le focus clavier est perdu. Le focus retourne au niveau du body, ce qui rompt la navigation au clavier et dégrade l'accessibilité.

Scénario de Reproduction

1. Ouvrir la popup avec au moins un domaine enregistré.
2. Utiliser la touche Tab pour naviguer jusqu'au bouton "Vider la liste".
3. Appuyer sur Entrée.
4. Confirmer la boîte de dialogue.
5. Appuyer à nouveau sur Tab et constater la perte de positionnement logique.

Comportement Attendu

Une fois la liste vidée avec succès, le focus doit être redirigé de manière programmatique vers le premier élément interactif pertinent, comme le bouton "Ajouter le domaine courant".

Pistes Techniques

Fichier concerné : popup.js.
Dans la fonction clearDomains, après l'appel à await renderList(), ajouter une instruction pour cibler le bouton principal :

document.getElementById("add-domain").focus();
```</issue_description>

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

<comments>
</comments>

• Fixes Perte de focus après le vidage complet de la liste des exceptions #71

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull request overview

This PR addresses an accessibility issue in the extension popup where keyboard focus is lost (falls back to body) after clearing the whitelist, by explicitly restoring focus after the list re-renders.

Changes:

• Update clearDomains in popup.js to focus #add-domain after renderList() completes.

---

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

[Copilot]

#add-domain can be disabled when no current domain is detected (initPopup sets addButton.disabled = true). Calling .focus() on a disabled button is a no-op, so focus may still drop to body in exactly the scenarios where accessibility is already degraded (e.g., internal pages). Consider checking add-domain exists and is not disabled before focusing it, and otherwise moving focus to another enabled control (e.g., #clear-domains) or making the status region programmatically focusable as a fallback.

Suggested change

	document.getElementById("add-domain").focus();
	const addButton = document.getElementById("add-domain");
	if (addButton && !addButton.disabled) {
	addButton.focus();
	} else {
	const clearButton = document.getElementById("clear-domains");
	if (clearButton && !clearButton.disabled) {
	clearButton.focus();
	} else {
	const statusRegion = document.getElementById("status");
	if (statusRegion) {
	if (!statusRegion.hasAttribute("tabindex")) {
	statusRegion.setAttribute("tabindex", "-1");
	}
	statusRegion.focus();
	}
	}
	}

[ormidales]

@copilot apply changes based on the comments in this thread

[Copilot]

@copilot apply changes based on the comments in this thread

Applied in b914da5. The focus logic in clearDomains now checks #add-domain first (skips it if disabled), falls back to #clear-domains, and finally focuses the #status region (adding tabindex="-1" if needed) so focus is never silently dropped to body.