Issue 109640 - Scrollbar position not restored after re-opening doc or OOo restart
Summary: Scrollbar position not restored after re-opening doc or OOo restart
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: ui (show other issues)
Version: OOO320m12
Hardware: All Unix, all
: P3 Trivial (vote)
Target Milestone: 3.4.0
Assignee: thorsten.martens
QA Contact: issues@framework
URL:
Keywords: regression
: 111104 (view as issue list)
Depends on:
Blocks:
 
Reported: 2010-02-26 15:14 UTC by Marcus
Modified: 2017-05-20 10:30 UTC (History)
5 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
Scrollbar not restored after reopening doc / restarting OOo (6.90 KB, application/vnd.oasis.opendocument.spreadsheet)
2010-02-26 15:16 UTC, Marcus
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description Marcus 2010-02-26 15:14:31 UTC
When moving the position of the horizontal scrollbar (right to the sheet tab
pages, not freeze or split), it is not restored when the doc is saved and opened
again resp. after OOo was restarted. It's always the default position.

This was working fine in OOo 3.1.1, so looks like a regression.

I can reproduce this on Linux 64-bit (Fedora 9 with Gnome) and Solaris Sparc
(SunRay) but not on Windows (OOO320m12 Build 9483).
Comment 1 Marcus 2010-02-26 15:16:28 UTC
Created attachment 68042 [details]
Scrollbar not restored after reopening doc / restarting OOo
Comment 2 Marcus 2010-02-26 15:17:47 UTC
set keyword
Comment 3 Marcus 2010-04-12 12:21:07 UTC
please try to fix for 3.3
Comment 4 niklas.nebel 2010-04-12 13:14:32 UTC
I can't reproduce this...
Comment 5 niklas.nebel 2010-04-28 13:18:23 UTC
This happens when the new window is opened in maximized state and has view
settings that don't fit into the non-maximized size. In 3.2, the window is
temporarily resized to the smaller size, and the view settings are adjusted to
that size. It affects not only the size of the TabBar, but also things like
split views (issue 111104).
Comment 6 niklas.nebel 2010-04-28 13:19:31 UTC
*** Issue 111104 has been marked as a duplicate of this issue. ***
Comment 7 philipp.lohmann 2010-05-06 16:20:11 UTC
This originates in a duplicate SetWIndowState, the second one happens when the
window is already visible and causes the issue.
Comment 8 philipp.lohmann 2010-05-06 16:47:01 UTC
@cd: thanks for taking over
Comment 9 carsten.driesner 2010-05-06 17:00:14 UTC
cd: The following stack traces show how SetWindowState is called.

1. Call
-------
#0  SystemWindow::SetWindowState (this=0xef7e572c, rStr=...)
    at /export/home/pl/vcl111/ooo/vcl/source/window/syswin.cxx:972
#1  0xefdf1ba1 in framework::LoadEnv::impl_applyPersistentWindowState (
    this=0xef809d94, xWindow=...)
    at /export/home/pl/vcl111/ooo/framework/source/loadenv/loadenv.cxx:1897
#2  0xefdf6d99 in framework::LoadEnv::impl_loadContent (this=0xef809d94)
    at /export/home/pl/vcl111/ooo/framework/source/loadenv/loadenv.cxx:1148
#3  0xefdf8087 in framework::LoadEnv::startLoading (this=0xef809d94)
    at /export/home/pl/vcl111/ooo/framework/source/loadenv/loadenv.cxx:436
#4  0xefdefcd2 in framework::LoadDispatcher::impl_dispatch (this=0xef809d48, 
    rURL=..., lArguments=..., xListener=...)
    at /export/home/pl/vcl111/ooo/framework/source/dispatch/loaddispatcher.cxx:169
#5  0xefdf006f in framework::LoadDispatcher::dispatchWithReturnValue (
    this=0xef809d48, rURL=..., lArguments=...)
    at /export/home/pl/vcl111/ooo/framework/source/dispatch/loaddispatcher.cxx:106
#6  0xf7cb1527 in
comphelper::SynchronousDispatch::dispatch(com::sun::star::uno::Reference<com::sun::star::uno::XInterface>
const&, rtl::OUString const&, rtl::OUString const&, long,
com::sun::star::uno::Sequence<com::sun::star::beans::Prope

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

2. Call
-------
#0  SystemWindow::SetWindowState (this=0xef7e572c, rStr=...)
    at /export/home/pl/vcl111/ooo/vcl/source/window/syswin.cxx:972
#1  0xefe4b675 in
framework::PersistentWindowState::implst_setWindowStateOnWindow (xWindow=...,
sWindowState=...)
    at
/export/home/pl/vcl111/ooo/framework/source/helper/persistentwindowstate.cxx:346
#2  0xefe4c162 in framework::PersistentWindowState::frameAction (
    this=0xef10c760, aEvent=...)
    at
/export/home/pl/vcl111/ooo/framework/source/helper/persistentwindowstate.cxx:166
#3  0xefd6d1a0 in framework::Frame::implts_sendFrameActionEvent (
    this=0xf0f06550, aAction=@0xffffb2cc)
    at /export/home/pl/vcl111/ooo/framework/source/services/frame.cxx:2792
#4  0xefd70af7 in framework::Frame::setComponent (this=0xf0f06550, 
    xComponentWindow=..., xController=...)
    at /export/home/pl/vcl111/ooo/framework/source/services/frame.cxx:1473
#5  0xf78b0c89 in ?? ()
   from
/export/home/pl/vcl111/oracle-openoffice-3.2-linux-en-US/oracle_open_office3/program/../basis-link/program/libsfxli.so
#6  0xf78b5a96 in ?? ()
   from
/export/home/pl/vcl111/oracle-openoffice-3.2-linux-en-US/oracle_open_office3/program/../basis-link/program/libsfxli.so
#7  0xefdf7206 in framework::LoadEnv::impl_loadContent (this=0xef809d94)
    at /export/home/pl/vcl111/ooo/framework/source/loadenv/loadenv.cxx:1215
#8  0xefdf8087 in framework::LoadEnv::startLoading (this=0xef809d94)
    at /export/home/pl/vcl111/ooo/framework/source/loadenv/loadenv.cxx:436
#9  0xefdefcd2 in framework::LoadDispatcher::impl_dispatch (this=0xef809d48, 
    rURL=..., lArguments=..., xListener=...)
    at /export/home/pl/vcl111/ooo/framework/source/dispatch/loaddispatcher.cxx:169
#10 0xefdf006f in framework::LoadDispatcher::dispatchWithReturnValue (
    this=0xef809d48, rURL=..., lArguments=...)
    at /export/home/pl/vcl111/ooo/framework/source/dispatch/loaddispatcher.cxx:106
#11 0xf7cb1527 in
comphelper::SynchronousDispatch::dispatch(com::sun::star::uno::Reference<com::sun::star::uno::XInterface>
const&, rtl::OUString const&, rtl::OUString const&, long,
com::sun::star::uno::Sequence<com::sun::star::beans::PropertyValue> const&) ()
   from
/export/home/pl/vcl111/oracle-openoffice-3.2-linux-en-US/oracle_open_office3/program/../basis-link/program/libcomphelp4gcc3.so
#

cd: Accepted.
Comment 10 carsten.driesner 2010-08-25 09:30:43 UTC
cd: I discussed a possible fix with the framework team. We concluded that the
fix has too many regression potential. Therefore I will set this issue to OOo 3.4.

If you believe that this issue should be handled as a showstopper for OOo3.3
please nominate it accordingly in the releases mailing list.
Comment 11 Marcus 2010-08-25 09:49:58 UTC
thanks for you investigations, of course I would like to see this issue fixed in
3.3. ;-)

however, IMHO it's far away from a potential showstopper, so I'm fine with
target 3.4
Comment 12 carsten.driesner 2011-03-11 14:51:19 UTC
cd: Fixed.
Comment 13 carsten.driesner 2011-03-11 14:59:36 UTC
cd: Set pl on CC.

cd->pl: Please review fix. See: http://hg.services.openoffice.org/cws/fwk167/rev/4bec1f6543df
Comment 14 philipp.lohmann 2011-03-11 15:02:19 UTC
fix reviewed
Comment 15 carsten.driesner 2011-03-23 12:26:25 UTC
cd->tm: Please send this issue to the right QA engineer for verification.
Comment 16 thorsten.martens 2011-03-28 10:34:01 UTC
Checked and verified in cws fwk167 -> OK !