brave/brave-browser

Add keyboard shortcut to update adblock lists

Open

#43.667 geöffnet am 31. Jan. 2025

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (16.194 Stars) (2.164 Forks)batch import
OS/DesktopQA/Yesenhancementfeature/keyboard-shortcutsfeature/shields/adblockgood first issuepriority/P4release-notes/include

Beschreibung

Requested at https://x.com/jarkk0sal0/status/1885404217864528039?s=46&t=c88ZoLNNPThnGggE75IoLQ

We don't need to assign a shortcut by default, but having it available for configuration in brave://settings/system/shortcuts could be useful.

Contributor Guide

Tech Stack
javascriptreact
Domain
frontend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic knowledge of Chromium's shortcut systemFamiliarity with Brave's settings UI
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
Examine how existing keyboard shortcuts are defined in Brave's settings, likely in files under `brave/settings/` or `chrome/browser/ui/webui/settings/`. The shortcut configuration file (e.g., `shortcuts mapping.js`) and the settings page for shortcuts (`shortcuts.html`) are key. Look at the tweet referenced for additional context. No existing PR or assignee, so the issue is open for implementation.