Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#32446 closed defect (bug) (worksforme)

Bundled Themes: `-moz-hyphens: auto` no longer needed

Reported by: kraftbj's profile kraftbj Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:


Previous to FF v35, -moz-hyphens: auto was needed to work around a FF bug that caused long words to overflow. For v35+, it adds an excessive amount of hyphens.

Example: (Twenty Thirteen)

What's the word on retracting work-arounds when no longer needed?

Attachments (1)

32446.patch (4.1 KB) - added by kraftbj 8 years ago.
Remove all -moz-hyphens:auto;

Download all attachments as: .zip

Change History (5)

8 years ago

Remove all -moz-hyphens:auto;

#1 @kraftbj
8 years ago

  • Keywords needs-testing added

Found in all of the Twenties. Need some eyes to confirm removing it doesn't create a problem.

#2 @lancewillett
8 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 4.3
  • Summary changed from Multiple Twenties: -moz-hyphens: auto no longer needed to Bundled Themes: `-moz-hyphens: auto` no longer needed

I think removing workarounds is a good thing for housekeeping provided that they don't cause problems in older versions. For modern browsers like Firefox it's nice to support 2 major versions back. Since Firefox is on v38 now it should be safe to remove the support and clean up the stylesheets a bit.

#3 @obenland
8 years ago

Does that not remove hyphenation entirely in FF? Testing with 2010 and 2013, I don't get them at all anymore.

#4 @lancewillett
8 years ago

  • Keywords needs-testing removed
  • Milestone 4.3 deleted
  • Resolution set to worksforme
  • Status changed from new to closed

I think we should leave things as-is for now. @kraftbj I thought you meant at first that the prefix for Gecko browsers wasn't needed any more — but it is still needed per

Note: See TracTickets for help on using tickets.