Ticket #4358 (closed New Feature: fixed)
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
Change History
comment:2 Changed 4 years ago by kurgbe
- Version set to 3.0
- Component changed from General to Core : Lists
comment:5 Changed 4 years 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
comment:7 Changed 3 years 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)
comment:8 Changed 3 years 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.
comment:11 Changed 3 years 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?
comment:13 Changed 3 years 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.
comment:14 Changed 3 years 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.
comment:15 Changed 3 years ago by kurgbe
Thanks for the patch. Suggestion: instead of <not set> or <Inherit> we could use <Default>
comment:17 Changed 3 years ago by m.nguyen
I using '<not set>' for user doesn't select type of list.
comment:18 Changed 3 years 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'.
comment:20 Changed 3 years 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' )
|| '' );
comment:22 Changed 3 years 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.
comment:24 Changed 3 years ago by garry.yao
- Keywords Review+ added; Review? removed
Be sure the following thing will happenen before committing:
- Register plugin into core config in alphabetic order;
- Run lang tool to update all other language files;
comment:25 Changed 3 years ago by m.nguyen
- Status changed from assigned to closed
- Resolution set to fixed
Fixed with [5409].


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?