Opened 8 years ago
Closed 8 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 8 years ago by
Milestone: | → CKEditor 4.6.0 |
---|---|
Status: | new → confirmed |
comment:2 Changed 8 years ago by
Owner: | set to Tade0 |
---|---|
Status: | confirmed → review |
comment:3 Changed 8 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 8 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Merged to major
with fbe67fa.
Added some new styles. Changes pushed to branch:t/16644.