Skip to content

901: center mobile navigation title and login button#905

Merged
RandyJDean merged 1 commit into
mainfrom
901
Apr 23, 2026
Merged

901: center mobile navigation title and login button#905
RandyJDean merged 1 commit into
mainfrom
901

Conversation

@RandyJDean

@RandyJDean RandyJDean commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

901

Description of changes

Centered the mobile navigation drawer title and login button. Verified in mobile view.

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

image

Staging

image

@github-actions

Copy link
Copy Markdown
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Center mobile navigation title.

  • Center login button in drawer.

  • Improve mobile drawer spacing.


File Walkthrough

Relevant files
Enhancement
Header.tsx
Refactor mobile drawer navigation and login button styling

js/src/components/ui/header/Header.tsx

  • Replaced the Drawer's title prop with a custom Text component for the
    "Navigation" title, allowing for centered alignment.
  • Wrapped the login button (renderButton) in a Flex component to center
    it horizontally and add top margin.
  • Applied w="100%" to the main Flex container within the drawer for full
    width.
+7/-3     

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

**🎫 Ticket compliance analysis **

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx Outdated
Comment thread js/src/components/ui/header/Header.tsx Outdated
@github-actions

Copy link
Copy Markdown
Contributor

Commit Validation Failed

The following commits do not start with the required Notion ID 901:

Update js/src/components/ui/header/Header.tsx
Update js/src/components/ui/header/Header.tsx

Please rebase and update your commit messages.
All messages should be of the following format: 901: Example commit

1 similar comment
@github-actions

Copy link
Copy Markdown
Contributor

Commit Validation Failed

The following commits do not start with the required Notion ID 901:

Update js/src/components/ui/header/Header.tsx
Update js/src/components/ui/header/Header.tsx

Please rebase and update your commit messages.
All messages should be of the following format: 901: Example commit

@github-actions

Copy link
Copy Markdown
Contributor

Commit Validation Failed

The following commits do not start with the required Notion ID 901:

Update js/src/components/ui/header/Header.tsx
Update js/src/components/ui/header/Header.tsx

Please rebase and update your commit messages.
All messages should be of the following format: 901: Example commit

1 similar comment
@github-actions

Copy link
Copy Markdown
Contributor

Commit Validation Failed

The following commits do not start with the required Notion ID 901:

Update js/src/components/ui/header/Header.tsx
Update js/src/components/ui/header/Header.tsx

Please rebase and update your commit messages.
All messages should be of the following format: 901: Example commit

@RandyJDean RandyJDean closed this Apr 16, 2026
@RandyJDean RandyJDean reopened this Apr 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Commit Validation Failed

The following commits do not start with the required Notion ID 901:

Update js/src/components/ui/header/Header.tsx
Update js/src/components/ui/header/Header.tsx

Please rebase and update your commit messages.
All messages should be of the following format: 901: Example commit

1 similar comment
@github-actions

Copy link
Copy Markdown
Contributor

Commit Validation Failed

The following commits do not start with the required Notion ID 901:

Update js/src/components/ui/header/Header.tsx
Update js/src/components/ui/header/Header.tsx

Please rebase and update your commit messages.
All messages should be of the following format: 901: Example commit

@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Center mobile navigation drawer title.

  • Center mobile login/action button.

  • Improve mobile drawer layout and spacing.


Diagram Walkthrough

flowchart LR
  Header --> Drawer["Mobile Navigation Drawer"]
  Drawer -- "Custom title rendering" --> FlexTitle["Flex (Centered Navigation Title)"]
  Drawer -- "Centered button wrapper" --> FlexButton["Flex (Centered Login Button)"]
Loading

File Walkthrough

Relevant files
Enhancement
Header.tsx
Refactor mobile drawer title and center action button       

js/src/components/ui/header/Header.tsx

  • Removed the title prop from the Drawer component to allow for custom
    title rendering.
  • Introduced a Text component within a Flex container to explicitly
    display and center the "Navigation" title with specific styling.
  • Wrapped the renderButton call in a Flex container with
    justify="center" to center the login/action button.
  • Modified the renderButton invocation to pass fullWidth: true as an
    object prop, aligning with Mantine's component prop patterns.
