Closed
Bug 1340159
Opened 8 years ago
Closed 6 years ago
Rename <br-style> to <line-style> in border-style page
Categories
(Developer Documentation Graveyard :: CSS, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sebo, Assigned: wbamberg)
References
()
Details
https://developer.mozilla.org/de/docs/Web/CSS/border-style defines the <br-style> data type, which was meanwhile renamed to <line-style> in the spec.[1], so it should be updated in the article and the syntax definition.
Sebastian
[1] https://drafts.csswg.org/css-backgrounds-3/#border-style
Comment 1•7 years ago
|
||
Sebastian, where can I change syntax definitions? I couldn't find it
Assignee: nobody → albert
Comment 2•7 years ago
|
||
Please ignore above line. Trying to get help atm via IRC or Discourse. Forgot to delete it when I assigned the task to myself.
The {{csssyntax}} macro actually lives here: https://github.com/mozilla/kumascript/blob/master/macros/csssyntax.ejs but as you can see in the code, it consumes another content which is located here: https://github.com/mdn/data/blob/master/css/properties.json
Albert, I hope this helps, please do tell if you need any other info :)
Comment 4•7 years ago
|
||
Thanks Julien!
Sent a pull request: https://github.com/mdn/data/pull/102
I additionally renamed
- <br-style> to <line-style> in all other border-* properties
- <br-width> to <line-width> in all border-* properties
- <br-style> to <border-style> in outline-style
- <br-width> to <border-width> in outline-width
Comment 5•7 years ago
|
||
currently in limbo - https://github.com/mdn/data/pull/104
Comment 6•7 years ago
|
||
There are updates required I don't understand. Unassign myself.
Assignee: albert → nobody
Updated•6 years ago
|
Priority: P2 → P1
Updated•6 years ago
|
Assignee: nobody → wbamberg
Assignee | ||
Comment 7•6 years ago
|
||
https://github.com/mdn/data/pull/104 is merged, and I've updated the Wiki pages to rename these values. I think that's everything here, so marking this one fixed.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•