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
positioning at line anchors is disturbed by the navigation bar #8648
Comments
You wrote:
|
The link is on the page std::result_of.
The end position is line 1, not line 02373. It is incorrect. It changes after executing this line in content.css({height:windowHeight + "px"}); The navigation bar is open but empty at this point.
The problem happens in Konqueror too.
I do not know how to do that. I have asked @jwakely to answer your questions.
No, I have no access to the API documentation typeset using the master version.
I do not know how to do it. Please understand that you have to build gcc in order to build the API documentation and that takes days to accomplish. |
…vigation bar It looks like the line numbering uses as anchor: ``` < name=... ``` instead of ``` <a id=... ``` See also: https://www.w3schools.com/tags/tag_a.asp and https://www.w3schools.com/tags/ref_standardattributes.asp > id Specifies a unique id for an element and the `name` is not mentioned as attribute.
I looked at the resulting HTML code of https://gcc.gnu.org/onlinedocs/libstdc++/libstdc++-api.20210701.html/a00221_source.html#l02373 and see at the mentioned line:
Looking at the code for a doxygen anchor I see that here not
and the Based on this information we can easily see the problem with other problems a well (any file will do when using (I vaguely remember that I've seen this problem in the past as will, but couldn't quickly find a reference to it). I've just pushed a proposed patch, pull request #8650 |
No it doesn't |
@jwakely Maybe you can try to compile the doxygen master with the proposed patch and see whether you still see the problem I, tried to, fix. |
Of course it does not when you have a supercomputer. |
While your change is obviously an improvement, it should be noted that Chrome does support navigating to a named anchor, which is clearly seen from the fact that it initially positions itself correctly but loses it in the process of opening the navigation bar. |
It doesn't take days on a normal laptop either. I do it all the time. Maybe you're doing it wrong. |
Or maybe I am not rich enough to have a normal laptop. |
@yecril71pl What do you mean with:
will the if not work or do you foresee other problems ? |
It just doesn't take days, unless you do something silly like build support for every language (which is not necessary for the libstdc++ docs) without using Make's Or just do this:
I just tried this, and it took a minute for the configure steps and another two for the doxygen step. Not days. |
The |
The documentation says I have to run the compiler build. Could you fix that?
Thank you very much, so I shall. I am in the process of fixing |
…vigation bar - some more places where for an anchor no `id=` was used - made usage consistent (i.e. everywhere `id=` and `name=`)
@yecril71pl you write:
With the, small test I did this does not happen and the position is at the right place.
In the mean time I found a few more places with potential problems, added these changes to the proposed patch #8650 (Note: depending on the number of cores in the system the creation of the documentation might take a bit longer due to the number of images to be generated). |
Common logic.
Yes, I did.
It worked. Corollary: Google Chrome defies logic. Why am I surprised? |
issue #8648 positioning at line anchors is disturbed by the navigation bar
Code has been integrated in master on github (please don't close the issue as this will be done at the moment of an official release). |
FWIW the https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a#attr-name |
Thanks, So Chrome and Edge are front-runners. |
We publish it as XHTML 1.0 Transitional, deprecation in HTML 5 does not apply. |
This issue was previously marked 'fixed but not released', |
Describe the bug
Navigating to libstdc++ does initially scrolls to line 02373 but the scrolling gets undone when the navigation bar loads.
Expected behavior
The browser should scroll to line 02373 and stay there.
To Reproduce
Version
1.9.1
Mention the version of doxygen used (output of
doxygen --version
) and the platform on which you run doxygen (e.g. Windows 10, 64 bit). If you run doxygen under Linux please also mention the name and version of the distribution used (output oflsb_release -a
) and mention if you compiled doxygen yourself or that you use a binary that comes with the distribution or from the doxygen website.Additional context
Originally reported downstream.
The text was updated successfully, but these errors were encountered: