From 87db67f979cf4c33a56f8ec99211573819cd8a35 Mon Sep 17 00:00:00 2001 From: James Kolpack Date: Mon, 5 Jan 2026 14:01:46 -0500 Subject: [PATCH] Refactor MudPaper component styling across various features for consistency Updated the MudPaper component styling in multiple files to use a consistent padding class of "pa-3 pa-md-6" instead of "pa-6". This change enhances the visual consistency of the UI across the Calendar, Events, Students, and Teams components, improving the overall user experience. --- .../Components/Features/Calendar/Import.razor | 4 +- .../Components/Features/Calendar/Index.razor | 2 +- .../Features/Events/CareerMapping.razor | 6 +- .../Components/Features/Events/Create.razor | 8 +-- .../Components/Features/Events/Details.razor | 2 +- WebApp/Components/Features/Events/Edit.razor | 8 +-- WebApp/Components/Features/Events/Index.razor | 2 +- .../Features/MeetingSchedule/Index.razor | 6 +- .../Components/Features/Students/Create.razor | 2 +- .../Features/Students/Details.razor | 2 +- .../Components/Features/Students/Edit.razor | 2 +- .../Features/Students/EventRanking.razor | 4 +- .../Components/Features/Students/Index.razor | 6 +- .../Features/Students/Registration.razor | 9 +-- .../Features/Teams/Assignment.razor | 4 +- .../Teams/Components/TeamStudents.razor | 62 ++++++++++--------- WebApp/Components/Features/Teams/Create.razor | 2 +- .../Components/Features/Teams/Details.razor | 2 +- WebApp/Components/Features/Teams/Edit.razor | 2 +- WebApp/Components/Features/Teams/Goals.razor | 2 +- WebApp/Components/Features/Teams/Index.razor | 2 +- .../Components/Shared/Layout/MainLayout.razor | 4 +- WebApp/wwwroot/app.css | 41 ++++++++++++ 23 files changed, 115 insertions(+), 69 deletions(-) diff --git a/WebApp/Components/Features/Calendar/Import.razor b/WebApp/Components/Features/Calendar/Import.razor index bc3aed3..afbc52e 100644 --- a/WebApp/Components/Features/Calendar/Import.razor +++ b/WebApp/Components/Features/Calendar/Import.razor @@ -19,7 +19,7 @@ - + Paste Event Occurrence Data - + Parsed Results @if (_isParsing) diff --git a/WebApp/Components/Features/Calendar/Index.razor b/WebApp/Components/Features/Calendar/Index.razor index 244776a..5998030 100644 --- a/WebApp/Components/Features/Calendar/Index.razor +++ b/WebApp/Components/Features/Calendar/Index.razor @@ -14,7 +14,7 @@ - + @if (_calendarItems == null) { diff --git a/WebApp/Components/Features/Events/CareerMapping.razor b/WebApp/Components/Features/Events/CareerMapping.razor index c1e4360..95e7798 100644 --- a/WebApp/Components/Features/Events/CareerMapping.razor +++ b/WebApp/Components/Features/Events/CareerMapping.razor @@ -14,14 +14,14 @@ @if (_isLoading) { - + Loading career mapping data... } else if (_networkData == null || !_networkData.Nodes.Any()) { - + No Career Field Mappings Found No events have related careers assigned that match any career fields. Edit events to add related careers. @@ -30,7 +30,7 @@ else if (_networkData == null || !_networkData.Nodes.Any()) } else { - + Event-Career Field Relationships This diagram shows the connections between events and their related career fields. diff --git a/WebApp/Components/Features/Events/Create.razor b/WebApp/Components/Features/Events/Create.razor index 76c0b0d..cf53ef6 100644 --- a/WebApp/Components/Features/Events/Create.razor +++ b/WebApp/Components/Features/Events/Create.razor @@ -25,7 +25,7 @@ - + Basic Information @@ -49,7 +49,7 @@ - + Event Details @@ -67,7 +67,7 @@ - + Team Configuration @@ -88,7 +88,7 @@ - + Competition Details diff --git a/WebApp/Components/Features/Events/Details.razor b/WebApp/Components/Features/Events/Details.razor index 549c7ad..82cb007 100644 --- a/WebApp/Components/Features/Events/Details.razor +++ b/WebApp/Components/Features/Events/Details.razor @@ -27,7 +27,7 @@ - + Basic Information diff --git a/WebApp/Components/Features/Events/Edit.razor b/WebApp/Components/Features/Events/Edit.razor index 9bf2363..63c5a93 100644 --- a/WebApp/Components/Features/Events/Edit.razor +++ b/WebApp/Components/Features/Events/Edit.razor @@ -32,7 +32,7 @@ - + Basic Information @@ -56,7 +56,7 @@ - + Event Details @@ -74,7 +74,7 @@ - + Team Configuration @@ -95,7 +95,7 @@ - + Competition Details diff --git a/WebApp/Components/Features/Events/Index.razor b/WebApp/Components/Features/Events/Index.razor index 2909070..d37b407 100644 --- a/WebApp/Components/Features/Events/Index.razor +++ b/WebApp/Components/Features/Events/Index.razor @@ -15,7 +15,7 @@ - + - + - + Time Slots @@ -81,7 +81,7 @@ - + - + Student Information diff --git a/WebApp/Components/Features/Students/Details.razor b/WebApp/Components/Features/Students/Details.razor index bfde763..395a8da 100644 --- a/WebApp/Components/Features/Students/Details.razor +++ b/WebApp/Components/Features/Students/Details.razor @@ -26,7 +26,7 @@ - + Student Information diff --git a/WebApp/Components/Features/Students/Edit.razor b/WebApp/Components/Features/Students/Edit.razor index 7dff7dd..0540e2f 100644 --- a/WebApp/Components/Features/Students/Edit.razor +++ b/WebApp/Components/Features/Students/Edit.razor @@ -32,7 +32,7 @@ - + diff --git a/WebApp/Components/Features/Students/EventRanking.razor b/WebApp/Components/Features/Students/EventRanking.razor index edb7687..22f7b31 100644 --- a/WebApp/Components/Features/Students/EventRanking.razor +++ b/WebApp/Components/Features/Students/EventRanking.razor @@ -18,7 +18,7 @@ else { - + Students by Rank @@ -64,7 +64,7 @@ else - + Events by Student diff --git a/WebApp/Components/Features/Students/Index.razor b/WebApp/Components/Features/Students/Index.razor index 5275d6c..3abf087 100644 --- a/WebApp/Components/Features/Students/Index.razor +++ b/WebApp/Components/Features/Students/Index.razor @@ -15,7 +15,7 @@ - + - +
@@ -40,7 +40,7 @@ { @context.Item.OfficerRole } - +
- + = Team Captain @@ -78,8 +78,9 @@ var teamsToDisplay = _showRegionalOnly ? context.Item.Teams.Where(t => t?.Event is { RegionalEvent: true }).OrderBy(t => t.Event.Name) : context.Item.Teams.Where(t => t?.Event != null).OrderBy(t => t.Event.Name); - - foreach (var team in teamsToDisplay) + } +
+ @foreach (var team in teamsToDisplay) { var isCaptain = team.Captain != null && team.Captain.Equals(context.Item.Student); var teamMembers = string.Join(", ", team.Students.Select(s => s.FirstName)); @@ -97,7 +98,7 @@ } - } +
diff --git a/WebApp/Components/Features/Teams/Assignment.razor b/WebApp/Components/Features/Teams/Assignment.razor index 22b079d..4c15082 100644 --- a/WebApp/Components/Features/Teams/Assignment.razor +++ b/WebApp/Components/Features/Teams/Assignment.razor @@ -21,7 +21,7 @@ - + Solve - + Students diff --git a/WebApp/Components/Features/Teams/Components/TeamStudents.razor b/WebApp/Components/Features/Teams/Components/TeamStudents.razor index 6f3e4f5..58cba4a 100644 --- a/WebApp/Components/Features/Teams/Components/TeamStudents.razor +++ b/WebApp/Components/Features/Teams/Components/TeamStudents.razor @@ -1,35 +1,37 @@ @using WebApp.Models -@foreach (var student in - Team.Students - .OrderBy(e => - e.EventRankings - .Find(er => er.EventDefinition == Team.Event)?.Rank ?? 10) - .ThenBy(s => s.Grade + s.TsaYear)) -{ - var eventRank = - student.EventRankings - .Find(e => e.EventDefinition == Team.Event)?.Rank; - var color = AppIcons.RankedEventColor(eventRank ?? 0); - var captain = Team.Captain != null && Team.Captain.Equals(student); - var rankLabel = eventRank.HasValue ? $"Rank {eventRank}" : "Unranked"; +
+ @foreach (var student in + Team.Students + .OrderBy(e => + e.EventRankings + .Find(er => er.EventDefinition == Team.Event)?.Rank ?? 10) + .ThenBy(s => s.Grade + s.TsaYear)) + { + var eventRank = + student.EventRankings + .Find(e => e.EventDefinition == Team.Event)?.Rank; + var color = AppIcons.RankedEventColor(eventRank ?? 0); + var captain = Team.Captain != null && Team.Captain.Equals(student); + var rankLabel = eventRank.HasValue ? $"Rank {eventRank}" : "Unranked"; - - - @if (eventRank.HasValue) - { - - } - @student.FirstName - @if (captain && Team.Event.EventFormat != EventFormat.Individual) - { - - } - - -} + + + @if (eventRank.HasValue) + { + + } + @student.FirstName + @if (captain && Team.Event.EventFormat != EventFormat.Individual) + { + + } + + + } +
@code { [Parameter] diff --git a/WebApp/Components/Features/Teams/Create.razor b/WebApp/Components/Features/Teams/Create.razor index 2def5a7..46a1bb5 100644 --- a/WebApp/Components/Features/Teams/Create.razor +++ b/WebApp/Components/Features/Teams/Create.razor @@ -30,7 +30,7 @@ - + @foreach (var evt in _events) diff --git a/WebApp/Components/Features/Teams/Details.razor b/WebApp/Components/Features/Teams/Details.razor index ac9be43..fbee908 100644 --- a/WebApp/Components/Features/Teams/Details.razor +++ b/WebApp/Components/Features/Teams/Details.razor @@ -29,7 +29,7 @@ - + Team Information diff --git a/WebApp/Components/Features/Teams/Edit.razor b/WebApp/Components/Features/Teams/Edit.razor index cb48a9e..06f083d 100644 --- a/WebApp/Components/Features/Teams/Edit.razor +++ b/WebApp/Components/Features/Teams/Edit.razor @@ -29,7 +29,7 @@ - + + @foreach (var student in _students) { diff --git a/WebApp/Components/Features/Teams/Index.razor b/WebApp/Components/Features/Teams/Index.razor index d542039..04a3d59 100644 --- a/WebApp/Components/Features/Teams/Index.razor +++ b/WebApp/Components/Features/Teams/Index.razor @@ -21,7 +21,7 @@ - + - TSA Chapter Organizer - @Configuration["ChapterSettings:Name"] + + TSA Chapter Organizer - @Configuration["ChapterSettings:Name"] +
diff --git a/WebApp/wwwroot/app.css b/WebApp/wwwroot/app.css index 5349abd..31f9bfc 100644 --- a/WebApp/wwwroot/app.css +++ b/WebApp/wwwroot/app.css @@ -56,6 +56,19 @@ display: flex; gap: 0.5rem; align-items: center; + flex-wrap: wrap; +} + +@media (max-width: 600px) { + .page-header-actions { + width: 100%; + margin-top: 0.5rem; + } + + .page-header > div > div:first-of-type { + flex-direction: column; + align-items: flex-start !important; + } } .form-actions { @@ -66,6 +79,17 @@ margin-bottom: 1rem; } +@media (max-width: 600px) { + .form-actions { + flex-direction: column; + width: 100%; + } + + .form-actions > * { + width: 100%; + } +} + .icon-button-hover-error:hover { color: var(--mud-palette-error) !important; } @@ -87,4 +111,21 @@ to { opacity: 1; } +} + +/* Toolbar title responsive styling */ +.appbar-title { + max-width: calc(100vw - 150px); +} + +@media (max-width: 600px) { + .appbar-title { + font-size: 0.875rem; + max-width: calc(100vw - 120px); + } + + /* Reduce base font size on mobile */ + body { + font-size: 0.875rem; + } } \ No newline at end of file