Bug Tracker

Opened 4 years ago

Closed 4 years ago

#12350 closed bug (fixed)

jQuery.trim should remove byte-order-mark (BOM) characters

Reported by: dmethvin Owned by:
Priority: low Milestone: 1.8.1
Component: core Version: 1.8.0
Keywords: Cc:
Blocked by: Blocking:

Description

Per discussion on ​https://github.com/jquery/jquery/pull/896 .

Change History (6)

comment:1 Changed 4 years ago by Wesley Walser

  • Resolution set to fixed
  • Status changed from new to closed

Fix #12350. Remove BOM in jQuery.trim. Close gh-897.

Changeset: 2b5b4ebbd78ce10cbbe2a6f057feea615fc69629

comment:2 Changed 4 years ago by dmethvin

  • Component changed from unfiled to core
  • Milestone changed from None to 1.8.1
  • Priority changed from undecided to low

comment:3 Changed 4 years ago by Dave Methvin

Revert "Fix #12350. Remove BOM in jQuery.trim. Close gh-897."

This reverts commit 2b5b4ebbd78ce10cbbe2a6f057feea615fc69629.

String.prototype.trim doesn't trim BOM in Safari 5.0 so this won't work without additional feature detects.

​http://swarm.jquery.org/result/165379

Changeset: ac043b1bc8096bfbb4be7995af4f5ccc5cad7c19

comment:4 Changed 4 years ago by dmethvin

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:5 Changed 4 years ago by dmethvin

  • Status changed from reopened to open

comment:6 Changed 4 years ago by Richard Gibson

  • Resolution set to fixed
  • Status changed from open to closed

Fix #12350: jQuery.trim should remove BOM

Changeset: 9e246dd7fa010f2b8e112ec5a57491167556c55a

Note: See TracTickets for help on using tickets.