microsoft/Terminal

Add a setting to manually set the Pane border width

Open

#3.063 geöffnet am 4. Okt. 2019

Auf GitHub ansehen
 (2 Kommentare) (29 Reaktionen) (0 zugewiesene Personen)C++ (3.212 Forks)batch import
Area-SettingsArea-UserInterfaceHelp WantedIssue-TaskProduct-Terminal

Repository-Metriken

Stars
 (35.764 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 27T 19h) (24 gemergte PRs in 30 T)

Beschreibung

Follow up from #994 and PR:#3060

In the aforementioned PR, we're hardcoding the border width to 2. This should really be configurable to any positive double.

Questions:

  • (Similar to the discussion in #3061) There's been discussion before about allowing custom theming, hypothetically powered by XAML. Would we want to wait for that discussion to be had before we do this?
    • If we do want to do this in XAML later, would enabling it in the json now negatively impact that future scenario?

Contributor Guide

Research-Richtung
Überprüfe PR #3060, um die aktuelle hartcodierte Rahmenbreiten Implementierung zu verstehen, und Issue #994 für Themen Diskussionen. Untersuche das Windows Terminal Einstellungsmodell (z.B. profiles.json), um zu bestimmen, wo eine 'borderWidth' Eigenschaft hinzugefügt werden könnte. Untersuche, ob vorhandene Einstellungen Fensterrahmen behandeln. Schlage vor, eine positive Double Einstellung mit entsprechender Validierung hinzuzufügen und aktualisi den Code für die Fensterrahmen Rendering, um sie zu verwenden.
Tech Stack
cpp
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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
C++Windows Terminal
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60