Closed Bug 878509 Opened 6 years ago Closed 6 years ago
Restore the HTMLImage
Element .lowsrc, x, y attributes that were unintentionally removed
171.46 KB, image/jpeg
171.62 KB, image/jpeg
1.26 KB, text/html
1.28 KB, text/html
2.84 KB, patch
|Details | Diff | Splinter Review|
11.58 KB, text/html
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:21.0) Gecko/20100101 Firefox/21.0 (Beta/Release) Build ID: 20130511120803 Steps to reproduce: I upgraded Firefox browser to latest version 21.0. Actual results: All web sites created with DHTML rollover swap image effect now display a broken image. Expected results: Mouse overs should produce a different linked image. The DHTML effect still visible in IE10 and other browsers
Do you have an example link where it isn't working?
bug 864367 mentions this started in Firefox 20. Changing product to match that one.
Assignee: nobody → english-us
Component: Untriaged → English US
Product: Firefox → Tech Evangelism
Target Milestone: --- → Jun
Version: 21 Branch → Trunk
Confirming that the image rollover code works by changing: document['fpAnimswapImgFP6'].src=document['fpAnimswapImgFP6'].lowsrc to document['fpAnimswapImgFP6'].src=document['fpAnimswapImgFP6'].getAttribute('lowsrc'); Bob, let us know when you've updated all the code on your sites replacing every instance of .lowsrc with .getAttribute('lowsrc') Please visit https://bugzilla.mozilla.org/show_bug.cgi?id=878509#c8 and reply there instead of directly to my email address.
Summary: DHTML rollover swap image effect no longer visible in version 21, version 18 OK → DHTML rollover swap image effect no longer visible in Firefox 20 onwards
Assignee: english-us → nobody
Component: English US → DOM
Product: Tech Evangelism → Core
Target Milestone: Jun → ---
6 years ago
Wait. If sites depend on these, don't we need a spec bug to have them in the spec?
And since you're around and smaug doesn't seem to be, feel free to r+ :)
Comment on attachment 757086 [details] [diff] [review] Patch v1 A test would be nice.
Attachment #757086 - Flags: review?(bugs) → review+
Hi Everyone, First let me say thank you for the attention and the workaround. I really would have hated to go to another browser as I have been using Firefox quite awhile now and really like it. Absolutely dislike all other browsers. IE10 destroys the drop menu graphics and button character used in my shopping carts! I'm impressed with your community of code debuggers and your jump to fix attitude. Thank you. FrontPage 2000 uses shared borders and the code search and replace had to be done in the separate border HTML file, it does not show in the main page HTML view on all but one site listed. These (below) are the web sites I manage which have the DHTML rollovers I created using the code found within FP2000 itself and your fix was right on target... http://www.wps.org/ http://www.catskillmtncrystal.com/ http://www.stonerivergear.com/ http://www.stonerivergearpromos.com/ http://www.stonerivertradingcompany.com/ http://www.stormphotoinc.com/ I realize FP 2000 is an EOL product, but when you have websites with huge shopping carts designed in that environment and clients who are unwilling to pay me to change the interface because of outdated software, one has a major problem! I'm stuck using the software until the client decides otherwise. Like I said earlier about the software, I'm the master of it and it's not the master of me, thankfully! Not to sound ungrateful, but this fix is on my side and if I read correctly in the bug fix CC's the error is in Firefox's domain. I would like to be using the software as is and not having to do a search and replace each time I decide to use a rollover. Is that possible? Thanks again for your help. My clients also thank you I'm sure! Bob Sickler
Landed the fix: https://hg.mozilla.org/integration/mozilla-inbound/rev/2d5a92daf4f1 Working on a test now
Comment on attachment 757086 [details] [diff] [review] Patch v1 [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 825527 User impact if declined: broken sites (and quite a few, it seems) Testing completed (on m-c, etc.): on inbound Risk to taking this patch (and alternatives if risky): low risk String or IDL/UUID changes made by this patch: none
Oops, I forgot the most important step(In reply to Mardeg from comment #17) > then do Panels -> Style Properties, in that dialog from the > "Apply styles to" dropdown choose "All elements of class.." and pick > "topbar" from the adjacent dropdown. You need to tick the box (below those dropdowns) for "when the pointer hovers over the element" to choose rollover colours!
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
I'm looking at writing a test as well.
Comment on attachment 757086 [details] [diff] [review] Patch v1 Low risk fix for recent regression with web regressions that will make it into b4. Approving.
Hi Folks.. Bug Finder here... If you implement a patch fix, how will that affect the change in code I made to my web sites to make it work currently? Bob Sickler
In the patched builds, both the old code (.lowsrc) and the new code (.getAttibute("lowsrc")) will work.
This needs ba+ due to it touching a .webidl file.
Oh, "binary approval"? webidl is not xpidl. It's JS-only; changing it does not affect binary anything.
I filed bug 879086 for fixing the commit hook.
Verified fixed with Firefox 22 beta 4 (build ID: 20130605070403), using the testcase from comment 7 and the link: http://www.catskillmtncrystal.com/, on both Ubuntu 12.10 32bit and Mac OSX 10.8.3
You need to log in before you can comment on or make changes to this bug.