Skip to content

Commit

Permalink
Fixed issue #06885: Long survey pages jump down the page in IE8 compa…
Browse files Browse the repository at this point in the history 
…tibility view

Dev Forced scroll to top when in IE compatibility mode
  • Loading branch information
@tpartner
tpartner committed Nov 15, 2012
1 parent 55a155b commit 71ddf2e
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions scripts/survey_runtime.js
View file
Expand Up @@ -99,6 +99,7 @@ $(document).ready(function()
function navbuttonsJqueryUi(){
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();
$(this).scrollTop(0);
}

$('[dir!="rtl"] #moveprevbtn').button({
Expand Down

3 comments on commit 71ddf2e

@c-schmitz
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix! :-)

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something i don't understand with this bug : why it work with citronade but no with other template ?
(see http://bugs.limesurvey.org/view.php?id=6758 ) .

First, i think it's focusfirst function, but tested in safari (not same focus function than FF) and have no problem with focusFirst.

Denis
( it's just by curiosity, not important ).

Oups, understand : citronade replace default navbuttonsJqueryUi by "nothing" ;) .

@tpartner
Copy link
Collaborator Author

@tpartner tpartner commented on 71ddf2e Nov 16, 2012 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.