Sidebar and Navigator menu entries not checked when the features are activated

[cosminh11]

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

1. Go to 'View'
2. See that the 'Sidebar' entry has no check mark
3. Click on 'Sidebar'
4. Open the 'View' menu again
5. See that the 'Sidebar' still has no check mark

With the tabbed view

1. Go to 'View'
2. See that 'Sidebar' is not selected
3. Click on 'Sidebar'
4. See that 'Sidebar' is still not selected

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

• Collabora version: 24.04.7.2
• OS and version: Ubuntu 22.04.5 LTS
• Browser and version: Firefox 130.0.1

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:

• Formatting Marks:
  • request: uno .uno:ControlCodes
  • response: statechanged: .uno:ControlCodes=true
• Sidebar:
  • request: uno .uno:SidebarDeck.PropertyDeck
  • response: statechanged: { "locale": "en-US", "commandName": ".uno:Sidebar", "state": "true" }
• Navigator:
  • request: uno .uno:Navigator
  • response: statechanged: { "locale": "en-US", "commandName": ".uno:SidebarDeck.NavigatorDeck", "state": "true" }

[cosminh11]

I tried to fix this in the frontend by mapping those different request and response values. I have a patch that mostly works. I would open a pull-request (should I?) but there's a problem: sometimes when you open the navigator, the sidebar button gets selected too. That's because we also get a "statechanged" event for it as well (steps to reproduce, below). So I'm not sure: do we want to do this in the frontend (and try to find a solution for this "both-are-selected" issue)? Or should we use different commands, maybe try to send the commands we're receiving (i.e. send .uno:Sidebar instead of .uno:SidebarDeck.PropertyDeck). I would need some advice.

Steps to reproduce the remaining "both-are-selected" issue:

• both sidebar and navigator are off
• click sidebar
  • the following are received:
  • statechanged: .uno:Sidebar=true
  • statechanged: { "locale": "en-US", "commandName": ".uno:SidebarDeck.NavigatorDeck", "state": "true" }
  • statechanged: { "locale": "en-US", "commandName": ".uno:SidebarDeck.NavigatorDeck", "state": "false" }
  • statechanged: { "locale": "en-US", "commandName": ".uno:Sidebar", "state": "true" }
  • sidebar becomes visible
  • sidebar button is selected
  • navigator button is not selected
• click navigator
  • the following are received:
  • statechanged: { "locale": "en-US", "commandName": ".uno:Sidebar", "state": "false" }
  • statechanged: { "locale": "en-US", "commandName": ".uno:SidebarDeck.NavigatorDeck", "state": "true" }
  • navigator becomes visible
  • sidebar button is not selected
  • navigator button is selected
• click navigator
  • the following are received:
  • statechanged: { "commandName": ".uno:SidebarDeck.NavigatorDeck", "state": "false" }
  • statechanged: .uno:Sidebar=false
  • neither sidebar nor navigator are visible
  • neither button is selected
• click navigator
  • the following are received:
  • statechanged: .uno:Sidebar=true
  • statechanged: { "locale": "en-US", "commandName": ".uno:SidebarDeck.NavigatorDeck", "state": "true" }
  • navigator becomes visible
  • both buttons are selected (expected: only the navigator button should be)
  • at this point, if you switch to compact view, then only "navigator" is checked, as expected; switching back to tabbed view shows only "navigator" as selected; however all this is done while 8 more "statechanged" messages are received