WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#22545 closed defect (bug) (fixed)

New media: 'Insert Embed' button remains disabled when pasting with context menu

Reported by: johnbillion Owned by: nacin
Priority: normal Milestone: 3.5
Component: Media Version: 3.5
Severity: normal Keywords: has-patch commit dev-reviewed
Cc: Focuses:

Description

Steps to reproduce:

  1. Click 'Add Media' on the post editing screen
  2. Switch to the 'Embed From URL' tab on the left
  3. Paste a URL into the URL field by using the 'Paste' command in the context menu

Note that the 'Insert Embed' button at the bottom remains disabled. It only gets re-enabled if you type something in the URL field or if you were to use Ctrl/Cmd+V to paste.

Attachments (1)

22545.diff​ (524 bytes ) - added by koopersmith 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 @nacin
14 years ago

  • Milestone Awaiting Review → 3.5

@koopersmith
14 years ago

#2 @koopersmith
14 years ago

Adds the input event to observable inputs for newer browsers, and the change event to catch any stragglers (like paste, autocomplete, etc).

#3 @koopersmith
14 years ago

  • Keywords has-patch added

#4 @nacin
14 years ago

  • Keywords commit added

Tested, seems solid.

#5 @ryan
14 years ago

  • Keywords dev-reviewed added

Tested with Chrome and Firefox on Mac OS. Tested typing in a URL manually, Cmd+V, and context menu paste. Looks good.

#6 @nacin
14 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In 22836:

Media: Watch more events for input changes to catch paste, autocomplete, etc.

Fixes issue where pasting a URL in 'From URL' (via the context menu) did not
enable the insert button, as the event was not yet fired.

props koopersmith.
fixes #22545.

Note: See TracTickets for help on using tickets.