ckeditor / ckeditor5-demos

Source codes for demos available at https://ckeditor.com/ckeditor-5/demo/
46 stars 65 forks source link

t/26 - add Production Pack demo + CKE5/CKBox version bump #27

Closed usqr closed 1 year ago

usqr commented 1 year ago

Closes: #26

mlewand commented 1 year ago

Quite a big PR, it would make sense for you to pass more context what the change is all about.

Since the changes were significant let's also ask our great @ckeditor/qa-team team to test the demos before going with next review round. @usqr can you ask them and handle the communication (including information what was changed and what was the reason for it)?

usqr commented 1 year ago

@mlewand So some background - There are lots of changes going on in nova on demos (bug fixes, feature updates, new demos). With that amount of work and a small team we don't have time to keep public repo up to date at all times so when we have some more time in a sprint we try to bundle those changes.

I will squash all the bugs and pass it on to the QA team. :+1:

usqr commented 1 year ago

Resolutins are not necessary any more.

Resolutions removed

  • It's a big PR so lest ask QA Team to test the demos,

QA task: cksource/qa-internal#239

  • One thing i've noticed - in Full featured demo there isn't Show blocks feature enabled. (on ckeditor.com also),

This Repo just reflects CKEditor.com demos - most of the changes are requested/approved by marketing... :man_shrugging:

  • new Productivity Pack demo - for me it looks a bit odd without any information why it does not work out of the box (i would add some info text in index.html),
  • similar situation in user-interface-document demo, but there is also o visibility issue, when column on the left is empty (it's reserved place for document outline),
  • in Full featured demo in content we mention about CKBox, but (same as above) to make it work we need to look in the code and read about missing license keys - info in index.html would be best from my point of view - but of course - discussion needed 🙂

This an issue of the fact that we are using premium features in those demos we can't provide licence keys/token/other access data in public facing demo yet we want to have feature parity with the demos on cke.com.
it's the same situation with source code for the demos in the docs: https://ckeditor.com/docs/ckeditor5/latest/examples/builds-custom/collaborative-document-editor.html#editor-example-configuration
But I agree that adding some sort of info in the index files would be good. WDYT @Witoso :thinking:

charlttsie commented 1 year ago

@usqr Slash command plugin is not added to the user-interface-button-grouping demo code (it's present on ckeditor.com).