Skip to content

Remove restriction on mathml deprecated features#2137

Merged
mattgarrish merged 4 commits intomainfrom
fix/issue-2118
Apr 1, 2022
Merged

Remove restriction on mathml deprecated features#2137
mattgarrish merged 4 commits intomainfrom
fix/issue-2118

Conversation

@mattgarrish
Copy link
Member

@mattgarrish mattgarrish commented Mar 28, 2022

I've also added the general caution from #2118 to the top of the relationships to other specifications section and removed the individual caution boxes from the HTML and SVG subsections.

I've also added a brief "relationship to mathml" section to note that we only directly support presentation markup.

Fixes #2118


Preview | Diff

add general caution about incompatible changes in underlying technologies;
fix misordered change log entries
@mattgarrish mattgarrish requested a review from iherman March 28, 2022 13:17
@mattgarrish
Copy link
Member Author

Since we're removing the restriction, probably another to get a WG resolution on before merging.

@iherman iherman added the Agenda+ Issues that should be discussed during the next working group call. label Mar 30, 2022
@mattgarrish mattgarrish merged commit af44283 into main Apr 1, 2022
@mattgarrish mattgarrish deleted the fix/issue-2118 branch April 1, 2022 00:32
@iherman
Copy link
Member

iherman commented Apr 1, 2022

The issue was discussed in a meeting on 2022-03-31

List of resolutions:

View the transcript

2. Remove restriction on mathml deprecated features (pr epub-specs#2137)

See github pull request epub-specs#2137.

See github issue epub-specs#2118.

Matt Garrish: duga had opened this initially, and we went back and forth.
… i think his concern is that we're imposing a restriction on MathML's deprecated features that MathML itself doesn't impose on its features.
… they are deprecated, but you're still allowed to have them in MathML.
… we are imposing on their spec.
… he's asked that we not restrict these deprecated features.
… I agree, and I don't think us putting a ban on MathML features moves the needle on MathML rendering.
… math rendering is made for web browsers, and we just use what they do.
… if getting rid of these restrictions smooths over a spec weirdness, that is fine.
… people don't hand write MathML anyway.
… so this PR takes out the MUST NOT that was there before.

Dave Cramer: this is in keeping with changes we've been making incrementally over the last 5-10 years.
… initially we were intervening with the goal of making it easier for people who were trying to write RS from scratch.
… to allow them to skip support for complicated features.
… this PR is in keeping with our take on epub as a container for these other technologies.
… we leave it up to these other specs.

Matt Garrish: I don't think these restrictions work anymore, but they were important at the time.

Dave Cramer: so I would propose we merge this.

Murata Makoto: +1.

Proposed resolution: merge PR 2137, close 2118. (Dave Cramer)

Dave Cramer: +1.

Shinya Takami (高見真也): +1.

Masakazu Kitahara: +1.

Matt Garrish: +1.

Toshiaki Koike: +1.

Matthew Chan: +1.

Resolution #2: merge PR 2137, close issue 2118.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Agenda+ Issues that should be discussed during the next working group call.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Does the restriction on Deprecated MathML cause retroactive content breakage?

2 participants