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

Add Accordion always_open property #840

Merged
merged 24 commits into from
Apr 3, 2022

Conversation

glsdown
Copy link
Contributor

@glsdown glsdown commented Mar 27, 2022

As requested in issue #834 I've added support for always_open property of the Bootstrap 5 Accordion. This includes:

  • JS tests
  • Docs
  • Doc tests

I have additionally bumped the react-bootstrap version.

@glsdown
Copy link
Contributor Author

glsdown commented Mar 27, 2022

@tcbegley - R and JL tests for the new accordion callback are failing (test_accordion.py:check_always_open_callback_callbacks). My guess is that the output format for the lists are different to in python (I'm not even sure if the way I am inserting them into the strings works for R or JL). As I don't have them installed, please can you take a look? Thanks!

@glsdown
Copy link
Contributor Author

glsdown commented Apr 2, 2022

@tcbegley - due to the fact that always_open does not work in R, I have removed this from the above and noted in the docs that it does not work with it. This will allow us to release for python and Julia, and revisit at a later date.

@tcbegley tcbegley merged commit b87c349 into facultyai:main Apr 3, 2022
@tcbegley tcbegley mentioned this pull request Apr 3, 2022
@tcbegley tcbegley mentioned this pull request Apr 3, 2022
tcbegley added a commit that referenced this pull request Apr 3, 2022
* Bump react-bootstrap version

* Add always_open prop to Accordion

* Add js tests for Accordion always_open

* Add always_open to Accordion docs

* Add always_open callback doc test

* Remove unnecessary imports

* Add callback file to flake8 ignore files

* Call the second callback test

* Bump react-bootstrap version

* Add always_open prop to Accordion

* Add js tests for Accordion always_open

* Add always_open to Accordion docs

* Add always_open callback doc test

* Remove unnecessary imports

* Add callback file to flake8 ignore files

* Call the second callback test

* Fix some issues with tests

* Remove always_open example in R

* Clean up Accordion source

* Fix accordion tests

* Remove note about R

* Revert debugging change to tests

Co-authored-by: tcbegley <tomcbegley@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants