Skip to content

Conversation

@choldgraf
Copy link
Member

No description provided.

@choldgraf choldgraf force-pushed the toggle_admonition branch from b4ce539 to a918a6a Compare June 9, 2020 16:03
@choldgraf choldgraf force-pushed the toggle_admonition branch from a918a6a to 8cebb39 Compare June 9, 2020 16:24
@choldgraf choldgraf merged commit c138989 into master Jun 9, 2020
@choldgraf choldgraf deleted the toggle_admonition branch June 9, 2020 17:26
rkdarst added a commit to rkdarst/sphinx-togglebutton that referenced this pull request Sep 12, 2020
- I was trying to use this extension, and got confused because the
  readme wasn't updated for the new `dropdown` class and
  `toggle-shown` semantics (as opposed to being shown by default with
  toggle-hidden as an option).  Ref: 8cebb39, executablebooks#13
- I fixed it to save others some time, since this took quite a while
  to figure out...
- Review: make sure it's correct, I didn't miss anywhere, this is what
  the future is.
  - also fix the "container" part at the bottom - should this be
    updated to the toggle directive or is that still true?
choldgraf pushed a commit that referenced this pull request Sep 12, 2020
- I was trying to use this extension, and got confused because the
  readme wasn't updated for the new `dropdown` class and
  `toggle-shown` semantics (as opposed to being shown by default with
  toggle-hidden as an option).  Ref: 8cebb39, #13
- I fixed it to save others some time, since this took quite a while
  to figure out...
- Review: make sure it's correct, I didn't miss anywhere, this is what
  the future is.
  - also fix the "container" part at the bottom - should this be
    updated to the toggle directive or is that still true?
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