Opened 11 years ago

Closed 7 years ago

#1890 closed New Feature (duplicate)

Table Styles: Border, Background color

Reported by: Stijn René Cumps Owned by:
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.5.1
Keywords: HasPatch Cc: pomu@…

Description

Hi guys,

I did some recoding of fck_table.html

For the user interface I copied the border color and background color from fck_tablecell.html, so the fcklang will still carry 'cell' instead of 'table'.

If you like it, i can clean up the code and take that part on me.

I have added the border-style myself.

So to have a translation for it, add the following lines to the language javascript files. (I have only English, Spanish, Dutch)

en.js: DlgTableBorderStyle : "Border Style", es.js: DlgTableBorderStyle : "Stilo de Borde", nl.js: DlgTableBorderStyle : "Randstijl",

the file fck_table.html will be attached.

Note, to have a better view on the dialog, you need to update the size of the dialog window in fckcommands.js

case 'Table' : oCommand = new FCKDialogCommand( 'Table' , FCKLang.DlgTableTitle , 'dialog/fck_table.html' , 480, 300 ) ; break ;

I hope you like it.

Attachments (1)

fck_table.html (13.1 KB) - added by Stijn René Cumps 11 years ago.
Modified fck_table.html

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by Stijn René Cumps

Attachment: fck_table.html added

Modified fck_table.html

comment:1 Changed 11 years ago by Frederico Caldeira Knabben

Component: UI : DialogsGeneral
Keywords: Confirmed HasPatch added; Table style border background color removed
Milestone: FCKeditor 2.7

Targeting to the 2.7 so we can review and possibly consider it.

comment:2 in reply to:  1 Changed 11 years ago by Stijn René Cumps

Replying to fredck:

Targeting to the 2.7 so we can review and possibly consider it.

I've noticed a small problem when you don't select a border.

Change the code lines from line 128 till 143 to:

table.style.backgroundColor = GetE('txtBackColor').value ; table.style.borderWidth = GetE('txtBorder').value + "px" ; table.style.borderColor = GetE('txtBorderColor').value ; table.style.borderStyle = GetE('selBorderStyle').value ;

comment:3 Changed 9 years ago by pomu0325

Cc: pomu@… added

comment:4 Changed 7 years ago by Alfonso Martínez de Lizarrondo

Resolution: duplicate
Status: confirmedclosed

dup of #595

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