-
Notifications
You must be signed in to change notification settings - Fork 22
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 'all' option and display labels if selected #172
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for neo4j-docs-ui 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.
I think it is useful to add an "All" option, even though this is going to be exactly the same as "Neo4j Enterprise Edition"? (maybe it won't always match EE though, which future proofs any decisions that come down the pipeline)
Just an observation - if you select All
in the built page, the link doesn't work - was that expected?
It wasn't expected as such, but the logic of whether or not it tries to update the URL could be improved. |
When an ID is set on a block that is part of a tabbed group the tab switching and scroll happens as it does when a user clicks on a tab.
``` :page-banner: note :page-banner-text: Lorem ipsum dolor sit est. :page-banner-link: https://neo4j.com/docs :page-banner-link-text: Link text ``` `note` is the default, `warning` can also be used.
Usage `[role=label--*]` Can be existing labels eg `[role=label--enterprise-edition]` or action-version labels such as `[role=label--new-5.11]` or `[role=label--deprecated-5.13]` - Roles are converted to labels. - Roles applied to headings and table captions are displayed inline, right-aligned. - Roles on block level elements are displayed before the corresponding block element.
Fixes a few issues in the initial implementation of neo4j-documentation#211: - adds `cluster` and `dynamic` labels - fixes padding in admonitions
…pher query parameters.
…ation#226) I think it's a bad UX if the page moves unexpectedly. When looking at examples, people may want to look at them together with the text leading to it, and they have to manually scroll back up to.
Allows for the addition of an
all
option in the products dropdown.all
is selectedlocalhost
is the url.hostall
is selected