Skip to content
This repository was archived by the owner on Sep 12, 2026. It is now read-only.

refactor: 画面タイトル削除と上部余白の調整 - #6

Merged
treetips merged 1 commit into
mainfrom
feature/remove-screen-title-and-padding
Sep 5, 2026
Merged

treetips merged 1 commit into
mainfrom
feature/remove-screen-title-and-padding

Conversation

@treetips

@treetips treetips commented Sep 5, 2026

Copy link
Copy Markdown
Owner

概要

各画面先頭の画面タイトルは左サイドナビで識別可能なため削除し、削除により失われた上部余白を再設定しました。合わせて作業ディレクトリに存在した未コミットの差分も本PRに含めています。

変更内容

画面タイトル削除

  • lib/ui/screens/convert_screen.dart:167 ScreenTitle(title: l10n.screenConvert) 削除
  • lib/ui/screens/settings_screen.dart:28 ScreenTitle と後続の SizedBox(height: 24) 削除
  • lib/ui/screens/about_screen.dart:44 ScreenTitle(title: l10n.screenAbout) 削除
  • lib/ui/widgets/section.dart:9 未使用となった ScreenTitle ウィジェット定義を削除

上部余白の追加

タイトル削除で上部余白が 0 になっていたため SingleChildScrollView の padding を修正:

  • lib/ui/screens/convert_screen.dart:350 fromLTRB(16, 0, 16, 16) → fromLTRB(16, 16, 16, 16)
  • lib/ui/screens/convert_screen.dart:369 fromLTRB(16, 0, 16, 72) → fromLTRB(16, 16, 16, 72)
  • lib/ui/screens/settings_screen.dart:24 fromLTRB(16, 0, 16, 16) → fromLTRB(16, 16, 16, 16)
  • lib/ui/screens/about_screen.dart:40 同上

その他、本PRに含めた未コミット差分

  • 壁紙画像の差し替え: assets/images/wallpaper/wallpaper1.jpg / wallpaper3.jpg
  • 画面キャプチャ更新: docs/design/feature/images/画面構成-About.webp / 画面構成-画像変換.webp / 新規 画面構成-設定-1.webp
  • docs/design/feature/application-settings.md 新規追加(設定ファイル仕様)

検証

  • fvm dart format 実行済み
  • fvm flutter analyze => No issues
  • fvm flutter test => 40 passed
  • fvm flutter build macos --debug => Built
  • 各画面でタイトルが非表示かつ上部に 16px の余白が確保されていることを目視確認

- 各画面先頭の ScreenTitle (画像変換/設定/About) を削除(左サイドナビで識別可能のため)
- lib/ui/widgets/section.dart: ScreenTitle ウィジェット定義を削除
- 上部余白が無くなったため SingleChildScrollView の padding を fromLTRB(16,0,16,16) → fromLTRB(16,16,16,16) に修正(convert: 2箇所、settings/about: 各1箇所)
- その他、作業ディレクトリに存在した未コミット差分も本PRに含めて反映
  - 壁紙画像の差し替え (wallpaper1/3.jpg, 画面構成画像)
  - docs/design/feature/application-settings.md の新規追加
@treetips
treetips enabled auto-merge September 5, 2026 12:57
@treetips treetips self-assigned this Sep 5, 2026
@treetips
treetips merged commit 9ba4cfc into main Sep 5, 2026
1 check passed
@treetips
treetips deleted the feature/remove-screen-title-and-padding branch September 5, 2026 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant