WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#36604 closed defect (bug) (fixed)

Emoji skin tone support test incorrectly passing in Chrome

Reported by: pento Owned by: pento
Priority: normal Milestone: 4.5.1
Component: Emoji Version: 4.5
Severity: normal Keywords: fixed-major
Cc: Focuses: javascript

Description

Chrome 50 fixed a bunch of their emoji rendering bugs, but emoji skin tone wasn't one of them.

Unfortunately, our skin tone test is incorrectly passing, we should figure out why.

Change History (4)

#1 @pento
10 years ago

  • Milestone 4.6 → 4.5.1
  • Owner set to pento
  • Status new → assigned
  • Version → 4.5

#2 @pento
10 years ago

  • Resolution → fixed
  • Status assigned → closed

In 37256:

Emoji: The diversity support test was incorrectly passing on all browsers.

[37028] missed some logic to capture the base emoji skin tone, to compare to the modified emoji skin tone. This caused all browsers to report that they supported skin tone modifiers, regardless of whether they actually did.

Fixes #36604 for trunk.

#3 @pento
10 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closed → reopened

#4 @pento
10 years ago

  • Resolution → fixed
  • Status reopened → closed

In 37257:

Emoji: The diversity support test was incorrectly passing on all browsers.

[37028] missed some logic to capture the base emoji skin tone, to compare to the modified emoji skin tone. This caused all browsers to report that they supported skin tone modifiers, regardless of whether they actually did.

Merge of [37256] to the 4.5 branch.

Fixes #36604.

Note: See TracTickets for help on using tickets.