Ticket #4637 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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

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

Change History

comment:1 Changed 5 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 5 years ago by pomu0325

Add 'setup' handler for wordWrap.

comment:2 Changed 5 years ago by fredck

  • Keywords HasPatch added
  • Milestone set to CKEditor 3.2

Changed 5 years ago by alfonsoml

Proposed patch

comment:3 Changed 5 years ago by alfonsoml

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

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

comment:4 Changed 5 years ago by damo

  • Keywords IBM added

comment:5 Changed 5 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 5 years ago by alfonsoml

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

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