Opened 12 years ago
Closed 12 years ago
#11309 closed New Feature (fixed)
Xmas card plugin.
| Reported by: | Piotr Jasiun | Owned by: | Piotr Jasiun |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | |
| Keywords: | Cc: |
Description
Create Xmas card plugin.
Change History (8)
comment:1 Changed 12 years ago by
| Owner: | set to Piotr Jasiun |
|---|---|
| Status: | new → review |
comment:3 Changed 12 years ago by
| Status: | review → review_failed |
|---|
comment:5 follow-up: 6 Changed 12 years ago by
| Status: | review → review_failed |
|---|
I pushed several commits to dev branch, fixing minor things here and there.
Additionally:
- Why not
CKEDITOR.templateforcoverHtml,styleandcardHtml? We don't need to re-define lots of static strings every time the card is displayed. - I significantly reduced the size of the fonts with http://www.fontsquirrel.com/tools/webfont-generator, using "Basic Subsetting" only (other tweaks off). If we don't want to support additional languages (the list is pretty long), we can use this trick. Otherwise, just ignore it ;)
- Unused file
GreatVibes-Regular.otf - A11y: there's no way to close the dialog. ESC key maybe?
- Is "SIL OPEN FONT LICENSE Version 1.1 - 1 February 2007" compatible with our licenses?
- [At least in latest Chrome]
- Open the card.
- Click the link at the bottom.
- The new browser tab is opened.
- Close the tab, so you go back to the page with CKEditor.
- The card disappears when returning to the page (some focusmanager issue?).
comment:6 Changed 12 years ago by
| Status: | review_failed → review |
|---|
Replying to a.nowodzinski:
I pushed several commits to dev branch, fixing minor things here and there.
Additionally:
- Why not
CKEDITOR.templateforcoverHtml,styleandcardHtml? We don't need to re-define lots of static strings every time the card is displayed.
I do not belive that cart will be displayed multiple times.
- I significantly reduced the size of the fonts with http://www.fontsquirrel.com/tools/webfont-generator, using "Basic Subsetting" only (other tweaks off). If we don't want to support additional languages (the list is pretty long), we can use this trick. Otherwise, just ignore it ;)
We want to support multiple languages.
- Unused file
GreatVibes-Regular.otf
This is original font file and I'm not sure if I can delete it because of licence.
- A11y: there's no way to close the dialog. ESC key maybe?
Fixed.
- Is "SIL OPEN FONT LICENSE Version 1.1 - 1 February 2007" compatible with our licenses?
Seba checked it.
- [At least in latest Chrome]
- Open the card.
- Click the link at the bottom.
- The new browser tab is opened.
- Close the tab, so you go back to the page with CKEditor.
- The card disappears when returning to the page (some focusmanager issue?).
Fixed.
Additionally I changed slightly CSS and now card looks more like card.
Changes in t/11309.
comment:7 Changed 12 years ago by
| Status: | review → review_passed |
|---|
Last but not least:
- Use
CKEDITOR.tools.cssVendorPrefixfor vendor prefixes. - Extra white space in L120 :D
- -5deg instead of -10deg and it will be perfect ;)
comment:8 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Putted in sepatare github repo: https://github.com/ckeditor/ckeditor-plugin-xmas

May someone make a review before publishing this plugin? Changes in t/11309.