Make WordPress Core

Opened 7 years ago

Last modified 3 weeks ago

#45991 new defect (bug)

Twenty Nineteen: Post headings are overflowing the content area

Reported by: dejan85's profile Dejan85 Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 5.0.3
Component: Bundled Theme Keywords: has-patch
Focuses: css Cc:

Description

Post titles are overflowing the content area.
In Twenty Seventeen the following code in style.css prevents titles from overflowing the content area. This code should be added to Twenty Nineteen as well.

#page {
    word-wrap: break-word;
}

Attachments (3)

45991.patch (236.6 KB) - added by Shital Patel 7 years ago.
word-wrap: break-word; added in style.css file
45991.diff (1.1 KB) - added by mukesh27 7 years ago.
Updated patch. Added missing css in style.scss and style-rtl.css
long-post-titles.jpg (338.2 KB) - added by poena 6 years ago.
Before patch, in Chrome on Windows

Download all attachments as: .zip

Change History (12)

@Shital Patel
7 years ago

word-wrap: break-word; added in style.css file

#1 @Shital Patel
7 years ago

  • Focuses ui added
  • Keywords has-patch added

@mukesh27
7 years ago

Updated patch. Added missing css in style.scss and style-rtl.css

#2 @laurelfulford
7 years ago

  • Keywords needs-testing added
  • Milestone changed from Awaiting Review to Future Release

@poena
6 years ago

Before patch, in Chrome on Windows

#3 @poena
6 years ago

Entry title already has

word-wrap: break-word;

So the existing patches does not make a difference.

@Dejan85 did you mean that the title should only be as wide as the post content?

#4 @poena
2 years ago

  • Keywords reporter-feedback added

#5 @karmatosed
20 months ago

  • Keywords needs-patch added; has-patch removed

This patch seems to not fix the problem reported so I am going to put this back to needing a patch so we can get one.

#6 @karmatosed
18 months ago

  • Keywords needs-testing reporter-feedback removed

#7 @sabernhardt
7 weeks ago

  • Keywords has-patch close 2nd-opinion added; needs-patch removed

[45258] added word-wrap for .entry-title and other selectors after this ticket was opened, and that fixed the post titles.

However, any heading within post content could still overflow the container. This ticket could follow that direction instead.

#8 @karmatosed
3 weeks ago

@sabernhardt are you recommending (which I think you are) that this gets closed and perhaps another is opened or we just accept the content overflow?

#9 @sabernhardt
3 weeks ago

  • Focuses css added; ui removed
  • Keywords close 2nd-opinion removed
  • Summary changed from Twenty Nineteen: Post titles are overflowing the content area to Twenty Nineteen: Post headings are overflowing the content area

Recommending to close probably was not the best idea, even though the post titles are fixed. If any change like the current patch is made, I would like to keep this ticket.

The summary is updated to reflect the issue that still occurs with Twenty Nineteen, which @poena reported in the issue tracker.

Note: See TracTickets for help on using tickets.