-
Notifications
You must be signed in to change notification settings - Fork 21
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
Deprecate/Remove merror #70
Comments
My understanding of the status is that
|
That looks good to me. |
Resolved on 2019/06//03 as in #70 (comment) |
Only remaining thing is to write a test to make sure style sheet (color, etc) is being implemented. |
cc @rwlbuis Adding back mathml core label since we still need to write tests. I think we have tests but they follow chromium's style which I'm not sure is aligned on the spec. Also reputting need resolution and spec update in case people want to bikeshed on the "Is the suggested merror { outline: solid thin red; background-color: lightYellow; } what we want?" |
This doesn't seem worth bikeshedding as it is at best a suggestion. I'm not sure how there could be something nomative in that? Could there? |
well, we just need to decide something to put in the UA stylesheet. If everybody is ok with "merror { outline: solid thin red; background-color: lightYellow; } " then we can choose that. |
Also, merror affects some tests so we need to decide a default value. Authors/Users can always change it if they don't like, so I don't think the exact values matters. Chromium has the following rules: Gecko has the following rule: WebKit has this: It's better not to modify the text style and we don't really explain how to handle outline in the core spec, so a yellow background and border (with optionally padding/margin) seems best. |
…g. (#23794) * Update style for merror and add a basic test for the default rendering. w3c/mathml#70 * fix bad markup
test added |
…test for the default rendering., a=testonly Automatic update from web-platform-tests Update style for merror and add a basic test for the default rendering. (#23794) * Update style for merror and add a basic test for the default rendering. w3c/mathml#70 * fix bad markup -- wpt-commits: a1bdc5baa9295e462601d2e8631471ef4075b7c1 wpt-pr: 23794
…test for the default rendering., a=testonly Automatic update from web-platform-tests Update style for merror and add a basic test for the default rendering. (#23794) * Update style for merror and add a basic test for the default rendering. w3c/mathml#70 * fix bad markup -- wpt-commits: a1bdc5baa9295e462601d2e8631471ef4075b7c1 wpt-pr: 23794
Opening this to discuss whether we should deprecate/remove merror from MathML core/full.
It seems to me generally people won't publish content with errors, unless they were generated from e.g. LaTeX or Computer Algebra system.
The text was updated successfully, but these errors were encountered: