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

Fix #1597 Add option to Toggle Nested List Items on Primary Action #1648

Closed
wants to merge 1 commit into from

Conversation

superstructor
Copy link

Fix #1597.

@shaurya947
Copy link
Contributor

@superstructor could you:

  • rebase
  • update the doc page for lists

Thanks!

@oliviertassinari
Copy link
Member

Could you also add the default value for primaryTogglesNestedList to false? Thanks

@superstructor
Copy link
Author

@shaurya947 @oliviertassinari thanks for the feedback. Will do within the next few days.

@oliviertassinari oliviertassinari added PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI and removed Review: review-needed labels Nov 24, 2015
@alitaheri
Copy link
Member

This was resolved by #2390.

@alitaheri alitaheri closed this Dec 6, 2015
@superstructor
Copy link
Author

@subjectix @oliviertassinari Apologies I didn't get back to this. Thanks for finishing it!

@superstructor superstructor deleted the feature/1597 branch December 6, 2015 23:29
@alitaheri
Copy link
Member

@superstructor anytime ^_^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: needs revision The pull request can't be merged. More details is available on the code review or fails in the CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants