Opened 11 years ago
Closed 11 years ago
#9479 closed Bug (fixed)
Custom class name is not any more applied to button and menu item
Reported by: | Frederico Caldeira Knabben | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.0 |
Component: | UI : Toolbar | Version: | 4.0 |
Keywords: | IBM | Cc: | Teresa Monahan |
Description
This one has been reported by a customer. It is a backwards compatibility issue that needs to be investigated:
It is no longer possible to specify a classname when defining a button or menu item. We rely on this for adding relevant class names to toolbar menu buttons and their menu items.
In v3, the render function for CKEDITOR.ui.button and CKEDITOR.menuItem performed a check for this.className and appended its value to the classes for that ui element if present. The new implementation of render for these ui elements does not check if a className has been specified in the definition though so our classes are never applied.
Change History (4)
comment:1 Changed 11 years ago by
Component: | General → UI : Toolbar |
---|---|
Status: | new → confirmed |
Version: | → 4.0 (GitHub - master) |
comment:2 Changed 11 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
comment:3 Changed 11 years ago by
Status: | review → review_passed |
---|
Please remove unnecessary spaces before ":" in obj literals before masterising.
comment:4 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
The reason that we removed the custom class application is because we have now a named UI item class name convention established in v4, to exemplify, with the bold button we now have the following classes:
So individual buttons can be actually styled by following over the 2nd class name in the above list.
But in order to maintain compatibility, we'd still back port the v3 feature here. Review opened on t/9479 on both dev and test.