Skip to content
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

Support using links in parents of nested items #1276

Closed
brc-dd opened this issue Sep 2, 2022 Discussed in #1275 · 1 comment · Fixed by #1279
Closed

Support using links in parents of nested items #1276

brc-dd opened this issue Sep 2, 2022 Discussed in #1275 · 1 comment · Fixed by #1279
Labels
bug Something isn't working theme Related to the theme

Comments

@brc-dd
Copy link
Member

brc-dd commented Sep 2, 2022

Discussed in #1275

Originally posted by erondpowell September 2, 2022
In our folder project, we are trying to use the nested sidebar feature.

  • Each parent is a link to a page. ex: /configuration/data-model.md
  • Each nested page is in a folder. ex: /configuration/data-model/relationships.md

Everything works fine in dev, but on prod testing, nested links are rendered multiple times.
We are not sure if it is a misconfiguration on our end, or a bug in the sidebar:
directus/docs#109

Thanks 🙏

@brc-dd brc-dd added bug Something isn't working theme Related to the theme labels Sep 2, 2022
@brc-dd brc-dd changed the title Nested sidebar renders duplicate links Support using links in parents of nested items Sep 3, 2022
@brc-dd
Copy link
Member Author

brc-dd commented Sep 3, 2022

x-ref: directus/docs#109 (comment)

Currently invalid HTML is being generated. Validator's output:

Start tag a seen but an element of the same type was already open.

image

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working theme Related to the theme
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant