quran/quran.com-frontend

Convert Smartbanner to styled-components

Open

#1 020 ouverte le 29 août 2018

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)JavaScript (953 stars) (358 forks)batch import
good first issue

Description

I'm submitting a ...

  • bug report
  • feature request
  • other

Current behavior: The Smartbanner component is heavily using CSS and should instead use styled-components

Expected / new behavior: Convert to styled-components please!

Guide contributeur

Stack technique
javascriptreactcss
Domaine
frontend
Type d'issue
refactor
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.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
React basicsstyled components knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Locate the Smartbanner component and its associated CSS files. Review the styled components documentation to understand how to convert the existing CSS into styled components. Check the issue comments for any suggested approaches or constraints. Ensure the conversion maintains the same visual appearance and functionality.