Sitepatching updates

tweaking the broken code until it works, one site at a time

Small update

By Ola P. Kleivenolak. Monday, June 18, 2012 12:40:16 PM

Added patches

PATCH-704, nbs.rs: fix iframe resize. Workaround for Opera issue that was fixed a long time ago now causes fail.

PATCH-703, insubuy.com: don't prevent mouse click.

PATCH-699, engfilms.ru: needs navigator.product to work.

Removed patches

PATCH-651, jcpenney: avoid broken sniffer. Site changed.

Various sniffer and script execution order patchesDell++

Comments

d4rkn1ght Monday, June 18, 2012 12:42:13 PM

up

Charles SchlossChas4 Monday, June 18, 2012 1:59:25 PM

Target version and time stamp of the active browser.js file is Opera Desktop 12.00 core 2.10.289, June 18, 2012. Active patches: 207 .

Use Opera try to apply for a job on http://smashburger.com/about/jobs/
sent to:
https://www.pfrsolutions.com/WDS3/Message.aspx?MessageCode=UnsupportedBrowser


If you mask as Firefox if works 100% fine

ouzowtfouzoWTF Tuesday, June 19, 2012 10:07:18 AM

up

rseiler Thursday, June 21, 2012 12:15:51 AM

Hi. Dell has changed their support URLs such that it's no longer support.dell.com but rather dell.com/support. I noticed support.dell.com is in browser.js, so I'm wondering if this is what accounts for driver download pages no longer displaying correctly.

To reproduce:
1) http://www.dell.com/support/drivers
2) "Choose from a list of Dell products" and Continue
3) Choose a specific laptop, desktop, etc model.
4) Once on your product page, try to expand any of the categories, such as Application, Audio, Chipset, etc.

The problem is that each item in a given category is broadly spaced away from adjacent items, with various pieces of text floating in between.

Ola P. Kleivenolak Thursday, June 21, 2012 5:00:44 PM

@rseiler: thanks for checking. That patch does indeed not apply anymore so I've disabled it. The broken display is caused by another issue. They have something like this:

.someclass{ 
  white-space: -moz-pre-wrap !important; 
  white-space: -pre-wrap; 
  white-space: -o-pre-wrap; 
 }


Now, these do not apply in any browser, except the -o-pre-wrap. If they applied it would be broken in Gecko as well.

I have a made a new browser.js patch for this for now, due on Monday. We have also removed support for -o-pre-wrap from Presto/2.11.310 (leaving only the unprefixed version) so this will fix itself eventually.

But in essence, those are three completely wasted lines of CSS ;-)

rseiler Friday, June 22, 2012 7:24:59 AM

It worked out well then, thanks.

Scorpion2012 Friday, June 22, 2012 10:05:28 AM

I'm getting a Warning your browser may not be supported at http://tennislink.usta.com/LEAGUES/Common/Default.aspx the only way to get rid of the message is to mask as Firefox. I reported this bug last year with the previous opera versions and I'm still getting the error with Opera 12, here's the bug number DSK-340429

Ola P. Kleivenolak Friday, June 22, 2012 1:24:36 PM

That is USTA's choice, not a bug in Opera. We have contacted them and they said they would look into maybe adding more browsers to the supported list. Chrome and Safari get the same message.

Scorpion2012 Friday, June 22, 2012 10:45:57 PM

Thanks Ola with the info, finally got answer on it. appreciate it.

Charles SchlossChas4 Sunday, June 24, 2012 7:31:31 PM

http://my.opera.com/community/forums/topic.dml?id=1445352 is about smashburger

Charles SchlossChas4 Sunday, June 24, 2012 7:40:19 PM

Originally posted by olak:

That is USTA's choice, not a bug in Opera. We have contacted them and they said they would look into maybe adding more browsers to the supported list


I see no message now using Opera smile

Write a comment

New comments have been disabled for this post.