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)

All
Other
defect
Not set
normal

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.
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
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.