Skip to content

Add side nav for menu navigation on small screens#296

Merged
eamansour merged 3 commits into
galasa-dev:mainfrom
eamansour:em-2424-fix-mobile-view-header
May 7, 2026
Merged

Add side nav for menu navigation on small screens#296
eamansour merged 3 commits into
galasa-dev:mainfrom
eamansour:em-2424-fix-mobile-view-header

Conversation

@eamansour
Copy link
Copy Markdown
Member

Why?

Refer to galasa-dev/projectmanagement#2424

Screenshot(s) of changes

Before:
Screenshot 2026-05-06 at 14 00 53

After:
https://github.com/user-attachments/assets/7d5fb0e0-10fc-46d3-8c8d-c2e1e716904a

Changes

  • Added sidenav that appears when the webui window becomes too small (handled by Carbon automatically), allowing users to access the "Users" and "Test Runs" pages
  • Combined the Galasa logo and "Galasa" header name into a single button rather than having two separate buttons that do the same thing
  • Simplified the language selector and theme switcher components to be more consistent while removing unnecessary divs and CSS by using equivalent Carbon components instead of raw HTML elements
  • Functionality
  • Unit tests

Copy link
Copy Markdown
Member

@jadecarino jadecarino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, just a few minor suggestions!

Comment thread galasa-ui/src/components/headers/PageHeader.tsx Outdated
Comment thread galasa-ui/src/components/headers/LanguageSelector.tsx Outdated
Comment thread galasa-ui/src/components/headers/ThemeSelector.tsx
eamansour added 2 commits May 7, 2026 11:24
Signed-off-by: Eamonn Mansour <47121388+eamansour@users.noreply.github.com>
…nto a separate component

Signed-off-by: Eamonn Mansour <47121388+eamansour@users.noreply.github.com>
@eamansour eamansour force-pushed the em-2424-fix-mobile-view-header branch from 5d58c0a to 0627a51 Compare May 7, 2026 10:25
Signed-off-by: Eamonn Mansour <47121388+eamansour@users.noreply.github.com>
@eamansour eamansour merged commit 8e0999a into galasa-dev:main May 7, 2026
5 checks passed
@eamansour eamansour deleted the em-2424-fix-mobile-view-header branch May 7, 2026 16:15
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