guardicore/monkey

Unify modal window styles

Open

#1 002 ouverte le 26 févr. 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (752 forks)batch import
Beginner friendlyComplexity: LowHelp wantedImpact: MediumUI

Métriques du dépôt

Stars
 (6 250 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Identifiez tous les composants modaux avec des styles en ligne dans le code en utilisant les liens fournis et une recherche grep, puis créez ou mettez à jour des modules CSS ou des feuilles de style pour consolider les styles et supprimer les attributs de style en ligne.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Refactorisation
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 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CSS
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80