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

implement transform-origin #157

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Conversation

adamwong246
Copy link

@adamwong246 adamwong246 commented Feb 22, 2022

fixes #156

Implements the transform-origin property as documented in the following sources:

Our team needed this feature, so I hope you'll find it useful as well. The work is ATM incomplete but suffices for our team. I'll be doing the pro-bono work needed to get this PR merged back upstream- let me know what else is required. A very narrow test has been made and the results are checked in as a pdf matching the official test image

Work left to do:

  • The property can only be defined as 2 absolute or 2 percentage values, not with single values or with keywords.
  • There's a problem where the classlist is returned as a list of strings, rather than a "live DOMTokenList collection". I suspect it is a web vs node thing but I need to investigate.

Also I added node_modules to the .gitignore ;-)

@adamwong246 adamwong246 marked this pull request as draft February 22, 2022 03:10
@adamwong246 adamwong246 changed the title implement transform-origin (work in progress) implement transform-origin Feb 22, 2022
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.

transform-origin is not respected
1 participant