News:

SMF 2.1.4 has been released! Take it for a spin! Read more.

Main Menu

loadTheme $context['browser'] -- no ie8?

Started by Angelina Belle, June 18, 2009, 08:29:47 PM

Previous topic - Next topic

Angelina Belle

I've got 1.1.9, as an upgrade from a clean 1.1.8 install (no mods at that point)

I noticed in loadTheme, where $context['browser'] is set, that the list does not include ie8.  Is this intentional? 

For ie8 users, $context['browser']['isie'] will also be incorrect.  I think this may cause a slight problem to simplePortal (sp_image_resize doesn't get called at the onload event, unless ie8 is run in compatability mode).

Is this a feature?  Or would it be a good idea to add an ie8 line in Load.php?

Thanks for the advice.

AB
Never attribute to malice that which is adequately explained by stupidity. -- Hanlon's Razor

JBlaze

Jason Clemons
Former Team Member 2009 - 2012

JBlaze

Quote from: cx_one on June 17, 2009, 07:48:39 PM
To implement IE8 Support modify Load.php

add
$context['browser']['is_ie8'] = strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE 8') !== false && !$context['browser']['is_opera'] && !$context['browser']['is_gecko'] && !$context['browser']['is_web_tv'];
before
$context['browser']['is_ie7'] = strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE 7') !== false && !$context['browser']['is_opera'] && !$context['browser']['is_gecko'] && !$context['browser']['is_web_tv'];


replace
$context['browser']['is_ie'] = $context['browser']['is_ie4'] || $context['browser']['is_ie5'] || $context['browser']['is_ie5.5'] || $context['browser']['is_ie6'] || $context['browser']['is_ie7'];
with
$context['browser']['is_ie'] = $context['browser']['is_ie4'] || $context['browser']['is_ie5'] || $context['browser']['is_ie5.5'] || $context['browser']['is_ie6'] || $context['browser']['is_ie7'] || $context['browser']['is_ie8'];
Jason Clemons
Former Team Member 2009 - 2012

karlbenson

#3
IE8, chrome, Safari4, Opera 10 were all released after 1.1 final and so are not fully detected.

Detection of these is built into 2.0 as standard.

Angelina Belle

#4
Thanks for the tip!
Never attribute to malice that which is adequately explained by stupidity. -- Hanlon's Razor

Snowy

Quote from: regularexpression on June 19, 2009, 08:05:21 AM
IE8, chrome, Safari4, Opera 10 were all released after 1.1 final and so are not fully detected.

Detection of these is built into 2.0 as standard.

Correct. I went to make the edits and they were already in my files. So... no change.

Advertisement: