microsoft/Terminal

Add a setting to manually set the Pane border width

Open

#3,063 opened on 2019年10月4日

GitHub で見る
 (2 comments) (29 reactions) (0 assignees)C++ (3,212 forks)batch import
Area-SettingsArea-UserInterfaceHelp WantedIssue-TaskProduct-Terminal

Repository metrics

Stars
 (35,764 stars)
PR merge metrics
 (平均マージ 27d 19h) (30d で 24 merged PRs)

説明

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?

コントリビューターガイド

調査方針
PR #3060 をレビューして、現在のハードコードされたボーダー幅の実装を理解し、issue #994 のテーマに関する議論を確認します。Windows Terminal の設定モデル（例：profiles.json）を調べて、'borderWidth' プロパティを追加できる場所を特定します。既存の設定でペーンボーダーを扱っているものがあるか調査します。適切な検証を伴う正の double 型設定を追加し、ペーンボーダーのレンダリングコードを更新してそれを使用することを提案します。
技術スタック
cpp
領域
frontend
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++Windows Terminal
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60