- Issue created by @carstenhager
- First commit to issue fork.
- Merge request !8Issue #3467117: Fullscreen stopped working after update to 1.0.0-beta9 → (Open) created by sourav_paul
- 🇮🇳India sourav_paul Kolkata
Sourav_Paul → changed the visibility of the branch 3467117-fullscreen-stopped-working to hidden.
- Status changed to Needs review
3 months ago 11:37am 9 August 2024 Thanks for the quick reply. Unfortunately the problem still exists.
- 🇮🇳India sourav_paul Kolkata
Its working after applying the patch.
Please find the attached SR with it. I found the problem.
Our users do not have the following permissions.access toolbar
Without this permission the button no longer works
- 🇮🇳India sourav_paul Kolkata
@carstenhager have you checked? Is it working with or without the patch?
It makes no difference whether you have a patch or not.
It is the permission that causes the error.- 🇨🇦Canada occupant Canada
Thanks both. And thanks for confirming the issue cause @carstenhager. The toolbar was introduced into the plugin here 🐛 Fullscreen mode interferes with link button Needs review but it slipped past that this would require the toolbar to be required. I'll have to break this down and see about backing out of those changes.
- 🇺🇸United States segovia94
We are using the https://www.drupal.org/project/layout_builder_iframe_modal → module and are having failures for the same reason. The patch adding the dependency on the toolbar library doesn't help though because the the toolbar itself is not loaded within the iframe. So there will be errors that the toolbar library throws like
Uncaught TypeError: document.querySelector('#toolbar-administration') is null
.I would recommend making the toolbar optional rather than a hard dependency. Currently, any site not using the toolbar will get errors.
- 🇨🇦Canada occupant Canada
I've added a check for the toolbar to the dev branch that should alleviate this issue - https://www.drupal.org/project/ckeditor5_fullscreen/releases/1.0.x-dev →
If all looks ok, I'll make a new release.
- Assigned to occupant
Unfortunately, I get the following error in the dev branch:
Uncaught CKEditorError: getElementById is not defined- 🇨🇦Canada occupant Canada
@carstenhager I have altered the toolbar check in the dev branch again (properly this time!) and it is working in my tests. If this looks good, I'll go ahead with a new release.
- Status changed to Fixed
2 months ago 5:31pm 10 September 2024 - Status changed to Fixed
2 months ago 5:32pm 10 September 2024