Make WordPress Core

Opened 18 years ago

Closed 17 years ago

#3596 closed defect (bug) (invalid)

Manage>Uploads Single Image View's wide image Input boxes are Too Small

Reported by: ryanfitzer's profile ryanfitzer Owned by: mdawaffe's profile mdawaffe
Milestone: Priority: high
Severity: normal Version: 2.1
Component: Administration Keywords: invalid
Focuses: Cc:

Description (last modified by foolswisdom)

When managing an image via the Manage>Uploads single-image-view screen, if the image is too wide it squishes the URL, Title and Description input boxes to where they are unusable.

Attachments (3)

screengrab.jpg (237.9 KB) - added by ryanfitzer 18 years ago.
screenshot of problem
3596.diff (1.6 KB) - added by mdawaffe 18 years ago.
3596b.diff (1.6 KB) - added by mdawaffe 18 years ago.

Download all attachments as: .zip

Change History (23)

@ryanfitzer
18 years ago

screenshot of problem

#1 @ryanfitzer
18 years ago

  • Version set to 2.1

This is in 2.1 beta 4

#2 @foolswisdom
18 years ago

  • Description modified (diff)
  • Owner changed from anonymous to mdawaffle
  • Summary changed from Manage>Uploads Single Image View's Input boxes are Too Small to Manage>Uploads Single Image View's wide image Input boxes are Too Small

#3 @mdawaffe
18 years ago

  • Owner changed from mdawaffle to mdawaffe

waffe - no syrup needed.

#4 @mdawaffe
18 years ago

I need more info about this bug.

For that file, was a thumbnail actually created? It would be in the same place on your server as the original but probably called whatever.thumbnail.jpg instead of whatever.jpg (the original).

I've attached a patch which may solve this problem.

@mdawaffe
18 years ago

#5 @mdawaffe
18 years ago

  • Keywords has-patch commit added
  • Status changed from new to assigned

#6 @ryanfitzer
18 years ago

Yes, thumbnails were created for the images. The images were no larger than 780px wide (landscape format) when uploaded. I witnessed this in FF 2 on OSX 10.3.9 using a localhost environment via MAMP.

#7 @mdawaffe
18 years ago

Do you have the ability to apply the patch above called 3596.diff? If so, does it help with this issue or the other issue you reported (#3597 - I think they are related)?

Can you also do the following debugging for me?

From your screengrab, it seems as though the image with ID = 9 is one for which this problem occurs. Do you have CocoaMySQL or PHPMyAdmin installed or some otherway to look at the raw database? If so, in your _postmeta table, find all the rows with post_id = 9 and paste each here.

We will figure this out :)

#8 @mdawaffe
18 years ago

  • Priority changed from normal to high

#9 @ryanfitzer
18 years ago

All this is doable. I won't be able to get at the computer (work computer) until tomorrow as I'm stuck at home sick. I'll get you everything The morning of the 19th.

@mdawaffe
18 years ago

#10 @ryan
18 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [4767]) Upload manager image viewer fixes from mdawaffe. fixes #3596

#11 @mdawaffe
18 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

I'm not convinced this is fixed yet. Anyone seeing this problem please include the debugging info I mentioned above.

#12 @ryanfitzer
18 years ago

Sorry, haven't had a chance to implement this. Still recovering. First chance I get I'll post back.

#13 @foolswisdom
18 years ago

  • Milestone changed from 2.1 to 2.2
  • Owner changed from mdawaffe to ryanfitzer
  • Status changed from reopened to new

#14 @ryanfitzer
18 years ago

  • Keywords patch tested-problem still exists added; manage image has-patch commit removed

I'm gettin' back to normal here. Sorry for the delay.
Plugged in 4767 to upload-functions.php. WP is finding the thumbs just fine on images uploaded after the upgrade from 2.0.x to 2.1. Images uploaded while using 2.0.x are still being shown using the full image, therefore, collapsing the input boxes according to the dimensions of the images.

Requested Database Lines:

INSERT INTO wp_postmeta VALUES (26, 9, '_wp_attached_file', '/homepages/30/d174426947/htdocs/wp-content/uploads/2007/01/primedwall.jpg');

INSERT INTO wp_postmeta VALUES (27, 9, '_wp_attachment_metadata', 'a:5:{s:5:"width";i:780;s:6:"height";i:585;s:14:"hwstring_small";s:23:"height=''96 width=128";s:4:"file";s:78:"/homepages/30/d174426947/htdocs/wp-content/uploads/2007/01/primedwall.jpg";s:5:"thumb";s:24:"primedwall.thumbnail.jpg";}');

#15 @foolswisdom
18 years ago

  • Owner changed from ryanfitzer to mdawaffe

#16 @foolswisdom
18 years ago

  • Keywords needs-patch added; patch tested-problem still exists removed
  • Milestone changed from 2.2 to 2.3

#17 @ryan
18 years ago

  • Milestone changed from 2.3 to 2.4 (next)

#18 @westi
17 years ago

  • Cc andy added

I assume this will be fixed in the new uploader.

This appears to be missing functionality at present though.

#19 @MtDewVirus
17 years ago

  • Keywords invalid added; needs-patch removed

Using r7236, Manage Media doesn't provide a way to edit anything for the uploads. Basically all you can do is delete, view the posts associated with an upload, or view the uploaded file.

#20 @lloydbudd
17 years ago

  • Milestone 2.5 deleted
  • Resolution set to invalid
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.