-
Notifications
You must be signed in to change notification settings - Fork 381
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: fixes anchor links #1516
docs: fixes anchor links #1516
Conversation
✅ Deploy Preview for tetragon ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, so those are values that will need to be manually edited if we edit the height of the navbar? could you add a comment to explain that :) thanks! checkpatch finds your commit too long, I think it's missing an empty line between the commit title and the sign-off
@mtardy Yes, that's correct. If the header height changes, then these values need to be adjusted to maintain the same visual quality. Thank you for the feedback. I will leave comments in the code. |
026c981
to
ec13bb1
Compare
Can you squash your commits into the first one? |
Signed-off-by: Evgenii Sazhin <prosazhin@gmail.com>
0486c35
to
7fec0be
Compare
@mtardy Done, ready to check |
Thanks a lot! |
This PR fixes anchor links in docs. Preview