-
Notifications
You must be signed in to change notification settings - Fork 443
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
chore: Refactor top-level search bar #1173
Conversation
✅ Deploy Preview for keda ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thank you for your contribution! 🙏 We will review your PR as soon as possible. While you are waiting, make sure to:
Learn more about:
|
Can you fix the DCO issues please? You can learn more in our contribution guide. |
@zroubalik @JorTurFer which one do you prefer? |
Signed-off-by: PixelRobots <[email protected]>
Signed-off-by: PixelRobots <[email protected]>
Signed-off-by: PixelRobots <[email protected]>
Signed-off-by: PixelRobots <[email protected]>
…re#1174) Signed-off-by: PixelRobots <[email protected]>
I think that fixed it. |
The reason I chose Auth providers is due to the button on the homepage being "Auth providers" |
As now it'll be a top level button, I prefer it shorter as possible to not taking all the screen just for a button |
Thanks a ton! |
Provide a description of what has been changed
Update config.toml to change menu order as in #872
Checklist
Fixes #