Closed
Bug 1157161
Opened 9 years ago
Closed 9 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•9 years ago
|
||
https://github.com/mozilla/kuma/pull/3199
Comment 2•9 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•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•4 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
•