Opened 8 years ago

Closed 8 years ago

#4637 closed Bug (fixed)

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

Reported by: pomu0325 Owned by: Alfonso Martínez de Lizarrondo
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 8 years ago.
Add 'setup' handler for wordWrap.
4637.2.patch (551 bytes) - added by Alfonso Martínez de Lizarrondo 8 years ago.
Proposed patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 8 years ago by pomu0325

Summary: Table cells' 'nowrap' attribute is not loaded to cell property dialog on non-English languagesTable 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 8 years ago by pomu0325

Attachment: 4637.patch added

Add 'setup' handler for wordWrap.

comment:2 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: HasPatch added
Milestone: CKEditor 3.2

Changed 8 years ago by Alfonso Martínez de Lizarrondo

Attachment: 4637.2.patch added

Proposed patch

comment:3 Changed 8 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review? added; HasPatch removed
Owner: set to Alfonso Martínez de Lizarrondo
Status: newassigned

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

comment:4 Changed 8 years ago by Damian

Keywords: IBM added

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

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

comment:6 Changed 8 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: assignedclosed

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 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy