iced-rs/iced

Superfluous alignment functions

Aperta

#286 aperta il 11 apr 2020

Vedi su GitHubGuida contributor
 (7 commenti) (5 reazioni) (0 assegnatari)Rust (1572 fork)batch import
good first issuehelp wantedlayoutquestion

Metriche repository

Star
 (30.491 stelle)
Metriche merge PR
 (Merge medio 24g 11h) (11 PR mergiate in 30 g)

Descrizione

The current api exposes multiple ways for alignment to be expressed:

  • Container::center_x is equivalent to Container::align_x(Center)
  • HorizonalAlignment, VerticalAlignement, and Align all express the same concept.

These introduce more names and relationships for users to learn about in the api, and I think reducing them would be positive. could we remove them?

Guida contributor

Direzione di ricerca
Identifica tutti gli usi di Container::center x, HorizontalAlignment, VerticalAlignment e Align nel codice. Rimuovi le funzioni ridondanti e aggiorna il codice e i test dipendenti.
Tech stack
rust
Dominio
desktop
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitRust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75