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.

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

Do we need this repo? #2328

Closed
fredck opened this issue Mar 3, 2017 · 15 comments
Closed

Do we need this repo? #2328

fredck opened this issue Mar 3, 2017 · 15 comments
Assignees
Labels
package:essentials type:task This issue reports a chore (non-production change) and other types of "todos".
Milestone

Comments

@fredck
Copy link
Contributor

fredck commented Mar 3, 2017

I don't see much purpose and benefit for presets:

  • We have builds as the current way to solve this problem.
  • If we ever add or remove something from a preset, people will get surprised that suddenly things change within their editors.
  • If it is not solving any serious issue, there is little to no benefit with it.
@Reinmar
Copy link
Member

Reinmar commented Mar 3, 2017

We have builds as the current way to solve this problem.

Builds force you to use a certain creator and a precise set of features. And they are not reusable. You can't compose many builds.

If we ever add or remove something from a preset, people will get surprised that suddenly things change within their editors.

The same is true for builds.

If it is not solving any serious issue, there is little to no benefit with it.

There's need for presets. Simplifying the work needed to create an editor with all the essentials features.

@fredck
Copy link
Contributor Author

fredck commented Mar 3, 2017

If this is just about Clipboard, Enter, Paragraph, Typing, Undo, etc.... let's then create a plugin called "Essentials" and the problem is solved. Let's just not overcomplicate things.

@Reinmar
Copy link
Member

Reinmar commented Mar 3, 2017

The problem is also solved now and it's solved better. Believe me, you're not creating editors every day. We are. I have enough of updating package.json's with missing deps and importing 10 features to have some reasonable editor for tests.

@fredck
Copy link
Contributor Author

fredck commented Mar 3, 2017

If we ever add or remove something from a preset, people will get surprised that suddenly things change within their editors.

The same is true for builds.

That's true as long as you don't create your own build. The problem is when I create my own build and use a preset there.

@fredck
Copy link
Contributor Author

fredck commented Mar 3, 2017

Believe me, you're not creating editors every day. We are.

Make it a dev tool then, just for you. Developers out there are not creating editor every day, but when they do it they want to be sure that things will be under their control.

Btw, what I've proposed in a previous comment seems reasonable.

@Reinmar
Copy link
Member

Reinmar commented Mar 3, 2017

We'll see. We don't need to promote presets initially, but I think that not sharing such a useful tool is a waste of energy.

Anyway, I plan to keep this package. Even if it's useful only for us. That's already enough.

@Reinmar Reinmar closed this as completed Mar 3, 2017
@fredck
Copy link
Contributor Author

fredck commented Aug 31, 2017

I renew my proposal to remove this repository, making the following changes:

  • Move the essentials plugin to ckeditor5-essentials.
  • Move the article plugin bundle, which seems to be used only inside tests, to an existing development-only repository, like ckeditor5. Optionally, it could have also it's own repo (like ckeditor5-dev-articlebundle, if it'll not find fit in existing repos.

This proposal is motivated by a recent talk with @wwalc, when it has been raised the confusion of the name "presets" with the current concept of presets used in CKEditor 4. This additional problem can be added to the list of issues that I've pointed in my initial post here.

@fredck fredck reopened this Aug 31, 2017
@Reinmar
Copy link
Member

Reinmar commented Aug 31, 2017

The article preset could land in ckeditor5-core/tests/_utils/ or something. I wouldn't like to add it to ckeditor5 because it's not a real package and because all other packages would need to have it in their devdeps.

I'm fine with ckeditor5-essentials if you both don't want the presets concept to exist (I agree that it may be too many concepts since we have builds and packages and stuff already). However, at some point, we'll need to address the issue with "which plugins do I need to enable to have that document editor setup?". We'll be talking here about 30+ plugins so it will be easy to miss something. That's the problem I wanted to resolve and I didn't imagine it because it existed in CKE4.

@fredck
Copy link
Contributor Author

fredck commented Aug 31, 2017

"which plugins do I need to enable to have that document editor setup?"

My hope is that specific builds for such setups will help to answer this. Let's wait for the problem to be raised to think what's the right approach.

@Reinmar Reinmar self-assigned this Sep 6, 2017
@Reinmar
Copy link
Member

Reinmar commented Sep 6, 2017

Just one thing – I'm going to move ArticlePreset to ckeditor5-core. However, it's now used in the docs – as a way allowing us to quickly create examples there without the need to import a dozen of plugins. There's no way to avoid that, so if anyone will look into the source of these examples, they will see the preset being used.

So, I'm also thinking about naming this StandardArticlePluginSet. Or something like this. To make the name as clear as possible.

@wwalc
Copy link
Member

wwalc commented Sep 6, 2017

For me it's fine to rename it to ArticlePluginSet. "Standard" prefix does not bring any additional semantic value I guess? Beside being a bit confusing due to the "Standard" preset in CKEditor 4?
👍 for changing XXXPreset into more meaningful XXXPluginSet.

@fredck
Copy link
Contributor Author

fredck commented Sep 7, 2017

I'm going to move ArticlePreset to ckeditor5-core.

:?

Right now, ckeditor5-core doesn't have any dependency on other packages. Shouldn't this be included into ckeditor5 instead, which already has all packages available?

@Reinmar
Copy link
Member

Reinmar commented Sep 7, 2017

Right now, ckeditor5-core doesn't have any dependency on other packages. Shouldn't this be included into ckeditor5 instead, which already has all packages available?

I explained why not ckeditor5 above:

I wouldn't like to add it to ckeditor5 because it's not a real package and because all other packages would need to have it in their devdeps.

It's not ideal to have nearly all packages as devdeps of ckeditor5-core too, but it's better this way. The third option is to have a special package for that (like ckeditor5-presets ;)).

@fredck
Copy link
Contributor Author

fredck commented Sep 8, 2017

This is just a development tool. As long as it is not messing up with the core package (like adding lots of deps in package.json), it can even be there. If it'll be messy though, then better to find a development dedicated place for it... like ckeditor5-dev or something.

Reinmar referenced this issue in ckeditor/ckeditor5-essentials Sep 8, 2017
…t util. See #1.

BREAKING CHANGE: The `Article` plugin was removed.
Reinmar referenced this issue in ckeditor/ckeditor5-essentials Sep 8, 2017
…`@ckeditor/ckeditor5-essentials`. The `Article` preset was removed. Caloses #1.

BREAKING CHANGE: This package was renamed to `@ckeditor/ckeditor5-essentials`.

BREAKING CHANGE: The `Article` presets was removed.
@Reinmar
Copy link
Member

Reinmar commented Sep 8, 2017

Closed by 7e5e82b.

@Reinmar Reinmar closed this as completed Sep 8, 2017
Reinmar referenced this issue in ckeditor/ckeditor5-basic-styles Sep 8, 2017
oleq referenced this issue in ckeditor/ckeditor5-core Sep 11, 2017
@mlewand mlewand transferred this issue from ckeditor/ckeditor5-essentials Oct 9, 2019
@mlewand mlewand added this to the iteration 12 milestone Oct 9, 2019
@mlewand mlewand added status:confirmed type:task This issue reports a chore (non-production change) and other types of "todos". package:essentials labels Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:essentials type:task This issue reports a chore (non-production change) and other types of "todos".
Projects
None yet
Development

No branches or pull requests

4 participants