Tidy up attributes appendix #225
Comments
AmeliaBR
changed the title
|
That row should actually be removed, and x/y/width/height for image listed as presentation attributes. Except we seem to have dropped the section of the table that listed the geometry presentation attributes & which elements they apply to. (And that's a rather important distinction, since some attributes by those names will affect CSS computed styles and some won't!) There are a number of other issues I noticed in there, so I'm broadening this issue to be a general clean-up. |
|
There are three more presentation attributes listed: |
It looks like that table has been moved to https://svgwg.org/svg2-draft/styling.html#PresentationAttributes. I would merge it with the property index. |
Major changes: - defines consistent layout rules for all embedded elements using the geometry properties (as discussed in #237) - removes redundant/conflicting definitions of x/y/width/height (closes #237) - explicitly states that min/max-width/height properties can be used to constrain width/height (closes #238) - defines the behavior of `object-fit` and `object-position` for embedded elements, including its interaction with `preserveAspectRatio` for `image` (closes #239) - update the definition of `preserveAspectRatio` for `image` to be consistent with CSS sizing - add clarifying note about the behavior of `overflow` on image and why it was changed with respect to SVG 1.1 and embedded SVG images - Add warning about the HTML parser not currently supporting HTML embedded content elements in SVG fragments (as discussed in #240) - Re-organized and updated the `foreignObject` section to reflect that XML does not rule the world; added warning about `requiredExtensions` not currently being defined in any useful way. - Moved the section on foreign namespaced private data to the Document Structure chapter. - Updated the note on `<base>` and its impact on cross-references to reflect changes made elsewhere. - Re-organized the "presentation attributes" table in the Styling chapter, updating to match changes elsewhere (addresses some of the issues discussed in #225) - Change the `overflow` rules so that `scroll` is expected to behave as normal for `foreignObject` and `iframe`. - Updated the bounding box algorithm for the `clipped` flag to reflect all types of clipping on all types of elements (as discussed in #243) - Updated changes appendix to match. -
|
Listing all elements on which an attribute can be specified looks like overkill to me. I suggest to replace them by element categories and link them to their definitions (which already exist for some of them). Examples:
Sebastian |
dirkschulze
modified the milestones:
SVG 2.1 Working Draft,
SVG 2.0 Proposed Recommendation
|
Marking it for SVG 2.0 to clean-up the table but not necessarily fixing the actual proposal. If we get to it for 2.0, great! If not, will be a future item. |
|
Checked the necessary parts and couldn't find changes required for SVG 2.0. Moving back to 2.1 |
dirkschulze
modified the milestones:
SVG 2.0 Proposed Recommendation,
SVG 2.1 Working Draft
The table on https://svgwg.org/svg2-draft/attindex.html needs to be updated so that
widthandheightattributes on<image>are marked as animatable.The text was updated successfully, but these errors were encountered: