Closed
Bug 1157161
Opened 10 years ago
Closed 10 years ago
"Article Summary" adds wrapper div that prevents summary extraction
Categories
(developer.mozilla.org Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: fs, Unassigned)
Details
(Whiteboard: [specification][type:bug])
What did you do?
================
1. Set a nice summary block like on https://developer.mozilla.org/en-US/docs/Web/JavaScript
2. Noticed the meta desc is containing the second paragraph instead of the nicely written summary from the box
3. Cried
What happened?
==============
<div class="summary"><p></p></div> gets inserted by our CKEditor config.
What should have happened?
==========================
<p class="summary"></p> should have been inserted as this allows Kuma to extract the summary from this <p> instead of the second.
Is there anything else we should know?
======================================
This effects our SEO.
PR upcoming.
Reporter | ||
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/kuma
https://github.com/mozilla/kuma/commit/9a1ed5870772111be2eaaf819d0dedde1e2445fe
Fix bug 1157161: Add summary class to p instead of a creating a wrapping div
https://github.com/mozilla/kuma/commit/bff7e6acaa6c3e95762b5d03d4b578505f859043
Merge pull request #3199 from Elchi3/bug1157161
Fix bug 1157161: Add summary class to p instead of a creating a wrapping div
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•