[Bug 122354] [sidebar] In slide sorter travers with arrow key is one off in layout style

[bu...@apache.org]

https://issues.apache.org/ooo/show_bug.cgi?id=122354

Andre <aw...@googlemail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |CONFIRMED
                 CC|                            |awf.aoo@googlemail.com
     Ever confirmed|0                           |1

--- Comment #1 from Andre <aw...@googlemail.com> ---
When the current slide is changed via keys in the slide sorter view (as opposed
to the slide sorter bar) then the slide change is not broadcast properly.  When
the current slide is changed via mouse then this works as expected.

Reason for this is lies in the way the slide change is notified:

- There are events for change in the slide sorter selection.  The time at which
the modified selection is notified differs between keyboard and mouse
processing. For keyboard events the selection is broadcast before the current
slide is changed, for mouse event the broadcast takes place after the current
slide changed.

- The current slide is also set as property at the XController.  However, the
slide sorter view has no real concept of a current slide and therefore the
"CurrentPage" property does not exist for the SdUnoSlideView sub controller. 
The SdUnoDrawView does have this property, which is the reason why the bug is
only reproducible in the slide sorter view.

This can be fixed in two ways:

1. Add the "CurrentPage" property to the SdUnoSlideView sub controller.  The
slide sorter keeps track of the current slide anyway, so that could be exposed
to the controller as well.

2. Notify the selection change of the slide sorter after the current slide has
been changed when the selection change is triggered by key events.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.