Ticket #4165 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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

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

Change History

comment:1 Changed 5 years ago by tobiasz.cudnik

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

comment:2 Changed 5 years ago by tobiasz.cudnik

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

Changed 5 years ago by tobiasz.cudnik

comment:3 Changed 5 years ago by tobiasz.cudnik

  • Keywords Review? added

comment:4 Changed 5 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 5 years ago by tobiasz.cudnik

Changed 5 years ago by tobiasz.cudnik

comment:5 Changed 5 years ago by tobiasz.cudnik

  • Keywords Review? added; Review- removed

comment:6 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:7 Changed 5 years ago by tobiasz.cudnik

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

Fixed with [4039].

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