Opened 9 years ago
Closed 9 years ago
#16644 closed Task (fixed)
Make it easier for users to learn on how to style widgets
| Reported by: | Wiktor Walc | Owned by: | Tade0 |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.6.0 |
| Component: | Documentation & Samples | Version: | |
| Keywords: | Cc: |
Description
I just spend a good 5 minutes (knowing where to search more or less) to find how am I supposed to define styles for widgets.
Suggestion:
- To the default styles.js add at least two sample widget styles, for embed media and image 2.
- To the default styles.js on top of it add a link to http://docs.ckeditor.com/#!/guide/dev_styles-section-style-rules
Change History (6)
comment:1 Changed 9 years ago by
| Milestone: | → CKEditor 4.6.0 |
|---|---|
| Status: | new → confirmed |
comment:2 Changed 9 years ago by
| Owner: | set to Tade0 |
|---|---|
| Status: | confirmed → review |
comment:3 Changed 9 years ago by
Overall it looks good.
I noticed one issue - when you have widget focused and apply/remove widget style the combo value (in closed state) does not update. It updates only after refocusing widget. However, this is not caused by the changes made for this task, but rather by the way stylescombo detects style changes and updates itself. Separate issue for this should be created.
I will ask @wwalc to validate the styles (from the user perspective as he requested it). If it is ok, I will merge it to the major.
comment:6 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Merged to major with fbe67fa.

Added some new styles. Changes pushed to branch:t/16644.