#14763 closed Task (fixed)
AC and Balloonpanel plugins moono-lisa skin adjustments.
Reported by: | kkrzton | Owned by: | kkrzton |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.6.0 |
Component: | UI : Skins | Version: | |
Keywords: | Cc: |
Description
Related to t/14569.
The balloonpanel and a11ychecker plugins have separate stylesheets which should be adjusted to the new moono-lisa skin.
Change History (10)
comment:1 Changed 9 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 9 years ago by
comment:3 Changed 9 years ago by
Owner: | set to kkrzton |
---|---|
Status: | confirmed → assigned |
comment:4 Changed 9 years ago by
Status: | assigned → pending |
---|
The AC skin adjustments are currently reviewed by the CKSource team (which probably means some polishing will be needed).
However, I already created the branches with adjustments: balloonpanel and a11ychecker (and also AC overlay PoC branch).
comment:5 Changed 8 years ago by
Status: | pending → review |
---|
Two branches were created for this task:
- One in
ckeditor-dev
repo with changes inballoonpanel
(while it became a core plugin in the latest major) - t/14763. - One in
ckeditor-plugin-a11ychecker
repo with changes in AC styles - t/14763.
The branch in ckeditor-dev
adds balloonpanel
CSS files for moono-lisa
skin. One important thing to notice - it is based on t/14569. Without changes from t/14569
the balloonpanel
could not be properly tested (visually).
The branch in ckeditor-plugin-a11ychecker
, apart from new skin files, introduces some changes in less
workflow. Previously it was not prepared to handle (build and use) more than one skin. These changes were introduced in 1349924 commit.
comment:6 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | review → closed |
Fixed with git:570f5aabd2 (merged to major).
I've also added a manual test for old moono skin.
comment:7 Changed 8 years ago by
@m.lewandowski the link to the merge commit does not work and in fact I don't see this merge on major branch. Are you sure you properly pushed this merge?
By the way, this ticket is also about branch t/14763 in ckeditor-plugin-a11ychecker
so before closing it, this branch should be also reviewed and merged.
comment:8 Changed 8 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
My bad, the branch wasn't pushed. As for closing ticket - all right, let's close it together with AC tickets.
comment:9 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
AC skin improvements were merged with ckeditor-plugin-a11ychecker#228 issue.
cc