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

Clean up definitions around transforms on root SVG element #360

Open
boggydigital opened this issue Nov 9, 2017 · 3 comments

Comments

@boggydigital
Copy link
Contributor

commented Nov 9, 2017

Splitting SVG part of w3c/csswg-drafts#894

What part of the SVG spec needs to be cleared up?

  • Somewhere it should state that the SVG root element is a CSS layout
    box for the purposes of transformations. Probably in the same place where
    we state that backgrounds, borders, and other properties apply on the root
    element.

  • In the SVG view spec (linking chapter), it should state that an
    svgView transformation applies in the same way as a transformation on the
    root element.

  • In the SVG view spec, it should clarify whether the transformation

    • is in addition to the transformation defined in the SVG file (e.g.,
      take whatever would normally be drawn, and transform it)
    • OR, a replacement for it, as if it was an !important style override
      (all the other svgView parameters override values specified in
      the SVG file)
@boggydigital

This comment has been minimized.

Copy link
Contributor Author

commented Jun 11, 2018

Not blocking updated 2.0 CR publication - assigning 2.0 Recommendation milestone to clean this up before 2.0 REC

@AmeliaBR

This comment has been minimized.

Copy link
Contributor

commented Jul 28, 2019

While working on #56, I realized that we never finished this. I also realized that it's even more complicated than I previously thought, because currentScale/currentTranslate need to factor in, too.

Demo/test case: https://bronze-soup.glitch.me/

AmeliaBR added a commit to AmeliaBR/svgwg that referenced this issue Aug 11, 2019
Remove zoomAndPan attribute
Closes w3c#56.

Adds a note about the legacy spec,
in the context of currentScale and currentTranslate.

Remaining ambiguities about these IDL properties
are covered in w3c#360.
An inline issue is included.
ewilligers added a commit that referenced this issue Aug 30, 2019
Remove zoomAndPan attribute (#724)
Closes #56.

Adds a note about the legacy spec,
in the context of currentScale and currentTranslate.

Remaining ambiguities about these IDL properties
are covered in #360.
An inline issue is included.
@AmeliaBR

This comment has been minimized.

Copy link
Contributor

commented Sep 1, 2019

Another thing to be included in this discussion: the implicit transformations created by vector-effect values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.