+7/-3     

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

**🎫 Ticket compliance analysis **

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx Outdated
Comment thread js/src/components/ui/header/Header.tsx
@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Center mobile navigation drawer title.

  • Center login button in drawer.

  • Improve spacing for cleaner layout.


Diagram Walkthrough

flowchart LR
  HeaderComponent["Header Component"] --> Drawer["Mantine Drawer (Mobile Nav)"]
  Drawer -- "Centers" --> DrawerTitle["Navigation Title"]
  Drawer -- "Centers & Adjusts Spacing" --> LoginButton["Login Button"]
Loading

File Walkthrough

Relevant files
Enhancement
Header.tsx
Enhance mobile navigation drawer layout and centering       

js/src/components/ui/header/Header.tsx

  • Applied styles prop to the Drawer component to center its header
    content and title.
  • Centered the navigation title text, set its font weight to bold,
    adjusted font size using var(--mantine-font-size-lg), and added bottom
    margin with var(--mantine-spacing-xs).
  • Modified the main Flex container within the drawer to occupy 100%
    width.
  • Wrapped the renderButton call (for the login button) in a new Flex
    container to center it horizontally and add top margin.
+15/-2   

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

Fix-Mobile-Menu-View-3387c85563aa80ee9892d789bf2d400e - PR Code Verified

Compliant requirements:

  • Center the mobile navigation drawer title.
  • Center the mobile navigation drawer login button.
  • Improve spacing for a cleaner layout.

Requires further human verification:

  • Verified in mobile view.
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx
Comment thread js/src/components/ui/header/Header.tsx
@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Centers mobile navigation drawer title.

  • Aligns login button in mobile drawer.

  • Improves mobile drawer layout and spacing.


Diagram Walkthrough

flowchart LR
  Header.tsx -- "Adds styles to Drawer" --> DrawerComponent
  DrawerComponent -- "Centers title" --> DrawerTitle
  DrawerComponent -- "Adjusts button container" --> NavigationButtonsFlex
  NavigationButtonsFlex -- "Centers login button" --> LoginButton
Loading

File Walkthrough

Relevant files
Enhancement
Header.tsx
Improve mobile navigation drawer styling and alignment     

js/src/components/ui/header/Header.tsx

  • Applied inline styles to the Drawer component's header and title to
    center the "Navigation" title, set its font weight, size, and bottom
    margin.
  • Modified the Flex container holding navigation buttons to ensure it
    takes w="100%" of the drawer's width.
  • Wrapped the renderButton call (likely for login/logout) in a new Flex
    container to center it horizontally and add top margin.
+15/-2   

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

Fix-Mobile-Menu-View-3387c85563aa80ee9892d789bf2d400e - PR Code Verified

Compliant requirements:

  • Center the mobile navigation drawer title.
  • Center the mobile navigation drawer login button.
  • Improve spacing for a cleaner layout.

Requires further human verification:

  • Verified in mobile view.
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx
Comment thread js/src/components/ui/header/Header.tsx
@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Center mobile drawer title.

  • Align drawer login button.

  • Improve mobile navigation layout.


File Walkthrough

Relevant files
Ui/ux
Header.tsx
Enhance mobile navigation drawer styling and alignment     

js/src/components/ui/header/Header.tsx

  • Applied styles to the Drawer component to center its header and title.
  • Set the main navigation Flex container to w="100%" for full width.
  • Wrapped the renderButton (login/logout) in a new Flex container to
    center it horizontally.
  • Updated the renderButton argument from w-full to full-width.
+15/-2   

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

**🎫 Ticket compliance analysis **

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx
Comment thread js/src/components/ui/header/Header.tsx
@RandyJDean

Copy link
Copy Markdown
Contributor Author

/deploy

@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Center mobile navigation drawer title.

  • Align login button within drawer.

  • Enhance drawer header styling.

  • Adjust drawer content layout.


