brave/brave-browser

Show BAT favicon when adding rewards internal page to sidebar

Open

#41 617 ouverte le 14 oct. 2024

Voir sur GitHub
 (5 commentaires) (0 réactions) (1 assigné)JavaScript (16 194 stars) (2 164 forks)batch import
OS/Desktopfeature/settingsfeature/sidebargood first issuepriority/P5

Description

When adding the rewards internal page to the sidebar, the icon doesn't appear and instead shows an "R". We should change the icon that shows so it shows the correct BAT icon

Figma https://www.figma.com/design/H11ZOl6JMYbCTW4ZJXqR5V/%F0%9F%A6%81-Browser?node-id=8165-19398&t=5wWBqB0CO1rtctLm-1

Actual result

Image

Expected result

Image

Guide contributeur

Stack technique
javascriptcssreact
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
blocked
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic React knowledgefamiliarity with Brave UI components
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Review the Brave sidebar code for adding internal pages. The icon for the rewards page is currently a placeholder 'R'. The Figma design (linked) shows the correct BAT icon. Locate where the sidebar item icon is set and replace it with the appropriate BAT icon asset. Check existing icon assets in the repository for the BAT icon.