ux(dateien): add an empty-state message to FilesView
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
src/LageBuch.App.Shared/Views/FilesView.axamlを読み、52行目あたりをsrc/LageBuch.App.Shared/Views/LinksView.axamlと比較してください。TextMutedBrushを使って空状態のドイツ語メッセージを追加し、リストの枠線の表示をファイル数にバインドしてください。変更はFilesViewに限定してください。make test PROJECT=tests/LageBuch.Acceptance.Tests FILTER=Name~Filesとmake runで確認してください。
索引モデルが issue の本文から書いたものです。
説明
The Dateien tab shows an empty bordered table when an incident has no
attachments. There is no sentence telling the user that this is normal and what
to do about it.
LinksView already solves exactly this, and it is three lines. From
src/LageBuch.App.Shared/Views/LinksView.axaml line 52:
<TextBlock DockPanel.Dock="Top" x:Name="EmptyText" Text="Keine Links hinterlegt."
Foreground="{StaticResource TextMutedBrush}"
IsVisible="{Binding !Links.Count}" />
with the counterpart IsVisible="{Binding Links.Count}" on the list Border so
the two never show at once.
What to do
Apply the same pattern to src/LageBuch.App.Shared/Views/FilesView.axaml:
- add an
EmptyTextblock bound to the inverse of the file collection's count - bind the existing list
Bordervisibility to the count so the empty frame
does not sit behind the message - use
TextMutedBrush, matching the Links tab
Keep the message German, calm and short. Something in the spirit of
Keine Dateien angehängt. fits the house style; the existing add button right
below already tells the user how to add one.
Scope this PR to FilesView only. Several other tabs have the same gap and will
get their own issues.
Verify
make test PROJECT=tests/LageBuch.Acceptance.Tests FILTER=Name~Files
make run
Difficulty: easy.
New here?
Comment to claim this issue and it is yours. If no pull request appears within
seven days it goes back in the pool, no hard feelings.
Before you open the PR, please skim CONTRIBUTING.md:
commits need DCO sign-off (git commit -s) and a Conventional Commit subject.
Changes under src/ additionally need an entry under ## [Unreleased] in
CHANGELOG.md — CI fails the PR otherwise.
Questions are welcome right here or in
Discussions.
- 主要言語
- C#
- スター
- 1
- フォーク
- 3
- 平均マージ
- 4時間 49分
- マージ済み PR(30日)
- 177
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CodeForFire/lagebuch のほかの issue
-
Stammdaten: in the Checkliste editor the PFLICHT header is not aligned with its column対応中かも @Saad-elnagar が 1 日前に担当しました。 オープンbug ux
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
CodeForFire/lagebuch#714 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Stammdaten: on a phone the Fahrzeuge editor squeezes Wache and Funkrufname to nothing対応中かも @Saad-elnagar が 1 日前に担当しました。 オープンbug ux
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
CodeForFire/lagebuch#713 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
CodeForFire/lagebuch#405 ·
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CodeForFire/lagebuch#349 ·
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CodeForFire/lagebuch#348 ·
メンテナーはふだん 1 日以内に返信
CodeForFire/lagebuch の issue をすべて見る
似ている issue
-
bug effort:S P2
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
nightscout/nocturne#2449 ·
メンテナーはふだん 1 日以内に返信
-
area: client bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
bug mcp misleading small
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
hazeliscoding/gil-sweep#25 ·
メンテナーはふだん 1 日以内に返信