Ticket #4358 (closed New Feature: fixed)

Opened 12 months ago

Last modified 4 months ago

List properties dialog box is missing

Reported by: wwalc Owned by: m.nguyen
Priority: Normal Milestone: CKEditor 3.3
Component: Core : Lists Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc: michel.denys@…

Description

The properties for bulleted/numbered lists are missing (this feature is available in FCKeditor: when you click on a list, a "Bulleted List Properties" / "Numbered List Properties" item is available in the context menu).

The issue was reported here:  http://cksource.com/forums/viewtopic.php?f=6&t=14342

Attachments

fck-list-properties.png Download (3.4 KB) - added by kurgbe 10 months ago.
liststyle_ref.zip Download (1.9 KB) - added by m.nguyen 5 months ago.
4358.patch Download (7.6 KB) - added by m.nguyen 5 months ago.
4358_2.patch Download (8.4 KB) - added by m.nguyen 5 months ago.
4358_3.patch Download (9.3 KB) - added by m.nguyen 5 months ago.
4358_4.patch Download (9.4 KB) - added by m.nguyen 4 months ago.
4358_5.patch Download (9.2 KB) - added by m.nguyen 4 months ago.
Thank Garry, I have got your point.
4358_7.patch Download (9.2 KB) - added by m.nguyen 4 months ago.

Change History

Changed 10 months ago by kurgbe

You can view the missing bullet properties @ http://ckeditor.com/demo

When do you think this could be fixed? Is there a workaround? I need the speed of CKeditor but without the list properties I have to stay with FCKeditor?

Changed 10 months ago by kurgbe

  • version set to 3.0
  • component changed from General to Core : Lists

Changed 10 months ago by kurgbe

Changed 10 months ago by kurgbe

Print-screen of FCKeditor function:

Changed 10 months ago by kurgbe

  • cc michel.denys@… added

Changed 10 months ago by garry.yao

  • keywords Confirmed removed
  • version changed from 3.0 to SVN (CKEditor)
  • milestone changed from CKEditor 3.1 to CKEditor 3.2

Changed 8 months ago by fredck

  • milestone changed from CKEditor 3.2 to CKEditor 3.3

Changed 6 months ago by kurgbe

Possible workaround:

In the Style combo plugin (plugins\stylescombo\default.js), add an attribute for the ol element (or the ul element). For example, for roman lists:

CKEDITOR.addStylesSet('default',[
{name :'Roman',element:'ol',attributes:{type:'I'}},
]);

It works BUT to apply the style, the ol element must be selected. The only method I saw is to select it via the name of the element shown at the bottom of the editor. Not very user friendly for casual users. (see the CKEDITOR FAQ  http://cksource.com/forums/viewtopic.php?f=11&t=17301)

Changed 6 months ago by garry.yao

  • keywords Confirmed added

Thanks kurgbe, we should be able to align this feature with v2 in this milestone, the styles combo is not sufficient to carry out this feature, we still need a classical dialog here.

Changed 5 months ago by m.nguyen

  • owner set to m.nguyen
  • status changed from new to assigned

Changed 5 months ago by m.nguyen

Changed 5 months ago by m.nguyen

Changed 5 months ago by m.nguyen

  • keywords Review? added

Changed 5 months ago by garry.yao

  • keywords Review- added; Review? removed

Alternate bulleted list type in IE doesn't work (it should be circle instead of "Circle"), besides, in V3 we should deprecate the "type" attribute in favor of  list-style-type CSS property.
Also please use "<not set>" for unknown list type instead of leaving it empty.
Besides, Can we make the Bulleted List Type field a bit wider, make it looks better?

Changed 5 months ago by m.nguyen

Changed 5 months ago by m.nguyen

  • keywords Review? added; Review- removed

Changed 5 months ago by garry.yao

  • keywords Review- added; Review? removed

Please use "<not set>" instead of 'inherit' as display option, which is more user-friendly, it's enough to just delete the style when "<not set>" is selected. Also this option is missing from the Bulleted List type.

Changed 5 months ago by garry.yao

Also note that while adapting to the CSS style, we should still provide compatibility to the "type" attribute which make comes from content produced by v2.

Changed 5 months ago by kurgbe

Thanks for the patch. Suggestion: instead of <not set> or <Inherit> we could use <Default>

Changed 5 months ago by m.nguyen

Changed 5 months ago by m.nguyen

  • keywords Review? added; Review- removed

Changed 5 months ago by m.nguyen

I using '<not set>' for user doesn't select type of list.

Changed 5 months ago by garry.yao

  • keywords Review- added; Review? removed

Finally, we need a mapping between attribute  type and  list-style-type.
Also, it's enough to use empty string for 'notset'.

Changed 4 months ago by m.nguyen

Changed 4 months ago by m.nguyen

  • keywords Review? added; Review- removed

Changed 4 months ago by garry.yao

  • keywords Review- added; Review? removed

That's great, but I think you misunderstood my points that the mapping should be applied to the 'setup' of list type instead of on 'commit':

this.setValue( element.getStyle( 'list-style-type' ) 
        || mapListStyle[ element.getAttribute( 'type' ) ] 
        || element.getAttribute( 'type' ) 
        || '' );

Changed 4 months ago by m.nguyen

Thank Garry, I have got your point.

Changed 4 months ago by m.nguyen

  • keywords Review? added; Review- removed

Changed 4 months ago by garry.yao

  • keywords Review- added; Review? removed

There's a problem with the current approach(also v2), with the following list, it's not able to recognize the list type properly.

<ul type="i">
	<li>item</li>
</ul>

Also, I can't get the 'start' field working.

Changed 4 months ago by m.nguyen

Changed 4 months ago by m.nguyen

  • keywords Review? added; Review- removed

Changed 4 months ago by garry.yao

  • keywords Review+ added; Review? removed

Be sure the following thing will happenen before committing: 1. Register plugin into core config in alphabetic order; 1. Run lang tool to update all other language files;

Changed 4 months ago by m.nguyen

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [5409].

Note: See TracTickets for help on using tickets.