- ๐จ๐ฆCanada mgifford Ottawa, Ontario
@cmcnamee did you remove the tag wcag336 on purpose? Looks like it was accidental so am adding it back.
- ๐บ๐ธUnited States cmcnamee
That's strange, I didn't even expand the metadata box when commenting, so I'm not sure how the tag got removed. Thanks for adding it back in.
- ๐จ๐ฆCanada mgifford Ottawa, Ontario
Sometimes it is just bad timing..
- ๐ซ๐ฎFinland Nchase
thanks for looking into this! Highly appreciated!
- ๐บ๐ธUnited States jonmarshall
Thanks for considering this. It's essential to many content editors that I'm working with.
- ๐จ๐ฆCanada occupant Canada
In case anyone is interested, I've taken the plunge and adapted the plugin mentioned by @joelpittet in #10 ๐ [upstream] Support full-screen editing - distraction free mode when it is available upstream Active and created it as a standalone project โ for now.
- ๐ฌ๐งUnited Kingdom alexharries
Thank you @Occupant - that module is perfect for our needs.
A
- Status changed to Needs review
over 1 year ago 11:00am 10 November 2023 - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
I learned yesterday that https://www.drupal.org/project/ckeditor5_premium_features โ contains a free to use feature that addresses this:
Full screen mode
The full screen mode is a free-to-use plugin that maximizes the editing area. It is especially useful when using features like Document outline or Comments which take up extra space around the editor.(It was actually announced by @witeksocha 2 weeks ago at https://github.com/ckeditor/ckeditor5/issues/1235#issuecomment-1783119311.)
See https://git.drupalcode.org/project/ckeditor5_premium_features/-/blob/1.2...
What do people think?
- Status changed to RTBC
over 1 year ago 1:35pm 10 November 2023 - ๐บ๐ธUnited States smustgrave
Tested the module and seems to be working
- ๐ช๐จEcuador jwilson3
> What do people think?
How hard would it be to extract the fullscreen feature into its own contrib module (or into core), so as not to confuse end user admins who are not interested in premium features. Would a Drupal maintainer then need to maintain the plugin JS ourselves as a fork?
- Status changed to Needs review
over 1 year ago 2:02pm 10 November 2023 - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
@smustgrave What does "RTBC" mean here to you? That we don't need to fix it in core?
- ๐บ๐ธUnited States nessthehero
+1 to #24. If the plugin is free to use, it should be separated from the rest of the "pay to use" plugins, as it is confusing to be required to pay for all except one of the included plugins under a nomenclature of "premium".
I've tried out the fullscreen feature on a local site and it's pretty much exactly what our users want, so it would be nice if it was standalone.
- ๐บ๐ธUnited States smustgrave
Whatโs there to fix in core?
And agree with the last few comments. I donโt want the paid module for one feature. And is there agreement that wouldnโt become paid later?
- ๐ช๐จEcuador jwilson3
I almost missed comment #20. We have https://www.drupal.org/project/ckeditor5_fullscreen โ contrib module that works without the โpremiumโ module getting in the way. Of course the premium module is supported by the core CKE team, however, seems like it may be more resilient to go with a standalone module (which depends on a cke contrib plugin) that would presumable be deprecated and have a max Drupal version constraint added once Drupal core supports a version of CKEditor where theyโve actually built this plugin into the upstream CKE core at some future date.
If anything, the issue status here seems like it has to be "postponed", and then would automatically go to "closed (outdated)" once we update to said CKE version in core.
- Status changed to Postponed
over 1 year ago 3:27pm 15 November 2023 - ๐บ๐ธUnited States smustgrave
Agree this could be closed out once upstream is updated and we update core.
- Status changed to Active
over 1 year ago 10:23am 16 November 2023 - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
My understanding is that they will not be adding this to "CKEditor 5 core", because there are reasons for why they cannot make this feature work universally. I didn't dig in deep, but my high-level understanding is that every "integrator" of CKEditor 5 would have to build their own full screen mode, it just happens to be that they wrote one for us and are shipping it in that "Premium Features" module.
So, I don't think makes sense in that case. Unless/until we can convince them to eventually add it to "CKEditor 5 core".
- ๐ต๐ฑPoland witeksocha
We will not be adding it now but it's still in our plans to provide this feature in the core.
Creating a module for Drupal was just faster and easier for us, as we didn't need to think about all corner cases and editor types.
- ๐จ๐ฆCanada andrew.wang
FWIW, full screen mode is already provided in the official CKEditor 5 Plugin Pack as a โpremium plugins available for freeโ: https://www.drupal.org/project/ckeditor5_plugin_pack โ
- ๐ฎ๐ณIndia somdeb dasgupta Kolkata
I would vote for having the full screen feature inside CKEditor 5 core. Any idea, tentatively when this feature will be incorporated in CKEditor 5 core?
- ๐ง๐ชBelgium dieterholvoet Brussels
CKEditor 45.0.0 has been released with support for full screen mode: https://github.com/ckeditor/ckeditor5/releases/tag/v45.0.0
I opened ๐ Update CKEditor 5 to 45.0.0 Active to update CKEditor in core. - First commit to issue fork.
- Merge request !12406Support full-screen editing - distraction free mode โ (Open) created by el7cosmos
- ๐บ๐ธUnited States smustgrave
IS seems incomplete
But question why do we need this when https://www.drupal.org/project/ckeditor5_plugin_pack โ offers fullscreen?
- ๐ฆ๐บAustralia kobusvw Adelaide
Hi @smustgrave,
In the latest Drupal releases (10.5.2 and 11.2.2) using CKEditor5 v45.2.0, the Fullscreen feature is intended to be part of core. While the button is referenced and the related functionality appears to be present in the codebase, the actual CKEditor 5 plugin code required to enable Fullscreen is missing.
The following merge request addresses this issue by properly including the plugin implementation in core:
๐ merge request !12406Once this is merged, the Fullscreen button will become available in the CKEditor 5 Toolbar configuration interface as expected.
Hope it will be included in the next release.
- ๐บ๐ธUnited States smustgrave
Tagging for approval but this MR isnโt ready to be merged yet. Missing test coverage