Opened 6 years ago

Closed 6 years ago

#4637 closed Bug (fixed)

Table cells' 'nowrap' attribute is not loaded to cell property dialog

Reported by: pomu0325 Owned by: alfonsoml
Priority: Normal Milestone: CKEditor 3.2
Component: UI : Dialogs Version:
Keywords: IBM Review+ Cc: pomu@…

Description

I tried with Japanese and also with French and Spanish.

to reproduce

  • Open CKEditor demo
  • Click 'Multi-language interface'
  • Select language other than English
  • Right-click on a table cell, select 'Cell Properties' from context menu.
  • Set 'Word Wrap' to 'No'
  • Click 'OK' to close the dialog
  • Open 'Cell Properties' dialog again, 'Word Wrap' is showing 'Yes'.

Attachments (2)

4637.patch (551 bytes) - added by pomu0325 6 years ago.
Add 'setup' handler for wordWrap.
4637.2.patch (551 bytes) - added by alfonsoml 6 years ago.
Proposed patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 6 years ago by pomu0325

  • Summary changed from Table cells' 'nowrap' attribute is not loaded to cell property dialog on non-English languages to Table cells' 'nowrap' attribute is not loaded to cell property dialog

My mistake, it happens whichever language I selected. It simply lacks 'setup' handler for the wordWrap.

Changed 6 years ago by pomu0325

Add 'setup' handler for wordWrap.

comment:2 Changed 6 years ago by fredck

  • Keywords HasPatch added
  • Milestone set to CKEditor 3.2

Changed 6 years ago by alfonsoml

Proposed patch

comment:3 Changed 6 years ago by alfonsoml

  • Keywords Review? added; HasPatch removed
  • Owner set to alfonsoml
  • Status changed from new to assigned

The patch is just the code from pomu0325 with full path and changelog.

comment:4 Changed 6 years ago by damo

  • Keywords IBM added

comment:5 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed
  • Please provide patches for the trunk root.
  • Be sure to have a changelog entry for this.

comment:6 Changed 6 years ago by alfonsoml

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

Ops, sorry.

Seems that I reuploaded the patch from pomu0325 instead of the one with full path.

Fixed with [4916]

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