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

Remove the HTML outline #269

Closed
rdeltour opened this issue Aug 8, 2019 · 4 comments
Closed

Remove the HTML outline #269

rdeltour opened this issue Aug 8, 2019 · 4 comments

Comments

@rdeltour
Copy link
Member

rdeltour commented Aug 8, 2019

The outline view currently displays 3 outlines side by side:

  • the ToC from the Nav Doc
  • the headings outline
  • the outline from the HTML outline algorithm

This last outline is basically useless for accessibility (it's not implemented by any browser or AT), and was mostly added there for people who found it useful as a way to check their markup structure. As it's probably more confusing than helpful for most users, I suggest we simply remove it for now, and maybe hide it as an opt-in feature later.

@danielweck
Copy link
Member

Should this issue be moved to the ace-gui repository?

@rdeltour
Copy link
Member Author

rdeltour commented Aug 9, 2019

Should this issue be moved to the ace-gui repository?

No, the data gathering is part of ace-core, and the rendering is part of ace-report.

That said, once the outline is removed, we'll probably need to also adapt the GUI code indeed.

@danielweck
Copy link
Member

I see, I initially thought this was only an issue in the GUI, but I do agree it is problematic in the HTML report too. Thanks for clarifying!

@danielweck
Copy link
Member

Related issue: #349

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants