UI3

[garyhtou]

The UI3 ship will comprise the following PRs (all combined into this PR):

• [UI] Visual refresh across core components #7741
• [UI] Add new icons #7773
• Organization home page Redesign home page #7105 (was already merged into main under the event_home_page_redesign_2024_09_21 feature flag)
  • Home page improvements #8179
• Redesign navigation #7828

---

• Closes [UI3] Compact sidebar, add in org switcher dropdown #7492
• Closes [UI3] Improve mobile navigation #7405

---

TODO

BEFORE MERGING, REMOVE UI3 SPECIFIC CODE (THE TODO COMMENT)

[garyhtou]

The redesigned navigation as of #7828 isn't complete, but I've merged it into this branch so that it's easier for Lachlan to build their #7741 work on top of it.

[Copilot]

Pull Request Overview

This PR implements a comprehensive UI3 redesign for HCB, modernizing the interface with improved navigation, responsive design, and visual consistency. The changes encompass major layout restructuring, navigation component updates, and numerous styling improvements across the application.

• Complete navigation redesign with mobile-responsive components
• Modernized settings interfaces with new header components
• Improved visual hierarchy and spacing throughout the application

Reviewed Changes

Copilot reviewed 138 out of 202 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
users/edit_*.html.erb	Updates user settings navigation and header components
users/_*.html.erb	New navigation components and user interface elements
events/*.html.erb	Event-related UI improvements and navigation updates
application/_*.html.erb	Core layout and component restructuring
layouts/*.html.erb	Major layout overhauls for responsive design
Various view partials	Styling improvements and component updates

Comments suppressed due to low confidence (1)

app/views/users/_payout_form.html.erb:35

• [nitpick] The class name 'hidden' is too generic and could conflict with other CSS frameworks. Consider using a more specific class name like 'display-none' or 'payout-section-hidden'.

  <section data-behavior="check_payout_method_inputs" class="<%= "hidden" unless user.payout_method.instance_of?(User::PayoutMethod::Check) %>">

[manuthecoder]

let the testing begin 🧙💣💥

[sampoder]

A couple of quick things from me:

This feels really empty? Can we put HCB by Hack Club there or something?

The grant invitation feels like it needs a refresh:

I'll keep playing around with it over the next few days to find things!

[sampoder]

Another quick thing!

[manuthecoder]

[Copilot]

Pull Request Overview

Copilot reviewed 133 out of 198 changed files in this pull request and generated 8 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}