Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#19341 closed defect (bug) (fixed)

Insert image from url not working on Press This

Reported by: ronbme Owned by: azaozz
Priority: high Milestone: 3.3
Component: General Version: 3.3
Severity: normal Keywords:
Cc: Focuses:

Description

The insert image from url is not working on Press This.
If the image does not show up in the preview window, there is no other way to insert the image.

I did a copy and paste on the image location and then clicked to insert it in the post. Nothing shows up.

This is using WP 3.3. trunk 19414.

Problem has existed for all versions of WP 3.3

Change History (9)

#1 follow-up: @jane
15 years ago

  • Keywords Insert image from url removed

@nacin: is this a regression? I have a buggy Press This now too, though wouldn't prioritize it and am not sure if it was working better in 3.2.
(Also, shouldn't we be pulling it out into a plugin? We should remember to do that.)

#2 @nacin
15 years ago

  • Milestone Awaiting Review3.3
  • Owner set to azaozz
  • Priority normalhigh
  • Status newassigned

Press This really isn't that difficult. It's just always had an ugly and neglected codebase. It should remain in core and be supplemented by browser extensions to replace the bookmarklet. It just needs the love of an enterprising developer for a single release.

It doesn't rely on the media code, it has it's own, so this wasn't broken from the media button merge work, AFAICT.

#3 @duck_
15 years ago

Caused by [19381] which caused image_selector() to use #tb_this_photo and #tb_this_photo_description. #photo-add-url-div also uses image_selector(), but it's inputs don't have the tb_ prefix.

#4 in reply to: ↑ 1 @ronbme
15 years ago

Replying to jane:

@nacin: is this a regression? I have a buggy Press This now too, though wouldn't prioritize it and am not sure if it was working better in 3.2.
(Also, shouldn't we be pulling it out into a plugin? We should remember to do that.)

There is a plugin named "Press This Reloaded". It actually goes further than Press This. But, unfortunately, it hasn't been updated.

#5 @azaozz
15 years ago

  • Resolutionfixed
  • Status assignedclosed

In [19424]:

Fix inserting images from url in Press This, fixes #19341

#6 @nacin
15 years ago

  • Resolution fixed
  • Status closedreopened

The margin: 10px pretty obviously broke styling of the stuffboxes.

#8 @nacin
15 years ago

  • Resolutionfixed
  • Status reopenedclosed

In [19491]:

Remove margin from h3's in press this. fixes #19341.

#9 @azaozz
15 years ago

In [19492]:

Fix css selectors mismatch in Press This, see #19341

Note: See TracTickets for help on using tickets.