Broken images when using unmanaged images in IE and Qx 1.1

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Broken images when using unmanaged images in IE and Qx 1.1

John Spackman

Hi,

 

Our app shows a list of images uploaded on the server, i.e. unmanaged images and while the image displays OK in IE it also has the “broken image” X overlaid.  The img tag has it’s src attribute set to the string “undefined”, but removing the attribute altogether (or hacking qx.bom.element.Decoration to not set the src attribute in the first place) just shows a different broken image icon over my image.

 

I found that by patching qx.bom.element.Decoration line 114 to supply a blank gif for the src attribute fixes the problem; shall I add a bug report or is there something else I should have done to fix this?

 

Thanks

John


------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
lucky parental unit.  See the prize list and enter to win:
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|

Re: Broken images when using unmanaged images in IE and Qx 1.1

Alexander Steitz
Hi John,

Am 14.06.2010 15:59, schrieb John Spackman:

> Our app shows a list of images uploaded on the server, i.e. unmanaged
> images and while the image displays OK in IE it also has the “broken
> image” X overlaid.  The img tag has it’s src attribute set to the string
> “undefined”, but removing the attribute altogether (or hacking
> qx.bom.element.Decoration to not set the src attribute in the first
> place) just shows a different broken image icon over my image.
>
> I found that by patching qx.bom.element.Decoration line 114 to supply a
> blank gif for the src attribute fixes the problem; shall I add a bug
> report or is there something else I should have done to fix this?
Sounds like you found a bug and a bugfix right together :)

Can you please file a bug report with your proposed patch? This way we
can take a closer look at the fix and incorporated it into the framework
code.

Thanks for the report and the fix!

cheers,
   Alex



------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
lucky parental unit.  See the prize list and enter to win:
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|

Re: Broken images when using unmanaged images in IE and Qx 1.1

John Spackman-3
Sure thing - http://bugzilla.qooxdoo.org/show_bug.cgi?id=3767

-----Original Message-----
From: Alexander Steitz [mailto:[hidden email]]
Sent: 15 June 2010 17:14
To: qooxdoo Development
Subject: Re: [qooxdoo-devel] Broken images when using unmanaged images in IE
and Qx 1.1

Hi John,

Am 14.06.2010 15:59, schrieb John Spackman:

> Our app shows a list of images uploaded on the server, i.e. unmanaged
> images and while the image displays OK in IE it also has the "broken
> image" X overlaid.  The img tag has it's src attribute set to the string
> "undefined", but removing the attribute altogether (or hacking
> qx.bom.element.Decoration to not set the src attribute in the first
> place) just shows a different broken image icon over my image.
>
> I found that by patching qx.bom.element.Decoration line 114 to supply a
> blank gif for the src attribute fixes the problem; shall I add a bug
> report or is there something else I should have done to fix this?
Sounds like you found a bug and a bugfix right together :)

Can you please file a bug report with your proposed patch? This way we
can take a closer look at the fix and incorporated it into the framework
code.

Thanks for the report and the fix!

cheers,
   Alex



----------------------------------------------------------------------------
--
ThinkGeek and WIRED's GeekDad team up for the Ultimate
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
lucky parental unit.  See the prize list and enter to win:
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel



------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
lucky parental unit.  See the prize list and enter to win:
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel