[TOBAGO-229] AJAX-reload of scrollable Sheets resets scrollbar to top position

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0.10
• Assignee: Volker Weber
• Reporter: Dennis Hauser
• Created: 04/Jan/07 10:13
• Updated: 11/Mar/07 10:02
• Resolved: 14/Jan/07 12:45

Description

A workaround could be to store the scrollbar position before reloading and to set it to the original after the reload is completed ( see attached code fragment exchange-energy-trade_win.js)

This does work theoretically, but the marked lines in the second code fragment (tobago-sheet.js) are resetting the sheet completely again.

-------------
exchange-energy-trade_win.js

function eetReloadOverviewWin() {

// -------- neu von hier--------
var dataDiv = Tobago.element("marketPage:overviewSheet_data_div");
var scrollTop = dataDiv.scrollTop;
var scrollLeft = dataDiv.scrollLeft;
// -------- bis hier --------

var options = {
insertion: function(receiver, response) {

if (response.substring(0, 20) == "<status code=\"304\"/>")

{ // no update needed, do nothing receiver.skipUpdate = true; }

else if (response.substring(0, 11) == "<input id=\"")

{ // update content Element.update(receiver, response); // -------- neu von hier-------- var dataDiv2 = Tobago.element("marketPage:overviewSheet_data_div"); dataDiv2.scrollTop = scrollTop; dataDiv2.scrollLeft = scrollLeft; // -------- bis hier -------- }

else {
// unknown response do full page reload
LOG.debug("initiating full reload");
var mos = Tobago.element("marketPage:overviewSheet");
if (mos != null)

{ Tobago.submitAction("marketPage:overviewSheet"); }

}
if (eetOverviewTimeoutWin)

{ clearTimeout(eetOverviewTimeoutWin); }

eetOverviewTimeoutWin = setTimeout(eetReloadOverviewWin, eetGetConfigWin().overviewSheetReloadTimeout);
},
createOverlay: false
};
Tobago.reloadComponent("marketPage:overviewSheet", "marketPage:overviewSheet", options);
}

-------------
resources/html/standard/standard/script/tobago-sheet.js

Tobago.Sheet.prototype.setup = function()

{ ... // Problematisches Codefragment, dessen Sinn mir sich nicht erschließt, // dessen Auskommentierung jedoch das Problem löst. :-) // -------- von hier -------- // ToDo: find a better way to fix this problem // IE needs this in case of ajax loading of style classes var outerDiv = Tobago.element(this.outerDivId); outerDiv.className = outerDiv.className; outerDiv.innerHTML = outerDiv.innerHTML; // -------- bis hier -------- ... }

-------------