Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Disable image and image2 plugin when Easy Image is present #1791

Closed
Comandeer opened this issue Mar 13, 2018 · 1 comment
Closed

Disable image and image2 plugin when Easy Image is present #1791

Comandeer opened this issue Mar 13, 2018 · 1 comment
Assignees
@jacekbogdanski
Labels
plugin:easyimage The plugin which probably causes the issue. plugin:image The plugin which probably causes the issue. plugin:image2 The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:task Any other issue (refactoring, typo fix, etc).
Milestone
4.10.1

Comments

@Comandeer
Copy link
Member

Are you reporting a feature request or a bug?

Task

Description

Currently it's possible to enable Easy Image alongside image and image2 plugins, resulting in strange editor's behaviour. Image plugin has code to prevent loading it when image2 is present. Adding similar code to both image and image2 plugin to prevent loading when EI is present seems like a sensible solution.
@Comandeer Comandeer added type:task Any other issue (refactoring, typo fix, etc). status:confirmed An issue confirmed by the development team. plugin:image The plugin which probably causes the issue. plugin:image2 The plugin which probably causes the issue. plugin:easyimage The plugin which probably causes the issue. labels Mar 13, 2018
@mlewand mlewand added this to the Backlog milestone Mar 13, 2018
@mlewand
Copy link
Contributor

mlewand commented Mar 13, 2018

Note that it should provide some kind of feedback for the developer. What I'm thinking is to include a log msg.

@jacekbogdanski jacekbogdanski self-assigned this Jul 19, 2018
@jacekbogdanski jacekbogdanski mentioned this issue Jul 20, 2018
Merged
2 tasks
@mlewand mlewand added the target:minor Any docs related issue that can be merged into a master or major branch. label Jul 20, 2018
@jacekbogdanski jacekbogdanski mentioned this issue Jul 20, 2018
Closed
@mlewand mlewand modified the milestones: Backlog, 4.10.1 Aug 9, 2018
@Comandeer Comandeer mentioned this issue Dec 31, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jacekbogdanski jacekbogdanski

Labels
plugin:easyimage The plugin which probably causes the issue. plugin:image The plugin which probably causes the issue. plugin:image2 The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:task Any other issue (refactoring, typo fix, etc).
Projects
None yet
Milestone
4.10.1
Development

No branches or pull requests
3 participants
@Comandeer @mlewand @jacekbogdanski