Skip to content

CSS Transforms Module Interactive Example #160

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

Merged
merged 5 commits into from
Dec 8, 2023

Conversation

zfox23
Copy link
Contributor

@zfox23 zfox23 commented Dec 4, 2023

(The below is copy/pasted from the Content PR's text.)

Description

  1. Improves the existing CSS Transforms module with a powerful new interactive example and additional prose for context.
  2. Implements a new guide related to CSS transforms called "Styling animated buttons with CSS transforms"

Motivation

Main motivation: Improving our documentation as a part of the Interop 2023 project.

Related issues and pull requests

Once these three PRs are merged, mdn/mdn#488 will be resolved.

Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

This is a fun demo. I think the UI could be simplified a bit, keeping it just as useful, but less busy. That would help with all the a11y features that will need to be added if we do keep all the UI elements.

The z0 doesn't get updated or change. It's helpful to know where zero is; explaining it in the text would be helpful.

Also, a toggle for backface-visibility and sliders for perspective-origin would be awesome.

Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

had two windows open. not sure if this edit came thru

@zfox23
Copy link
Contributor Author

zfox23 commented Dec 7, 2023

Thanks for your feedback. It should all be addressed now.

Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

Fantastic!

@estelle estelle merged commit fd4c867 into mdn:main Dec 8, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants