View Issue Details

This bug affects 1 person(s).
 16
IDProjectCategoryView StatusLast Update
06885Bug reportsTheme editorpublic2012-11-16 10:40
ReporterpfpDave Assigned Totpartner  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.00+ 
Summary06885: 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
  1. Create a long page survey
  2. assign default or Limespired templates (or any others that use the standard survey_runtime.js include)
  3. Switch on IE 8 compatibility view
    load the page.
TagsNo tags attached.
Bug heat16
Complete LimeSurvey version number (& build)121115
I will donate to the project if issue is resolvedNo
BrowserIE8
Database type & versionSQL Express 2012
Server OS (if known)Server 2008
Webserver software & version (if known)IIS 7
PHP Version5.4.8

Relationships

has duplicate 06758 closedc_schmitz Survey Screnn not opens on top of page on IE8 

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2012-11-15 21:44

administrator   ~22217

Don't switch on compatibility mode.

pfpDave

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

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

c_schmitz

2012-11-15 22:04

administrator   ~22220

Tony, any ideas?

tpartner

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

c_schmitz

2012-11-15 22:46

administrator   ~22222

Tony, there is an example survey attach to the related issue.

tpartner

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

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

tpartner

2012-11-15 23:56

partner   ~22225

Fixed in master and 2.1.

c_schmitz

c_schmitz

2012-11-16 10:40

administrator   ~22242

New version 2.00+ Build 121116 released

Related Changesets

LimeSurvey: master 71ddf2eb

2012-11-15 23:53

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 23:54

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

Issue History

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