WordPress.org

Make WordPress Core

Opened 3 months ago

Last modified 2 weeks ago

#52621 new defect (bug)

Leave site? / Save alert

Reported by: garbagedisco Owned by:
Milestone: 5.8 Priority: normal
Severity: normal Version: 5.6.2
Component: Editor Keywords: needs-patch
Focuses: ui Cc:

Description

The pop up alert for "Changes may not be saved" is still present in 5.6.2

Site example:
The classic editor plugin is installed on the site,
Gutenberg is disabled completely
Custom post types which don't have the WYSIWYG editor on still have this message appearing.

This bug was introduced in 5.6.1 and said to be fixed in 5.6.2 - it's still in the codebase

Attachments (1)

Screenshot 2021-02-23 at 08.30.18.png (107.8 KB) - added by garbagedisco 3 months ago.

Download all attachments as: .zip

Change History (8)

#1 @SergeyBiryukov
3 months ago

  • Component changed from General to Editor
  • Version changed from trunk to 5.6.2

Hi there, welcome to WordPress Trac! Thanks for the report.

Just linking to the previous ticket here: #52440.

#2 @garbagedisco
3 months ago

  • Component changed from Editor to General
  • Version changed from 5.6.2 to trunk

The fix here seems to work for anyone with this issue:

https://core.trac.wordpress.org/ticket/52440#comment:52

#3 @SergeyBiryukov
3 months ago

  • Component changed from General to Editor
  • Version changed from trunk to 5.6.2

#4 @SergeyBiryukov
3 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 5.8

Yeah, the intention of [50232] was to restore the functionality as it was in 5.6, but it looks like that can still unnecessarily display the message under some circumstances, as noted in comment:27:ticket:52440.

So apparently the enhanced fix suggested in comment:28:ticket:52440 would still be preferable here.

Moving to 5.8 for now, pending a patch. When a patch is available and tested, it can potentially go into the next minor release.

This ticket was mentioned in Slack in #core by desrosj. View the logs.


3 months ago

#6 @archon810
2 weeks ago

As I mentioned in https://core.trac.wordpress.org/ticket/52440#comment:41, the solution there wasn't working for us either and I had to resort to the functions.php fix. Glad someone else pushed to have a new ticket for this and I'm not the only one with the issue.

#7 @bradley2083
2 weeks ago

I've tried the functions.php fix and it isn't working for me. I have two custom post types registered in plugins I've added and those both support title, editor and excerpt. I also see the issue if I delete those plugins (and therefore post types).

Note: See TracTickets for help on using tickets.