WordPress.org

Make WordPress Core

Opened 7 months ago

Last modified 7 months ago

#22567 new defect (bug)

Preview button triggers popup blocker in some browsers if no title is set

Reported by: miqrogroove Owned by:
Priority: normal Milestone: Awaiting Review
Component: Editor Version:
Severity: normal Keywords: has-patch
Cc: mdhansen@…

Description

Steps to reproduce:

  1. Click on Add New (for a post)
  2. DO NOT enter a title.
  3. Start typing in the post body.
  4. Click Preview

Expected result:

New tab opens with preview.

Actual result:

If popup blocking is disabled, a new window opens with the preview. On subsequent clicks of the Preview button, it reverts to opening a new tab as it should.

Found in WordPress 3.5-RC1.

Attachments (2)

22567.patch​ (13.9 KB) - added by mario-siteground 7 months ago.
Patch adding async false
22567-new.patch​ (431 bytes) - added by mario-siteground 7 months ago.
Update with proper tabs

Download all attachments as: .zip

Change History (10)

comment:1 miqrogroove — 7 months ago

Able to reproduce this in Chrome, Firefox, and IE, but not in Opera.

comment:2 miqrogroove — 7 months ago

  • Summary changed from Preview Button Triggering Popup Blocker in RC1 to Preview Button Triggering Popup Blocker
  • Version changed from trunk to 3.2

ocean90 says this goes back to 3.2. Is there another ticket?

comment:3 ocean90 — 7 months ago

  • Severity changed from major to normal
  • Summary changed from Preview Button Triggering Popup Blocker to Preview button triggers popup blocker in some browsers if no title is set
  • Version 3.2 deleted

Even 2.9 and probably before.

mario-siteground — 7 months ago

Patch adding async false

comment:4 mario-siteground — 7 months ago

  • Keywords has-patch added

Disabling the autosave async mode seems to be fixing the issue without causing any extra troubles (tested in trunk and 3.4)

comment:5 SergeyBiryukov — 7 months ago

There's no need to patch minified files, a post-commit bot takes care of that.

Also, please use tabs rather than spaces (per ​WordPress Coding Standards).

mario-siteground — 7 months ago

Update with proper tabs

comment:6 mario-siteground — 7 months ago

Sergey, better now?

comment:7 SergeyBiryukov — 7 months ago

22567-new.patch​ looks proper and appears to fix the issue in my testing.

comment:8 MikeHansenMe — 7 months ago

  • Cc mdhansen@… added

I actually cam across this the other day as well. Tested 22567-new.patch and it corrected the problem for me as well.

Version 0, edited 7 months ago by MikeHansenMe (next)
Note: See TracTickets for help on using tickets.