06885: Long survey pages jump down the page in IE8 compatibility view

This bug affects 1 person(s).

ID	Project	Category	View Status	Date Submitted	Last Update

06885	Bug reports	Theme editor	public	2012-11-15 21:19	2012-11-16 10:40

Reporter	pfpDave	Assigned To	tpartner
Priority	normal	Severity	minor
Status	closed	Resolution	fixed
Product Version	2.00+

Summary	06885: Long survey pages jump down the page in IE8 compatibility view
Description	When using default or Limespired templates with a long page of questions in IE8 having Compatibility view switched on, pages load part way down. After much searching I've narrowed it down to lines 100 - 102 of /scripts/survey_runtime.js: `if ($.browser.msie && $.browser.version.substr(0,1)<8 && $('button.submit').length > 0) { // Get rid of the focus outline in IE7 $('#movenextbtn, #movesubmitbtn').focus().blur(); }` I've commented out this code as for my purposes I don't need it.
Steps To Reproduce	Create a long page survey assign default or Limespired templates (or any others that use the standard survey_runtime.js include) Switch on IE 8 compatibility view load the page.
Tags	No tags attached.

Bug heat	16
Complete LimeSurvey version number (& build)	121115
I will donate to the project if issue is resolved	No
Browser	IE8
Database type & version	SQL Express 2012
Server OS (if known)	Server 2008
Webserver software & version (if known)	IIS 7
PHP Version	5.4.8

User List	There are no users monitoring this issue.

c_schmitz 2012-11-15 21:44 administrator ~22217	Don't switch on compatibility mode.

pfpDave 2012-11-15 21:47 reporter ~22218	Compatibility view is enabled by default for Intranet sites in IE8 which means corporate customers using Limesurvey on an intranet will have this issue. See more info here: http://blogs.msdn.com/b/ie/archive/2009/06/17/compatibility-view-and-smart-defaults.aspx

pfpDave 2012-11-15 21:50 reporter ~22219	I've now tested this in an IE7 configuration (using IE Tester) and I get the same result so this issue will also affect those using IE7

c_schmitz 2012-11-15 22:04 administrator ~22220	Tony, any ideas?

tpartner 2012-11-15 22:19 partner ~22221	Ah, Bill strikes again! pfpDave, can you please attach a survey with an appropriately long group (page)?

c_schmitz 2012-11-15 22:46 administrator ~22222	Tony, there is an example survey attach to the related issue.

tpartner 2012-11-15 23:54 partner ~22223	Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=10317

tpartner 2012-11-15 23:55 partner ~22224	Fix committed to 2.1 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=10318

tpartner 2012-11-15 23:56 partner ~22225	Fixed in master and 2.1.

c_schmitz 2012-11-16 10:40 administrator ~22242	New version 2.00+ Build 121116 released

LimeSurvey: master 71ddf2eb 2012-11-15 22:53:34 tpartner Details Diff	Fixed issue 06885: Long survey pages jump down the page in IE8 compatibility view Dev Forced scroll to top when in IE compatibility mode	Affected Issues 06885
	mod - scripts/survey_runtime.js	Diff File
LimeSurvey: 2.1 eacadb50 2012-11-15 22:54:36 tpartner Details Diff	Fixed issue 06885: Long survey pages jump down the page in IE8 compatibility view Dev Forced scroll to top when in IE compatibility mode	Affected Issues 06885
	mod - scripts/survey_runtime.js	Diff File

Date Modified	Username	Field	Change
2012-11-15 21:19	pfpDave	New Issue
2012-11-15 21:44	c_schmitz	Note Added: 22217
2012-11-15 21:44	c_schmitz	Status	new => closed
2012-11-15 21:44	c_schmitz	Assigned To	=> c_schmitz
2012-11-15 21:44	c_schmitz	Resolution	open => won't fix
2012-11-15 21:47	pfpDave	Note Added: 22218
2012-11-15 21:47	pfpDave	Status	closed => feedback
2012-11-15 21:47	pfpDave	Resolution	won't fix => reopened
2012-11-15 21:50	pfpDave	Note Added: 22219
2012-11-15 21:50	pfpDave	Status	feedback => assigned
2012-11-15 22:04	c_schmitz	Assigned To	c_schmitz => tpartner
2012-11-15 22:04	c_schmitz	Note Added: 22220
2012-11-15 22:19	tpartner	Note Added: 22221
2012-11-15 22:45	c_schmitz	Relationship added	has duplicate 06758
2012-11-15 22:46	c_schmitz	Note Added: 22222
2012-11-15 23:54	tpartner	Changeset attached	=> LimeSurvey master 71ddf2eb
2012-11-15 23:54	tpartner	Note Added: 22223
2012-11-15 23:55	tpartner	Changeset attached	=> LimeSurvey 2.1 eacadb50
2012-11-15 23:55	tpartner	Note Added: 22224
2012-11-15 23:56	tpartner	Note Added: 22225
2012-11-15 23:56	tpartner	Status	assigned => resolved
2012-11-15 23:56	tpartner	Resolution	reopened => fixed
2012-11-16 10:40	c_schmitz	Note Added: 22242
2012-11-16 10:40	c_schmitz	Status	resolved => closed
2015-12-11 14:40	c_schmitz	Category	Templates => Theme editor