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 panel toc editor #198

Merged
merged 4 commits into from
Aug 29, 2024
Merged

Remove panel toc editor #198

merged 4 commits into from
Aug 29, 2024

Conversation

TylerZeroMaster
Copy link
Contributor

@TylerZeroMaster TylerZeroMaster commented May 28, 2024

depends on openstax/ce#1910

related to openstax/ce#1706
related to openstax/ce#1484
related to openstax/ce#2110
related to openstax/ce#1967

@TylerZeroMaster TylerZeroMaster force-pushed the remove-panel-toc-editor branch from 375466c to 6420033 Compare August 8, 2024 14:29
@TylerZeroMaster TylerZeroMaster marked this pull request as ready for review August 8, 2024 14:38
@TylerZeroMaster TylerZeroMaster requested a review from a team August 8, 2024 14:38
@TylerZeroMaster TylerZeroMaster self-assigned this Aug 8, 2024
Copy link
Contributor

@sparksam sparksam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

TocTreesProvider and TocTreeItem were only used in tests

Move toggleTocTreesFilteringHandler to book-tocs
@TylerZeroMaster TylerZeroMaster force-pushed the remove-panel-toc-editor branch from 8fca038 to dccbb81 Compare August 29, 2024 18:42
@TylerZeroMaster TylerZeroMaster merged commit de8a57e into main Aug 29, 2024
1 check passed
@TylerZeroMaster TylerZeroMaster deleted the remove-panel-toc-editor branch August 29, 2024 18:45
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.

2 participants