linebender/druid

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

Open

#777 geöffnet am 30. März 2020

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (567 Forks)batch import
help wantedshell

Repository-Metriken

Stars
 (9.091 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Identifiziere alle Methoden in druid shell::Window, die den Zustand ändern, aber kein set Präfix haben, benenne sie mit dem set Präfix um und aktualisiere alle Referenzen im Codebase.
Tech Stack
rust
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85