GoogleChromeLabs/quicklink

An additional `div` markup is wrapped around each router component by `withQuicklink` HOC

Open

#193 aperta il 2 lug 2020

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)JavaScript (431 fork)batch import
help wanted

Metriche repository

Star
 (10.861 star)
Metriche merge PR
 (Merge medio 47g 12h) (4 PR mergiate in 30 g)

Descrizione

Describe the bug As you can see in the HOC, we have an additional div markup wrapping around the Component (probably it would be a route component) so inserting that div markup could break the app styling since the app styling does not expect another div to be inserted by withQuicklink HOC. Capture

To Reproduce If we use nest CSS selector without considering that div, we might break the styling.

Expected behavior I hope we could avoid adding any wrapping markup around a route component if possible. At least, I hope we manage to avoid impacting the rest of the app as well as the styling in any case.

Version:

  • Quicklink 2.0

cc @addyosmani @demianrenzulli

Guida contributor

Direzione di ricerca
Ispeziona l'implementazione dell'HOC `withQuicklink` per trovare dove viene aggiunto il `div` extra e rifattorizzalo per evitare il wrapping senza influenzare lo stile.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ReactJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70