diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2024-03-21 17:01:20 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2024-03-21 21:30:17 +0100 |
commit | d8061151acf4e287b60a055a67b84c56989a37af (patch) | |
tree | 049db2a79623890ca9c069e7983c3a263e16464f /drawinglayer/source/primitive2d/Tools.cxx | |
parent | a91bb0fe19a24c0473937af82a352901a08e2933 (diff) |
cool#8278 sfx2 lok: fix lost hide notification on sidebar switch
Similar to commit 55feb670ca28e0a48ac82a65b5559598704d993e (cool#8278
sfx2 lok: fix unexpected non-json sidebar status update, 2024-03-21),
the trouble here was not around hiding the sidebar but around switching
between decks.
This went wrong in commit aaf6ce108e91b1504befe19afcee471e3316ae7a
(cool#7492 sfx2 lok: set language/locale on async sidebar update,
2024-01-11), where I didn't notice that
SidebarController::SwitchToDeck() may emit two callbacks, so the effort
to avoid code duplication went a bit too far: the hide+show case only
emitted a show callback.
Fix this by building a list of state changes to emit, so once se switch
decks, not only the new one is "on", but also the old one is "off".
Change-Id: I1678ee61e004697a6a5c6ecaf40a18b2d1d47e61
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165108
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'drawinglayer/source/primitive2d/Tools.cxx')
0 files changed, 0 insertions, 0 deletions