Opened 6 years ago

Closed 6 years ago

#4165 closed Bug (fixed)

rtrim and ltrim fails in various browsers

Reported by: tobiasz.cudnik Owned by: tobiasz.cudnik
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

From CKEDITOR.tools, rtrim and ltrim fails in various browsers (FF3.5, Safari4, FF2, FF3, Safari3). Full trim in done instead of trimming only the desired side.

Attachments (3)

4165.patch (643 bytes) - added by tobiasz.cudnik 6 years ago.
4165_2.patch (723 bytes) - added by tobiasz.cudnik 6 years ago.
4165_3.patch (1.7 KB) - added by tobiasz.cudnik 6 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 6 years ago by tobiasz.cudnik

  • Owner set to tobiasz.cudnik
  • Status changed from new to assigned

comment:2 Changed 6 years ago by tobiasz.cudnik

Actually this issue is related to CKEDITOR.dom.node, not tools namespace.

Changed 6 years ago by tobiasz.cudnik

comment:3 Changed 6 years ago by tobiasz.cudnik

  • Keywords Review? added

comment:4 Changed 6 years ago by fredck

  • Keywords Review- added; Review? removed

I've talked with Garr yabout something similar today, and it's better to have the test changed so it doesn't need to have spaces at the end of the lines, as they will be definitely removed by our dev tools.

Changed 6 years ago by tobiasz.cudnik

Changed 6 years ago by tobiasz.cudnik

comment:5 Changed 6 years ago by tobiasz.cudnik

  • Keywords Review? added; Review- removed

comment:6 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:7 Changed 6 years ago by tobiasz.cudnik

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

Fixed with [4039].

Note: See TracTickets for help on using tickets.
© 2003 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy