linebender/druid

Mutating setters on druid_shell::Window should have `set_` names.

Open

#777 aperta il 30 mar 2020

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Rust (567 fork)batch import
help wantedshell

Metriche repository

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

Descrizione

At least including set_resizeable, and set_show_titlebar, although the latter name isn't great. I might change it to something like titlebar_is_visible and set_titlebar_is_visible or something?

Hmm must not have noticed this, but these should be called set_.

Originally posted by @cmyr in https://github.com/xi-editor/druid/pull/773

Guida contributor

Direzione di ricerca
Identifica tutti i metodi in druid shell::Window che mutano lo stato ma non hanno il prefisso set , rinominali usando il prefisso set e aggiorna tutti i riferimenti nel codebase.
Tech stack
rust
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85