Make WordPress Core

Opened 14 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:


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 13 years ago.

Download all attachments as: .zip

Change History (12)

#1 @caesarsgrunt
14 years ago

Sorry, Firefox 3.5.5...

#2 @azaozz
14 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
14 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
14 years ago

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

#5 @caesarsgrunt
14 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
14 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
13 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.

13 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.