Make WordPress Core

Opened 15 years ago

Closed 11 years ago

Last modified 11 years ago

#11344 closed defect (bug) (invalid)

'Show' link doesn't appear for about 10s after 'Crunching' is finished.

Reported by: caesarsgrunt's profile caesarsgrunt Owned by:
Milestone: Priority: normal
Severity: minor Version: 2.9
Component: Upload Keywords:
Focuses: Cc:

Description

After uploading an image, the 'Show' link doesn't appear for about 10s after 'Crunching' is finished.

Firefox 2.5.5, WP r12320 (but applies to all versions).

Attachments (1)

11344.diff (1.6 KB) - added by solarissmoke 14 years ago.

Download all attachments as: .zip

Change History (12)

#1 @caesarsgrunt
15 years ago

Sorry, Firefox 3.5.5...

#2 @azaozz
15 years ago

'Crunching' is the time when the server makes the image sub-sizes and depends on the image size. There was an idea to make it asynchronous so the next image can be uploaded while sub-sizes are being created but the memory usage would go through the roof.

What we can improve is image_resize(). It can create all sub-sizes in one go without destroying and then loading again the 'big' image for every sub-size.

#3 @caesarsgrunt
15 years ago

Yes, but what I meant was that after the Crunching has finished (the notice has disappeared), there is a longish delay before the "show" link appears for the image.

#4 @azaozz
15 years ago

Don't see this here, perhaps slow/congested Internet connection?

#5 @caesarsgrunt
15 years ago

Nope; I also get it on my local test server. I'll do some investigation at some stage, but I'm very busy at the moment.

#6 follow-up: @caesarsgrunt
15 years ago

Hmm... on my testing server, it actually appears after about 1s in tests I've just done. So I guess it is a slow internet connection as yo say (my internet connection is certainly slow, but it seems to happen at least to some extent when I use faster connections too).

Anyway - once the crunching message is gone, can't the 'show' link appear immediately? What is happening in between 'crunching' being hidden and 'show' being, er, shown?

#7 @nacin
14 years ago

  • Milestone changed from Unassigned to 3.1

#8 @nacin
14 years ago

  • Keywords reporter-feedback added
  • Milestone changed from Awaiting Triage to Future Release

#9 in reply to: ↑ 6 @solarissmoke
14 years ago

  • Keywords has-patch added; needs-patch reporter-feedback removed

Replying to caesarsgrunt:

What is happening in between 'crunching' being hidden and 'show' being, er, shown?

The image data is being loaded asynchronously, and the "Show" link appears once it has loaded - otherwise when you click on it the edit section would not yet be populated...

I think it would be better to leave the "Crunching..." message there until the data is loaded, to avoid that delay. Patch attached. I also noticed some variable duplication which I've remedied.

@solarissmoke
14 years ago

#10 @iseulde
11 years ago

  • Keywords has-patch removed
  • Resolution set to invalid
  • Status changed from new to closed

Media has been redone in 3.5.

#11 @helen
11 years ago

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.