Opened 11 years ago
Closed 11 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 11 years ago by
Owner: | set to Piotr Jasiun |
---|---|
Status: | new → review |
comment:3 Changed 11 years ago by
Status: | review → review_failed |
---|
comment:5 follow-up: 6 Changed 11 years ago by
Status: | review → review_failed |
---|
I pushed several commits to dev branch, fixing minor things here and there.
Additionally:
- Why not
CKEDITOR.template
forcoverHtml
,style
andcardHtml
? 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 11 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.template
forcoverHtml
,style
andcardHtml
? 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 11 years ago by
Status: | review → review_passed |
---|
Last but not least:
- Use
CKEDITOR.tools.cssVendorPrefix
for vendor prefixes. - Extra white space in L120 :D
- -5deg instead of -10deg and it will be perfect ;)
comment:8 Changed 11 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.