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

Display hide hint text when admonition is expanded #59

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

gabalafou
Copy link
Contributor

@gabalafou gabalafou commented Dec 20, 2023

There is a puzzling inconsistency between the toggle button created with the toggle directive, versus one created via an admonition.

For the toggle-directive button, you click "Click to show", the widget expands, and the text is replaced with "Click to hide".

But for the admonition-directive button, you click "Click to show", the widget expands, and the text goes away.

You can corroborate this behavior at the docs.

This PR makes the admonition behavior match the toggle-directive behavior.

Copy link

welcome bot commented Dec 20, 2023

Thanks for submitting your first pull request! You are awesome! 🤗

If you haven't done so already, check out EBP's Code of Conduct and our Contributing Guide, as this will greatly help the review process.

Welcome to the EBP community! 🎉

Copy link
Member

@choldgraf choldgraf left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for improving consistency here

@choldgraf choldgraf merged commit f468ca6 into executablebooks:master Dec 22, 2023
2 checks passed
Copy link

welcome bot commented Dec 22, 2023

Congrats on your first merged pull request in this project! 🎉
congrats

Thank you for contributing, we are very proud of you! ❤️

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