Skip to content

Conversation

@raymondkingston
Copy link
Contributor

Extended options with:

  • show/hide wordmark
  • show/hide 'explore nu' menu
  • enter skip link selector value

Also updated UI a little bit.

Copy link

@CeeRoo CeeRoo left a comment

Choose a reason for hiding this comment

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

Approve-iddly-ooved! Just FYI, there's a commented out section in nu_global_elements.php from line 37 to 59, but maybe that was just to keep a historical reference in the file.

@raymondkingston
Copy link
Contributor Author

Approve-iddly-ooved! Just FYI, there's a commented out section in nu_global_elements.php from line 37 to 59, but maybe that was just to keep a historical reference in the file.

Thanks, Chris! Good catch. Just updated with some housekeeping.

Copy link
Contributor

@cam-vanorman cam-vanorman left a comment

Choose a reason for hiding this comment

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

Small change that Chris mentioned, otherwise this looks great! Thanks Ray!

@cam-vanorman cam-vanorman linked an issue Jan 3, 2025 that may be closed by this pull request
@raymondkingston raymondkingston merged commit 8e9806e into main Jan 6, 2025
@raymondkingston raymondkingston deleted the options_updates branch January 6, 2025 20:20
@kenwilliams-nu kenwilliams-nu self-requested a review January 6, 2025 20:41
Copy link
Contributor

@kenwilliams-nu kenwilliams-nu left a comment

Choose a reason for hiding this comment

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

Awesome, job, Ray!

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.

Toggle Wordmark visibility

5 participants