View Issue Details

IDProjectCategoryView StatusLast Update
0004452mantisbtotherpublic2009-06-15 00:53
Reporterastax Assigned To 
PrioritynormalSeveritytweakReproducibilityalways
Status acknowledgedResolutionopen 
Summary0004452: All images should have sizes defined
Description

All displayed images should have correct sizes defined. Otherwise, when images are switched off in browser, page looks corrupted.

This affects "My View", "View Issues", "Summary" ... almost all pages.

TagsNo tags attached.

Relationships

child of 0004181 closed Features in Mantis 1.1 release 

Activities

DGtlRift

DGtlRift

2004-09-01 07:48

reporter   ~0007353

This should also speed up page rendering in some browsers since the browser will not have to wait for images to load to determine table sizes based on the images - in other browsers it will keep that seizure loading style to a minimum since they will not dynamically re-render the page every time an image is loaded.

jlatour

jlatour

2004-09-01 17:36

reporter   ~0007376

I take it this is mainly for the images we ship, but also for the attached images?

astax

astax

2004-09-01 23:09

reporter   ~0007389

Last edited: 2004-09-01 23:10

I don't think this makes sense for attached images, as you'll have to rely on GD functions in PHP to determine size and also this will increase server load to do this each time.

edited on: 09-01-04 23:10

jlatour

jlatour

2004-09-02 01:59

reporter   ~0007392

Well, we can also do it when adding the files. Oh well, let's do the other images first.

DGtlRift

DGtlRift

2004-10-27 10:33

reporter   ~0008185

Last edited: 2004-10-27 10:34

Issue 1: Icons and various built-in Mantis images.
Option 1: Go through and verify sizes on all image refs, repeat everytime an image is updated.
Option 2: Use lang_get() when refering to images.
ie. view_all_inc.php:125 reads
echo '<img src="' . $t_icon_path . 'attachment.png' . '" alt="" />';
change to
echo '<img src="' . $t_icon_path . lang_get('image_attachment') . '" height="' . lang_get('image_attachment_Y') . '" width="' . lang_get('image_attachment_X') . '" alt="" />';
and add coresponding tokens to lang/strings* files.

Issue 2: Uploaded images by users.
Option 1: add new column (meta) to mantis_bug_file_table

mantis_bug_file_table
id, bug_id, ..., file_type, ..., content, meta

where meta will be a string based on file_type.. if file_type is image/*, meta will contain a string 340X240, or some well known format that indicates height and width. This information will be populated and calculated once when a file is uploaded to the server. For upgrades there would be an admin utility to process all images already in the db when adding the new column.

Comments?

edited on: 10-27-04 10:34

Matt_wc

Matt_wc

2004-10-27 20:03

reporter   ~0008189

I would like to see Option 2 for Issue 1: Allows for customisation of the images properly.

astax

astax

2004-10-28 09:38

reporter   ~0008198

I don't think these icons really need customization, so it's better to keep the system simple and not overload configuration files with each image name and dimentions.

Matt_wc

Matt_wc

2004-11-03 00:35

reporter   ~0008230

Now that I think about it - I agree with astax. The icons are standard, only the main images would need to be changed though customisation.

siebrand

siebrand

2009-06-13 04:11

developer   ~0022132

Unassigned after having been assigned for multiple years without progress.

astax

astax

2009-06-14 22:14

reporter   ~0022156

Just curious, why so simple change that would take 5 or 10 minutes to do takes years to actually be done? (I don't expect any answer)

vboctor

vboctor

2009-06-15 00:53

manager   ~0022157

Patches are welcome. It all comes to priority and a dev or community member deciding to spend their time on a specific issue.