WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 15 years ago

#8785 closed defect (bug) (fixed)

2.7 new press-this closes the pictures after picking one

Reported by: briantokyo's profile BrianTokyo Owned by: noel's profile noel
Milestone: 2.9 Priority: normal
Severity: major Version: 2.7
Component: Press This Keywords: needs-patch
Focuses: Cc:

Description

Since 2.7 whenever I pick a picture on the pictures box, it closes, so if I want to pick another one I have to open that box again and the whole thing reloads, making this useless (or very annoying/time consuming) if you want to use more than 1 picture.

Attachments (5)

press-this-5-12-7pm.patch​ (3.9 KB) - added by noel 16 years ago.
Fixes multiple issues, including layout, and image selection UI
press-this-6-9.patch​ (3.5 KB) - added by noel 16 years ago.
For rev 11538
press-this-css-tweaks-6-9.patch​ (3.7 KB) - added by noel 16 years ago.
some css tweaks
press-this-full-patch.patch​ (5.0 KB) - added by noel 16 years ago.
press-this-2.8-final-patch.patch​ (5.6 KB) - added by noel 16 years ago.
actual final patch

Download all attachments as: .zip

Change History (14)

#1 follow-up: @BrianTokyo
16 years ago

Noticed this isn't fixed yet in the nightly builds. Is this change to press-this done in purpose?

#2 @DD32
16 years ago

  • Keywords needs-patch added; press this pictures removed
  • Milestone set to 2.8

#3 in reply to: ↑ 1 @Denis-de-Bernardy
16 years ago

  • Milestone changed from 2.8 to Future Release

Replying to BrianTokyo:

Noticed this isn't fixed yet in the nightly builds.

Because there is no patch

Is this change to press-this done in purpose?

Probably not. It must have crept into the code base when #7949 was looked into.

#4 @Denis-de-Bernardy
16 years ago

  • Severity changed from critical to normal

@noel
16 years ago

Fixes multiple issues, including layout, and image selection UI

#6 @noel
16 years ago

  • Keywords has-patch added; needs-patch removed

@noel
16 years ago

For rev 11538

@noel
16 years ago

some css tweaks

@noel
16 years ago

@noel
16 years ago

actual final patch

#7 @Denis-de-Bernardy
16 years ago

  • Keywords needs-testing added
  • Milestone changed from Future Release to 2.9
  • Severity changed from normal to major

#8 @Denis-de-Bernardy
16 years ago

  • Keywords needs-patch added; has-patch needs-testing removed

r11540 broke the css tweaks in patch, else works great.

#9 @ryan
15 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.