-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add options to choose where the debug toggle sits #12
Comments
+ 1 for this one. IMHO you don't want this toggle to be around all the time. |
Ah I think I read it incorrectly. I was confused and thought the toggle enables adhoc task debugging 🤦♂️ |
We just had a request from a client to have the icon not visible on the navbar at all. Maybe we have a option in the settings, to choose where this option sits. Have two options - visible all the time like it is now or in the usermenu? |
The other benefit of moving it to the user menu over render_navbar_output is that render_navbar_output just accepts random html, it would be better if we used a callback or hook which accepted an action_link so that no matter where it was injected if would look correct in the current theme. This would need to land first in core: |
There has been some discussion around where the toggle should sit. Some options:
The text was updated successfully, but these errors were encountered: