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

Treeview missing due to missing setting in header_package.html #7956

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

albert-github
Copy link
Contributor

@albert-github albert-github commented Jan 2, 2024

The PR #7946 was withdraw because the $cookie was not necessary anymore in the header... files as it was part of the of the $treeview. In header_package.html the $treeview has been disables and as such the cookie.js is not added/ Adding the cookie.js explicitly.

The effect was that there was a divider between the treeview and the textual documentation but it was on the far left (and thus invisible), it could be moved but after a refresh it was "gone" again.

The PR CGAL#7946 was withdraw because the `$cookie` was not necessary anymore in the `header...`  files as it was part of the of the `$treeview`.
In `header_package.html` the `$treeview` has been disables and as such the `cookie.js` is not added/
Adding the `cookie.js` explicitly.

The effect was that there was a divider between the treeniew and the textual documentation but it was on the far left (and thus invisible), ith could be moved but after a refresh it was "gone" again.
@sloriot sloriot added Batch_2 Second Batch of PRs under testing Under Testing and removed Batch_2 Second Batch of PRs under testing labels Jan 3, 2024
@sloriot
Copy link
Member

sloriot commented Jan 11, 2024

I don't see any difference between CGAL-6.0-Ic-147 and CGAL-6.0-I-145

@albert-github
Copy link
Contributor Author

Please compare https://cgal.geometryfactory.com/CGAL/Manual_doxygen_test/CGAL-6.0-Ic-147/master/Algebraic_foundations/group__PkgAlgebraicFoundationsRef.html and https://cgal.geometryfactory.com/CGAL/Manual_doxygen_test/CGAL-6.0-I-145/master/Algebraic_foundations/group__PkgAlgebraicFoundationsRef.html when going through:

  • treeview "Package overview"
  • select Reference manual of Algebraic Foundations

147
image

145
image

@lrineau lrineau added this to the 6.0-beta milestone Jan 11, 2024
@lrineau lrineau self-assigned this Jan 11, 2024
@lrineau lrineau added the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Jan 11, 2024
@lrineau lrineau merged commit 22d7ee7 into CGAL:master Jan 11, 2024
8 checks passed
@lrineau lrineau removed the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Jan 11, 2024
@lrineau lrineau deleted the feature/bug_cookie branch January 11, 2024 12:00
@lrineau lrineau added the Doc label Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants