Skip to content

Original Sidebar + Triage Redesign #813

Open
BrendannnD wants to merge 2 commits intomasterfrom
SidebarCodeReview
Open

Original Sidebar + Triage Redesign #813
BrendannnD wants to merge 2 commits intomasterfrom
SidebarCodeReview

Conversation

@BrendannnD
Copy link
Copy Markdown
Contributor

@BrendannnD BrendannnD commented Mar 3, 2026

This pull request has the base redesign with the original triage redesign, new sidebar, and footer redesign. Note that this is only the original redesign and it has since had many more updates (including further triage improvements, sidebar improvements, and redesigns for the rest of the pages). It also standardizes Python script execution to use python3 throughout the Java backend and utility code to fix errors trying to start servers. The most important changes are grouped below.

UI Redesign and Layout Updates:

  • Introduced a new sidebar navigation by adding sidebar.scala.html and updating the main layout (main.scala.html) to use the sidebar instead of the old header, reorganizing page structure for better usability and accessibility. Added Google Fonts and Material Symbols for a modern look. [1] [2]
  • Refactored the search partial to integrate with the sidebar, improve accessibility, and simplify code. Each page now has a unique search field ID and button with proper ARIA labels.
  • Redesigned the footer for a cleaner appearance and better alignment with the new layout, including new CSS classes and improved structure.
  • Updated the triage input button partial for improved styling and clarity, wrapping gender labels in a span and adding new CSS classes.

Backend Consistency and Python 3 Migration:

  • Standardized all Java backend and utility code to use python3 instead of python when invoking Python scripts, ensuring compatibility with modern Python environments. [1] [2] [3] [4] [5]

<link rel="stylesheet" href="@assets.path("css/libraries/jquery.dataTables.min.css")">
<link rel="stylesheet" href="@assets.path("css/libraries/bootstrap.min.css")">
<link rel="stylesheet" href="@assets.path("css/femr.css")"/>
<link rel="stylesheet" href="@assets.path("css/redesign.css")"/>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe mention here in a todo so we remember to remove the femr.css after we are done

@@ -0,0 +1,782 @@
.femr-triage {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe we use the tailwind format for css?

@lab596
Copy link
Copy Markdown
Contributor

lab596 commented Mar 3, 2026

Overall maybe lets not add underline on hover for icons

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