guardicore/monkey

Unify modal window styles

Open

#1002 aperta il 26 feb 2021

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (752 fork)batch import
Beginner friendlyComplexity: LowHelp wantedImpact: MediumUI

Metriche repository

Star
 (6250 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Some modal windows have their style inlined in their JSX:

https://github.com/guardicore/monkey/blob/bc3283c4a5177c7d08ff65bb13370d061aae60d3/monkey/monkey_island/cc/ui/src/components/ui-components/IslandMonkeyRunErrorModal.js#L72-L82

https://github.com/guardicore/monkey/blob/bc3283c4a5177c7d08ff65bb13370d061aae60d3/monkey/monkey_island/cc/ui/src/components/pages/ConfigurePage.js#L178-L195

https://github.com/guardicore/monkey/blob/26dc2aeb146a50651a0df223d979a0fdf2161a5c/monkey/monkey_island/cc/ui/src/components/configuration-components/UnsafeOptionsConfirmationModal.js#L7-L27

Other modals, such as ImageModal and ScoutsuiteRuleModal have separate style sheets.

Unify the styles for all modal windows. Use style sheets instead of hardcoding style in the JSX

Guida contributor

Direzione di ricerca
Identifica tutti i componenti modali con stili inline nel codice utilizzando i link forniti e una ricerca grep, quindi crea o aggiorna moduli CSS o fogli di stile per consolidare gli stili e rimuovere gli attributi di stile inline.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Refactoring
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CSS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80