Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#22048 closed defect (bug) (duplicate)

Confirm Navigation dialog on post.php when no changes have been made

Reported by: c3mdigital's profile c3mdigital Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.5
Component: Editor Keywords:
Focuses: Cc:

Description

To reproduce bug:

Go to any post and click update. Click view post or navigate away from post.php

You will get the confirm navigation javascript alert message.

Change History (10)

#1 follow-up: @TobiasBg
12 years ago

I can't reproduce this on trunk.
Tried different scenarios, but only got the message when I had made changes without saving, as expected.

#2 in reply to: ↑ 1 ; follow-up: @c3mdigital
12 years ago

Replying to TobiasBg:

I can't reproduce this on trunk.
Tried different scenarios, but only got the message when I had made changes without saving, as expected.

I've been doing some more testing and this is only happening on posts that have an inserted image in visual mode.

#3 in reply to: ↑ 2 @SergeyBiryukov
12 years ago

  • Milestone changed from Awaiting Review to 3.5

Replying to c3mdigital:

this is only happening on posts that have an inserted image in visual mode.

Confirmed.

#4 @TobiasBg
12 years ago

Ah, yes, for that case I can also confirm it.
Clicking "Update" is not even necessary. Just opening a post/page with an image results in that message when navigating away without doing changes, but only when the "Visual" editor is used.

#5 @nacin
12 years ago

Can someone confirm this is not the case in 3.4? Also, there were two distinct changes here that could have caused this: either r21545 (TinyMCE 3.5.6) or the more recent media changes.

#6 @SergeyBiryukov
12 years ago

Introduced in [22012].

#7 @markjaquith
12 years ago

I did a git bisect, and it says [22120] was the introduction of the bug.

#8 @markjaquith
12 years ago

Need to be on Visual tab.

  • [22012] is where it broke when a single image is embedded.
  • [22120] is where it broke when an image gallery is embedded.

#9 @SergeyBiryukov
12 years ago

Related: #22266 (has an explanation)

Last edited 12 years ago by SergeyBiryukov (previous) (diff)

#10 @SergeyBiryukov
12 years ago

  • Keywords needs-patch removed
  • Milestone 3.5 deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate of #22266.

Note: See TracTickets for help on using tickets.