[dev] Re: [cvs] commit: imp mailbox.php

Eric Rostetter eric.rostetter at physics.utexas.edu
Thu Feb 13 17:14:59 PST 2003


Quoting Jan Schneider <jan at horde.org>:

> > >   Chora Links:
> > >   http://cvs.horde.org/diff.php/imp/mailbox.php?r1=2.470&r2=2.471&ty=u
> >
> > Chuck et al -- are these changes okay?
> 
> Yes, looks fine. Though I'm not sure if we should set the image dimensions.

That was one of my concerns :)

> If we don't set them, people can easily customize the ui design without
> using the same graphic sizes or modifying the code.

But, if we don't set them, the MSIE does rather nasty rendering (draws the
page with assumed sizes, then redraws it as it receives each image and finds
out how big it really is) making it look like the screen is jumping around
while it is loading.  Netscape/Mozilla don't have this problem, as they wait
until they know the sizes and then render...
 
I have a local patch to /horde/lib/Horde.php (the Horde::img() function)
to insert sizes.  But, I'm willing to edit all the darn files I need to
in order to insert the sizes in-line if it is desirable.  But, I'd like
feedback on this.

Once this is done, changing the size of an image will be a pain.  But, if we
standardize the images to 13x13 and 16x16, then we should never have to
change the size settings, as any new images we make would be the same size.
This wouldn't always apply to others making local mods with larger graphics,
for example, but for the Horde project itself it could hold.

Open to suggestions, ideas, complaints, etc.

> Jan.

-- 
Eric Rostetter
The Department of Physics
The University of Texas at Austin

Why get even? Get odd!


More information about the dev mailing list