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

Update painting.html stroke-width #935

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JimJJewett
Copy link

(1) Specify that the stroke-width is centered on the path, rather than inside or outside the shape. (2) Correct(?) the stroke-width of a dash.

(A) It is expected that the location of the stroke's own paint will be controllable in the next SVG edition, but the default/initial position will still need to be defined.

(B) The algorithm for painting a dash collects all points on a line perpendicular to the path that are within a distance of stroke-width. Because the path has two sides, this effectively doubles the stroke-width. Either that should be corrected (as I proposed here) or it should be made explicit that dashes are double-thick.

(1)  Specify that the stroke-width is centered on the path, rather than inside or outside the shape.
(2)  Correct(?) the stroke-width of a dash.

(A)  It is expected that the location of the stroke's own paint will be controllable in the next SVG edition, but the default/initial position will still need to be defined.

(B)  The algorithm for painting a dash collects all points on a line perpendicular to the path that are within a distance of stroke-width.  Because the path has two sides, this effectively doubles the stroke-width.  Either that should be corrected (as I proposed here) or it should be made explicit that dashes are double-thick.
@JimJJewett
Copy link
Author

I consider this an editorial clarification, which I think is classified as non-substantive. It isn't clear whether I can (or even should be able to) make that classification.

Alternatively, I'm probably willing to sign the appropriate waivers, though I didn't see a link to what they are. (Apologies if they are visible to someone less oblivious and/or not using a high-contrast viewer.)

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.

1 participant