1138455 - [Compat Data][Importer] Allow (without prefix) support text

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: General, enhancement)

Description

[John Whitlock [:jwhitlock]]

What problems would this solve?
===============================
On many MDN pages, when a feature was supported with a prefix but a later version supported it without a prefix, the text "(without prefix)" appears.  This is currently registered as a parse issue.

Who would use this?
===================
MDN staff cleaning up compat data issues

What would users see?
=====================
MDN staff would not see "(without prefix)" as unknown support text

What would users do? What would happen as a result?
===================================================
MDN staff could import pages with "(without prefix)" support text without adjusting the MDN page.

Is there anything else we should know?
======================================
An example of a page with this issue:
https://browsercompat.herokuapp.com/importer/5

Comment 1

[John Whitlock [:jwhitlock]]

Also "(unprefixed)" on https://browsercompat.herokuapp.com/importer/12

Comment 2

[John Whitlock [:jwhitlock]]

I'm going to handle these instances as support=Yes:
(unprefixed)
(no prefix)
without prefix
(without prefix)

This is based on the analysis of current "inline-text" issues, using data/gather_import_issues.py and this script:

https://gist.github.com/jwhitlock/43fa8de00d683a6fbc10

For the curious, here are the top 20 "inline-text" issues, with counts and sample URLs:

137: '(unprefixed)' https://browsercompat.herokuapp.com/importer/18
134: ', partial' https://browsercompat.herokuapp.com/importer/193
119: 'behind pref' https://browsercompat.herokuapp.com/importer/1546
92: 'as' https://browsercompat.herokuapp.com/importer/66
74: 'disabled by default' https://browsercompat.herokuapp.com/importer/2272
67: 'iOS 8' https://browsercompat.herokuapp.com/importer/386
62: '(24 only)' https://browsercompat.herokuapp.com/importer/1039
61: ')' https://browsercompat.herokuapp.com/importer/38
42: 'Reintroduced in 16.0' https://browsercompat.herokuapp.com/importer/52
38: 'yes' https://browsercompat.herokuapp.com/importer/680
38: '(' https://browsercompat.herokuapp.com/importer/38
37: 'Experimental support' https://browsercompat.herokuapp.com/importer/2272
37: '<code>webkitURL</code>' https://browsercompat.herokuapp.com/importer/515
30: 'Yes' https://browsercompat.herokuapp.com/importer/545
29: '(non standard)' https://browsercompat.herokuapp.com/importer/96
21: '\xe2\x80\x94' https://browsercompat.herokuapp.com/importer/859
18: 'prefixed as' https://browsercompat.herokuapp.com/importer/3548
17: '.' https://browsercompat.herokuapp.com/importer/192
17: '#' https://browsercompat.herokuapp.com/importer/65
16: 'with' https://browsercompat.herokuapp.com/importer/3548

The ones that include the string "prefix":

137: '(unprefixed)' https://browsercompat.herokuapp.com/importer/18
18: 'prefixed as' https://browsercompat.herokuapp.com/importer/3548
10: '(prefixed as' https://browsercompat.herokuapp.com/importer/1782
4: '(no prefix)' https://browsercompat.herokuapp.com/importer/313
2: 'without prefix' https://browsercompat.herokuapp.com/importer/831
2: 'unprefixed in nightlies' https://browsercompat.herokuapp.com/importer/539
2: '(without prefix)' https://browsercompat.herokuapp.com/importer/1460
1: 'prefixed with' https://browsercompat.herokuapp.com/importer/1107
1: 'Presto 2.12 will remove the prefix.' https://browsercompat.herokuapp.com/importer/813
1: '(possibly earlier; prefixed as' https://browsercompat.herokuapp.com/importer/1782

Comment 3

[MDN Team (:mdn-dev)]

Commits pushed to master at https://github.com/mozilla/web-platform-compat

https://github.com/mozilla/web-platform-compat/commit/1afd13443c0c458b4fcd749879b08bb96c60b897
bug 1138455 - Support variants of (unprefixed)

When parsing a compatability text segment, treat these as support=yes:

(unprefixed)
(no prefix)
without prefix
(without prefix)

https://github.com/mozilla/web-platform-compat/commit/a6ae08f39e790a383d2acc191e711d1bfa23e8aa
fix bug 1138455 - Support variants of (partial)

When parsing a compatability text segment, treat these as
support=partial:

, partial
(partial)

https://github.com/mozilla/web-platform-compat/commit/2e7b99a32fad4359af138bfa8557be3d4082ce5b
Merge pull request #38 from jwhitlock/1138455_more_inline_text

Fix bug 1138455 - Support 'no prefix' and 'partial' inline text