WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#36266 closed defect (bug) (fixed)

Emoji Diversity support test fails in Safari

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

Description

Safari supports the entire Unicode 8.0 set, but it's failing our test for emoji diversity support, which suggests our test is wrong.

Change History (5)

#1 @kirasong
10 years ago

  • Owner set to pento
  • Status new → assigned

This ticket was mentioned in ​Slack in #core by mike. ​View the logs.
10 years ago

#3 @pento
10 years ago

  • Resolution → fixed
  • Status assigned → closed

In 37028:

Emoji: Fix the diversity emoji check in Safari.

When the browser test for diversity emoji was added in [36160], it included a workaround for Chrome not being able to compare Uint8ClampedArray objects directly, by converting them to a string. Unfortunately, Safari doesn't support the Uint8ClampedArray.toString() method correctly, so the test was incorrectly failing in Safari.

Fixes #36266 for trunk.

#4 @pento
10 years ago

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

#5 @pento
10 years ago

  • Resolution → fixed
  • Status reopened → closed

In 37090:

Emoji: Fix the diversity emoji check in Safari.

When the browser test for diversity emoji was added in [36160], it included a workaround for Chrome not being able to compare Uint8ClampedArray objects directly, by converting them to a string. Unfortunately, Safari doesn't support the Uint8ClampedArray.toString() method correctly, so the test was incorrectly failing in Safari.

Merge of [37028] to the 4.4 branch.

Fixes #36266.

Note: See TracTickets for help on using tickets.