Skip to content

feat: add DialogHeader component and FormField.Required marker#55

Merged
andregoepel merged 1 commit into
mainfrom
feature/dialog-header
Jul 22, 2026
Merged

feat: add DialogHeader component and FormField.Required marker#55
andregoepel merged 1 commit into
mainfrom
feature/dialog-header

Conversation

@andregoepel

Copy link
Copy Markdown
Owner

Summary

Prompted by nerdventures-studio/customer-portal#53: comparing that app's live UI against its design mockup ("AppFoundation Screens") showed every form dialog falling short of the intended two-line header (title + subtitle, border-separated from the body) — a limitation of DialogService.OpenAsync(string title, ...), which only accepts a plain string.

  • DialogHeader — a small presentational component (Title + optional Subtitle, bottom-bordered via a new .ag-dialog-header class) to pass as the titleContent RenderFragment overload of OpenAsync. Radzen still renders its own close button/wrapper around it — this only replaces the title area's content.
  • FormField.Required — new bool param, renders a red .ag-required asterisk after the label when set. Purely visual (pair with RadzenRequiredValidator for actual validation) — closes another gap from the same comparison (mockups mark required fields, the app didn't).
  • DESIGN.md: new "Dialogs" recipe under §5, Required documented under "Forms", both new classes added to the §6 reference table.
  • README.md: DialogHeader added to the components list.
  • Version bumped 1.2.11.3.0 (new feature).

Test plan

  • dotnet restore --locked-mode — no lock-file drift (no new package deps)
  • dotnet build -c Release — 0 warnings, 0 errors
  • dotnet test — 152/152 passing (6 new: DialogHeaderTests, 2 new FormFieldTests cases)
  • dotnet csharpier format . — clean

Radzen's DialogService.OpenAsync only takes a plain title string, so every
form dialog in a consuming app fell short of the design's richer two-line
header (title + subtitle, border-separated from the body). DialogHeader
closes that gap as a titleContent RenderFragment; FormField.Required adds
the matching visual asterisk for required fields.
@andregoepel
andregoepel merged commit 37d780b into main Jul 22, 2026
2 checks passed
@andregoepel
andregoepel deleted the feature/dialog-header branch July 23, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant