Opened 10 years ago
Last modified 10 years ago
#12870 closed New Feature
Replace "alert()" with warning notification — at Version 1
| Reported by: | Piotr Jasiun | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.5.0 Beta |
| Component: | General | Version: | |
| Keywords: | Cc: |
Description (last modified by )
Since we have notifications we can replace some usage of alert() method with warning notification. I found 3 potential places where we could change it:
- https://github.com/ckeditor/ckeditor-dev/blob/major/plugins/clipboard/plugin.js#L640
- https://github.com/ckeditor/ckeditor-dev/blob/major/plugins/pastefromword/filter/default.js#L1162
- https://github.com/ckeditor/ckeditor-dev/blob/major/plugins/filebrowser/plugin.js#L342
Warning notification could be used if notification plugin is loaded and alert function otherwise.
Change History (1)
comment:1 Changed 10 years ago by
| Description: | modified (diff) |
|---|---|
| Milestone: | → CKEditor 4.5.0 |
Note: See
TracTickets for help on using
tickets.
