[upstream] CKEditor 5 not asking confirmation before closing or reloading tab

Created on 27 November 2023, 7 months ago
Updated 28 November 2023, 7 months ago

Problem/Motivation

Drupal 10 with CKE 5 not showing Reload site? popup after body change

Steps to reproduce

Edit the existing node
Make the change to the body or field with CKEditor
Now hit on browser reload icon
It dones not show Roload site? browser popup

Feature request
Status

Active

Version

11.0 🔥

Component
CKEditor 5 

Last updated less than a minute ago

Created by

🇮🇳India vipul tulse

Live updates comments and jobs are added and updated live.
  • Usability

    Makes Drupal easier to use. Preferred over UX, D7UX, etc.

Sign in to follow issues

Comments & Activities

  • Issue created by @vipul tulse
  • 🇺🇸United States cilefen

    I do not understand what this bug report means.

  • Status changed to Postponed: needs info 7 months ago
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    Can you provide some screenshots of what you were expecting to see?

  • 🇮🇳India vipul tulse

    Please check attached image.
    When you do edit in the node and make the changes to the node body where the Creditor available and try to refresh the page, it is not asking me the Reload Page? browser popup.
    I will make video and post for more information.

  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    I don't ever recall seeing that functionality, in CKE4 or 5.
    Are you sure it wasn't provided by custom code or a module?

  • Status changed to Active 7 months ago
  • 🇧🇪Belgium Wim Leers Ghent 🇧🇪🇪🇺

    That's indeed browser behavior, not CKEditor 4/5-specific logic.

    I do think it's fair to expect CKEditor to be working in such a way that it protects the user against data loss.

    And I think it's because the <textarea> is not updated until the user hits "Save".

    Verify this yourself using document.querySelector('textarea#edit-body-0-value').value: it remains the empty string. I suspect this behaved differently in CKEditor 4.

    We should report this upstream — I can't find an existing issue for it: https://github.com/search?q=repo%3Ackeditor%2Fckeditor5+changes+may+not+...

Production build 0.69.0 2024