iced-rs/iced

Superfluous alignment functions

Offen

#286 geöffnet am 11.04.2020

Auf GitHub ansehenContributor Guide
 (7 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)Rust (1.572 Forks)batch import
good first issuehelp wantedlayoutquestion

Repository-Metriken

Stars
 (30.491 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 24T 11h) (11 gemergte PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Identifiziere alle Verwendungen von Container::center x, HorizontalAlignment, VerticalAlignment und Align im Code. Entferne die redundanten Funktionen und aktualisiere abhängigen Code und Tests.
Tech Stack
rust
Domain
desktop
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.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitRust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75