-
Notifications
You must be signed in to change notification settings - Fork 692
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sidebar and Navigator menu entries not checked when the features are activated #10083
Comments
I tried to fix this in the frontend by mapping those different request and response values. I have a patch that mostly works. Steps to reproduce the remaining "both-are-selected" issue:
|
Same issue somne of the other buttons on the View tab/menu:
These following buttons have are check in the menu in Compact view but not in Tabbed view
COOLWSD version: 24.04.7.2(git hash: d5ebff5 (E)) |
Describe the Bug
When you activate or deactivate the sidebar or the navigator, either from the menu or from the notebook bar, neither do the menu entries get checked or unchecked, nor do the buttons get selected or deselected.
Other features, such as "formatting marks" do show the current state, so for consistency I would expect the sidebar and the navigator to do the same.
Steps to Reproduce
With the compact view
With the tabbed view
Expected Behavior
The 'Sidebar' and 'Navigator' menu entries and notebook bar buttons appear as checked and selected, respectively, depending on whether their corresponding feature is shown or hidden.
Actual Behavior
The 'Sidebar' and 'Navigator' menu entries and notebook bar buttons never appear as checked / selected.
Screenshots
Desktop
Additional Context
The status of these buttons and menu entries seems to be determined by the UNO commands that are sent and received.
The difference between these two and 'Formatting Marks' (which works as expected) is that in the case of 'Formatting Marks' the command name is the same both for the request and the response ('.uno:ControlCodes'), while for Sidebar and Navigator they are different:
uno .uno:ControlCodes
statechanged: .uno:ControlCodes=true
uno .uno:SidebarDeck.PropertyDeck
statechanged: { "locale": "en-US", "commandName": ".uno:Sidebar", "state": "true" }
uno .uno:Navigator
statechanged: { "locale": "en-US", "commandName": ".uno:SidebarDeck.NavigatorDeck", "state": "true" }
The text was updated successfully, but these errors were encountered: