Skip to content

Added help navigation highlighting#62

Merged
dutow merged 1 commit intohackorum-dev:mainfrom
moozzi:help_navigation_highlight
Feb 13, 2026
Merged

Added help navigation highlighting#62
dutow merged 1 commit intohackorum-dev:mainfrom
moozzi:help_navigation_highlight

Conversation

@moozzi
Copy link
Contributor

@moozzi moozzi commented Feb 12, 2026

I added a small highlighting system to /help pages navigation.
It highlights the section at the top of screen.

I also found a bug in there.
Navigation links for has:attachment Conditions and has:patch Conditions are not working, because of a differences in href attribute of navigation element (href="#hasattachment-conditions") and id value in content (id="has-attachment-conditions")

.help-outline-link {
&.active {
font-weight: bold;
color: your-active-color;
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a copy-paste mistake, your-active-color doesn't exists in the css? To me it looks like find even with only bolding it, not sure if we need a color change or a border

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right. Fixed

@dutow
Copy link
Collaborator

dutow commented Feb 12, 2026

I also found a bug in there.
Navigation links for has:attachment Conditions and has:patch Conditions are not working, because of a >\differences in href attribute of navigation element (href="#hasattachment-conditions") and id value in content (id="has-attachment-conditions")

Thanks for the bug report, I pushed a fix for this

@moozzi moozzi force-pushed the help_navigation_highlight branch from 45a3221 to 732afc7 Compare February 13, 2026 07:09
@moozzi moozzi force-pushed the help_navigation_highlight branch from 732afc7 to 851f58c Compare February 13, 2026 07:10
@dutow dutow merged commit abc4bbb into hackorum-dev:main Feb 13, 2026
5 checks passed
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