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

Exclude Headings with 'unlisted' Class from Table of Contents #15

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

PxaMMaxP
Copy link

This commit enhances the functionality to selectively exclude headings from the table of contents based on their class.

Changes made:

  • PHP:
    • Implemented logic to identify headings with the 'unlisted' class. These headings are now skipped and not included in the table of contents. This feature provides more control over which headings appear in the generated table of contents.

PxaMMaxP and others added 7 commits November 30, 2023 18:09
Changes
- Configuration:
  - toggle
     Activate the button or not.
  - initially_hide
     Hidden in initial state or not.
  - hide_text
     The text of the button to hide the menu.
  - show_text
     The text of the button to make the menu visible.

- CSS:
   Corresponding classes created for the button (with ID) and for hiding or making visible.

- PHP:
   Added logic to add the button. This also works without a heading text being set.
Added Configurable Show/Hide Button Functionality
…of the heading. In the case of repeated headings (e.g. “Summary”), this leads to an ambiguous ID.

The problem is solved by adding the index of all headings to the ID generated in this case.

Changes to
- PHP:
   If the ID does not exist, the given index is appended to the generated ID.
Unique ID Generation for Headings with Duplicate Titles
… table of contents.

Changes to
- PHP:
   Logic for noting any unlisted classes for the headings running through. If such a class is found, the corresponding heading is not added.
Exclude Headings with 'unlisted' Class from Table of Contents
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

Successfully merging this pull request may close these issues.

None yet

1 participant