[CompatData][Importer] Handle <br> tags in feature names

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jwhitlock, Assigned: jwhitlock)

Tracking

Details

(Whiteboard: [specification][type:bug], URL)

(Assignee)

Description

3 years ago
What did you do?
================
With the new importer from bug 1171957:
1. Go to https://browsercompat.herokuapp.com/importer/5235 (importer page for https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Lexical_grammar)
2. Press "Reset" button


What happened?
==============
Get warning "HTML element <br> (but not wrapped content) was removed."

What should have happened?
==========================
Converted to a space

Is there anything else we should know?
======================================
Appears 46 times in current pages
(Assignee)

Updated

3 years ago
Blocks: 1181140
Depends on: 1171957
(Assignee)

Updated

3 years ago
Assignee: nobody → jwhitlock

Comment 1

3 years ago
Commits pushed to master at https://github.com/mdn/browsercompat

https://github.com/mdn/browsercompat/commit/8728e8a272762dcddd978158e2880cd82434e404
bug 1198907 - Unify <br> into HTMLElement

Instead using HTMLSelfClosingElement (derived from HTMLOpenTag), expand
HTMLElement to handle self-closing elements.  Makes .process a little
saner and intuitive.

https://github.com/mdn/browsercompat/commit/2656b9cb531683a6d6606e481fcc760d38d391dd
fix bug 1198907 - Drop <br> in feature names

Continue dropping <br> tags from feature names, but don't add a
tag_dropped issue.

https://github.com/mdn/browsercompat/commit/5c44596ebb7b11e5494f0adadd08508944d55f71
Merge pull request #49 from mdn/ignore_br_1198907

fix bug 1198907 - ignore <br> in feature names

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.