Diagram Walkthrough

flowchart LR
  HeaderComponent["Header.tsx"] -- "Updates Drawer styles" --> MobileDrawer["Mobile Navigation Drawer"]
  MobileDrawer -- "Centers title & login button" --> UIAlignment["Improved UI Alignment"]
Loading

File Walkthrough

Relevant files
Enhancement
Header.tsx
Improve mobile navigation drawer alignment and appearance

js/src/components/ui/header/Header.tsx

  • Added styles prop to the Drawer component to center the header
    content.
  • Applied specific styling to the Drawer title, including textAlign,
    fontWeight, fontSize, and marginBottom.
  • Updated the main Flex container inside the Drawer to occupy full width
    (w="100%").
  • Wrapped the renderButton call in a new Flex container to center it
    horizontally and add top margin.
  • Modified the argument passed to renderButton from "w-full" to
    "full-width".
+15/-2   

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

Fix-Mobile-Menu-View-3387c85563aa80ee9892d789bf2d400e - PR Code Verified

Compliant requirements:

  • Centered the mobile navigation drawer title.
  • Centered the login button.
  • Improved spacing for a cleaner layout.

Requires further human verification:

  • Verified in mobile view.
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx
Comment thread js/src/components/ui/header/Header.tsx
@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Center mobile navigation drawer title.

  • Style drawer title for better readability.

  • Center and space login button.

  • Improve overall mobile drawer layout.


Diagram Walkthrough

flowchart LR
  User -->|Opens| MobileDrawer["Mobile Navigation Drawer"]
  MobileDrawer -- "Center Header & Title" --> DrawerHeader["Drawer Header"]
  MobileDrawer -- "Center Login Button" --> LoginButton["Login Button"]
  MobileDrawer -- "Adjust Spacing" --> DrawerContent["Drawer Content"]
Loading

File Walkthrough

Relevant files
Ui changes
Header.tsx
Refactor mobile navigation drawer styling and layout         

js/src/components/ui/header/Header.tsx

  • Applied inline styles to the Drawer component to center its header and
    title.
  • Increased font weight and size, and added bottom margin to the drawer
    title.
  • Ensured the inner Flex container for navigation items spans full
    width.
  • Wrapped the login button in a centered Flex container with top margin.
+15/-2   

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

**🎫 Ticket compliance analysis **

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx
Comment thread js/src/components/ui/header/Header.tsx
901: Copilot rebase
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

901: More accessible header navigation
@github-actions

Copy link
Copy Markdown
Contributor

Title

901: center mobile navigation title and login button


PR Type

Enhancement


Description

  • Center mobile navigation drawer title

  • Enhance mobile navigation title styling

  • Center login button in mobile drawer

  • Improve mobile navigation layout spacing


File Walkthrough

Relevant files
Enhancement
Header.tsx
Adjust mobile navigation drawer styling and layout             

js/src/components/ui/header/Header.tsx

  • Added styles prop to the Drawer component to center its header and
    title.
  • Applied specific styling to the drawer title, including textAlign,
    fontWeight, fontSize, and marginBottom.
  • Modified the main Flex container within the drawer to occupy full
    width.
  • Wrapped the renderButton call in a new Flex container to center it
    horizontally and add top margin.
+15/-2   

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

Fix-Mobile-Menu-View-3387c85563aa80ee9892d789bf2d400e - PR Code Verified

Compliant requirements:

  • Center the mobile navigation drawer title.
  • Center the mobile navigation drawer login button.
  • Improve spacing for a cleaner layout.

Requires further human verification:

  • Verified in mobile view.
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread js/src/components/ui/header/Header.tsx
Comment thread js/src/components/ui/header/Header.tsx
@RandyJDean

Copy link
Copy Markdown
Contributor Author

/deploy

@RandyJDean RandyJDean merged commit 0ae1d8e into main Apr 23, 2026
41 of 42 checks passed
@RandyJDean RandyJDean deleted the 901 branch April 23, 2026 19:53
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.

2 participants