Conversation
画面全体のレイアウトを整理し、`EditForm`の外側に`FluentStack`を追加。パンくずリストの配置を変更し、各入力項目・アクション部のUI構造を見直して可読性・操作性を向上。カテゴリー欄の属性修正、表示優先度選択欄にバリデーションメッセージ追加。登録ボタン処理のインデント調整のみで、機能的な変更はなし。
AnnouncementEdit.razorのレイアウトをFluentStack/FluentCard中心に刷新し、パンくず・入力ラベル・アクション部のUI/UXを向上。編集カードの削除ボタン配置や履歴表示の視認性を改善。CSS不要スタイル削除と新規スタイル追加、保存ボタンの状態管理をプロパティ化しUI更新を安定化。
UI構造を整理し、<FluentNavLink>を<FluentAnchor>へ統一、ローディング・ページング状態管理を改善しました。ページネーションやリンク生成のロジックも見直し、可読性・保守性・一貫性を向上させました。掲載日表示の未設定対応やインデント整理も含みます。
AnnouncementDeleteComplete.razor のレイアウトを FluentStack/FluentCard ベースに刷新し、パンくずやアクションリンクを FluentAnchor 化。未設定項目の表示文言を追加し、更新履歴表示を FluentAccordion＋FluentDivider で整理。不要なCSS (.container, .border) を削除し、コード構造も簡素化・可読性向上。
お知らせメッセージ削除確認画面のレイアウトをFluentStackとFluentCard中心に刷新し、UIを整理・視認性を向上。パンくずリストやアクション部のリンクをFluentAnchor化、削除ボタンの状態管理を改善。未設定項目の表示やエラーメッセージの扱いを明確化し、更新履歴表示もFluentAccordion＋FluentDividerで分かりやすく。不要なCSS定義を削除。
Error.razorのレイアウトをFluentStack/FluentCardを用いて整理し、タイトルやメッセージ、スタックトレース表示のデザインを統一しました。不要となったカスタムCSSとdiv要素も削除し、全体的な一貫性と可読性を向上させました。
トップページのタイトルを「Dressca-CMS - トップ」に変更し、内容をh3見出しとFluentStackを用いたレイアウトに刷新しました。案内文もFluentStack内のdiv要素として表示するよう修正しました。
ログイン画面のUIを<FluentStack>と<FluentInputLabel>を用いて整理・改善し、タイトルも「Dressca-CMS - ログイン」に変更しました。従来のRequiredMarkコンポーネントとそのスタイルを削除し、必須表示は<FluentInputLabel>のRequired属性で対応するようにしました。不要となったusingディレクティブも削除しています。
NotFound.razorにページタイトルを追加し、FluentStackでレイアウトを整理しました。英語メッセージを日本語「申し訳ありません、お探しのコンテンツが見つかりません。」に変更しました。
各ページに `<PageTitle>` タグを追加し、ブラウザタブのタイトルを「Dressca-CMS - [各ページ名]」に統一しました。また、各ページ先頭に日本語の `<h3>` タイトルを表示するよう修正しました。さらに、`AnnouncementDeleteComplete.razor` の初期化処理から不要な遅延（`Task.Delay(1000)`）を削除しました。
スタックトレース表示部分のHTML構造変更に伴い、テストコードで検証対象をdiv.stack-trace-boxからfluent-card要素へ修正しました。見出しタグもh3からh4へ変更し、例外が表示されない場合の検証もfluent-cardの有無で確認するように更新しています。
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request refactors the UI to maximize the use of Fluent UI components, improving consistency and maintainability. The changes replace custom components and CSS with Fluent UI equivalents and update the page layout structure.
Key Changes:
- Replaced custom
RequiredMarkcomponent with Fluent UI'sFluentInputLabelcomponent withRequiredattribute - Converted navigation links from
FluentNavLinktoFluentAnchorwithAppearance.Hypertextfor breadcrumbs and in-page links - Refactored layout from
FluentGridtoFluentStackand wrapped content sections inFluentCardcomponents
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| ErrorTest.cs | Updated test assertions to match new component structure (removed wrapper div, changed h3 to h4) |
| app.css | Modified content width constraints using min-width and max-width instead of min() function |
| _Imports.razor | Removed unused import for DresscaCMS.Web.Components.Shared namespace |
| RequiredMark.razor | Deleted custom required mark component |
| RequiredMark.razor.css | Deleted associated CSS file |
| NotFound.razor | Added PageTitle, wrapped content in div, and used FluentStack for layout |
| Login.razor | Replaced FluentLabel + RequiredMark with FluentInputLabel, added VerticalGap for spacing |
| Home.razor | Added PageTitle and FluentStack layout wrapper |
| Error.razor | Updated PageTitle, restructured layout with FluentStack, changed h3 to h4 in stack trace |
| Error.razor.css | Removed custom CSS class (now using FluentCard) |
| AnnouncementsManagement.razor | Renamed properties (isLoading→IsLoading, pageNumber→actualPageNumber), replaced FluentNavLink with FluentAnchor, added PageTitle |
| AnnouncementEdit.razor | Replaced FluentGrid with FluentStack layout, replaced FluentLabel with FluentInputLabel, reorganized action panel positioning, added IsSubmitting property with StateHasChanged |
| AnnouncementEdit.razor.css | Removed custom container/border CSS, added card-with-action button positioning styles |
| AnnouncementDeleteConfirm.razor | Similar layout refactoring as Edit page, replaced FluentGrid with FluentStack, added IsSubmitting property |
| AnnouncementDeleteConfirm.razor.css | Removed custom container/border CSS classes |
| AnnouncementDeleteComplete.razor | Layout refactoring with FluentStack and FluentCard, improved consistency in displaying null values |
| AnnouncementDeleteComplete.razor.css | Removed custom container/border CSS classes |
| AnnouncementCreate.razor | Similar refactoring as Edit page, replaced custom components with Fluent UI equivalents, added IsSubmitting property with StateHasChanged |
| AnnouncementCreate.razor.css | Removed custom container CSS, added card-with-action button positioning styles |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
複数の .razor ファイルにて、IsSubmitting/IsLoading プロパティを単純なフィールド（isSubmitting, isLoading）に置き換え、状態変更時に明示的に StateHasChanged() を呼ぶ形にリファクタリングしました。これにより、状態管理の実装がシンプルかつ統一され、可読性・保守性が向上しています。各 UI の Disabled 制御やローディング表示も新しいフィールド参照に修正しています。
Test Result 📝Test was a success. Coverage 📐Summary
DresscaCMS.Announcement - 4.3%
DresscaCMS.Web - 3.5%
|
kharada7
approved these changes
Dec 22, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
この Pull request で実施したこと
Fluent UI のコンポーネントを可能な限り使うようにリファクタリングしました。
この Pull request では実施していないこと
なし
Issues や Discussions 、関連する Web サイトなどへのリンク