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

Concepts do not respect groups in tree view #8498

Closed
eseiler opened this issue Apr 13, 2021 · 5 comments
Closed

Concepts do not respect groups in tree view #8498

eseiler opened this issue Apr 13, 2021 · 5 comments
Labels
bug C++20 For features introduced in the 2020 spec of C++

Comments

@eseiler
Copy link

eseiler commented Apr 13, 2021

Describe the bug
Related to #2732
Usually, if some entity is in added to a group, the entity is also attached to the group in the tree view.
This does not seem to be the case with concepts.
This also causes some trouble if you DoxygenLayout.xml does not handle concepts yet, as it jump to the root in the tree view.

Expected behavior
Stay in group.

Screenshots
Clicking on something that is not a concept:
stay
Clicking on something a concept (jumps to the concept part of the tree instead of staying in the modules part):
no_stay

To Reproduce
example.tar.gz

Version
1.9.2 (e9185650216f3c3fa97759caf67ee79db66cb5e1)

@albert-github
Copy link
Collaborator

You wrote:

This also causes some trouble if you DoxygenLayout.xml does not handle concepts yet, as it jump to the root in the tree view.

You mean that you have an old DoxygenLayout.xml in your run directory?
Most likely cause is that in that case there are no concepts in that file and for the attribute visible the default value of no will be taken, but also the warning form the documentation:

Warning
at the moment you should not remove elements from the layout file as a way to hide information. Doing so can cause broken links in the generated output!

should be taken into consideration.

@eseiler
Copy link
Author

eseiler commented Apr 13, 2021

You mean that you have an old DoxygenLayout.xml in your run directory?
Most likely cause is that in that case there are no concepts in that file and for the attribute visible the default value of no will be taken, but also the warning form the documentation:

Yes, in the actual project we need to make concepts visible at least within groups to be able to see them. And there it will just jump to the root on click, because there is no file (?) concepts layout configured.

In the example it's the default Layout, and it still jumps somewhere else.

@doxygen
Copy link
Owner

doxygen commented Apr 13, 2021

@eseiler I've just pushed a fix for this. Please verify if it works for you.

@albert-github albert-github added bug C++20 For features introduced in the 2020 spec of C++ fixed but not released Bug is fixed in github, but still needs to make its way to an official release labels Apr 14, 2021
@eseiler
Copy link
Author

eseiler commented Apr 14, 2021

Thank you, works perfectly!

@doxygen
Copy link
Owner

doxygen commented Aug 18, 2021

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.2.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Aug 18, 2021
@doxygen doxygen closed this as completed Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug C++20 For features introduced in the 2020 spec of C++
Projects
None yet
Development

No branches or pull requests

3 participants