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

Correct orientation interaction docs and add usage examples to ssr demo #154

Merged
merged 1 commit into from Sep 2, 2020

Conversation

zephraph
Copy link
Contributor

@zephraph zephraph commented Sep 2, 2020

This is a small start to help correct some of the documentation we had around orientation interactions. Specifically prepending not all and is an important addition to ensure the negation of other potential orientation matches.

We'll follow up with #153 to improve the docs further.

@alloy
Copy link
Collaborator

alloy commented Sep 2, 2020

Specifically prepending not all and is an important addition to ensure the negation of other potential orientation matches.

Yes, in this specific case, but some other interaction rules may not require it, which is why we ended up not adding that negation by default. (It was added by default in earlier versions, but I think it was due to the definition of the hover interactions that I realised we do not always want negation for interaction rules.)

Copy link
Collaborator

@alloy alloy left a comment

Choose a reason for hiding this comment

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

Unsure if this really needs a patch bump, so I’ll leave merging to you.

@zephraph
Copy link
Contributor Author

zephraph commented Sep 2, 2020

Definitely don't need a bump because we're not changing any runpath code here.

Thanks for the 🍐!

@zephraph zephraph merged commit b828429 into master Sep 2, 2020
@zephraph zephraph deleted the fix-orientation branch September 2, 2020 14:11
@artsyit
Copy link
Contributor

artsyit commented Sep 14, 2020

🚀 PR was released in v1.2.2 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants