Improve site usability and structure#64
Open
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
Open
Improve site usability and structure#64devin-ai-integration[bot] wants to merge 1 commit intomainfrom
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
Conversation
- Add prominent 'Register' CTA button to both desktop and mobile navigation - Improve footer with 4-column sitemap layout (Tournaments, Resources, Organization, Connect) - Add copyright notice and reorganize footer bottom bar - Fix About page team grid to stack on mobile (grid-cols-1 → md:grid-flow-col) - Add skip-to-content link for keyboard/screen reader accessibility - Add aria-label attributes to nav elements - Remove unused Inter font CSS import - Rename content-wrapper to main-content for semantic skip-link target Co-Authored-By: Oliver Ni <oliver.ni@gmail.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Usability and structure improvements to the BMT website, focused on making key actions more discoverable and improving accessibility fundamentals.
Changes:
<footer>elementaria-labelon<nav>elements#content-wrapper→#main-content(for skip-link target); updated the TOC scroll JS reference in macros.html to matchDesktop nav with Register button:

Mobile menu with Register button:

Footer:

Review & Testing Checklist for Human
https://contestdojo.com/. Confirm this is the correct destination (vs. a BMT-specific registration URL)#content-wrapper→#main-contentID rename affects the JS that highlights the current section in the sidebar TOC/events/bmmt-2026/,/events/bmmt-2026-online/) will need updating each tournament cycle, same as the nav links. Decide if this is acceptable or if you want a different approachNotes
⁄separator before the Register button in desktop nav uses the existing nav separator patternnow()functionDevin Session: https://app.devin.ai/sessions/84d4995094994f54bf2bd04da2cd5d6f
Requested by: @oliver-ni