Workspaces size doesn't update as you change the options (easy)

[jstressman]

If you change the number of columns and rows for Workspaces, it doesn't update the size of the Workspaces window to correctly fit the workspaces in there in relation to their aspect ratio. So you end up with a bunch of squished workspaces until you actually click on the Workspaces window to resize it... then it suddenly snaps to the proper dimensions.

Shouldn't this update as you make the settings changes? Perhaps not exceeding the starting width x height dimensions so as not to end up outside the current desktop borders?

[jstressman]

This is in hrev43809 by the way.

[axeld]

This could certainly be done, best based on the current aspect ratio. It could simply move the window to prevent it from leaving the screen.

[devine]

I will a attach a patch for this. It is depended on the changes made in the patch #8188 though.

[devine]

Updates the size (by calling Zoom()) when the workspace layout is updated. Depends on patch in #8188

[devine]

Updates the size (by calling Zoom()) when the workspace layout is updated. Depends on patch in #8188

[leavengood]

Taking ownership.

[pulkomandy]

Several comments on the patch (which proabably does not apply anymore to the current codebase).

It does not feel right to have the notification sent from the screen preferences. There could be other places able to control the workspace count. Instead, the message should be sent from app_server when the workspaces are actually created or deleted.

Handling of the window position in this case will be tricky.

• If the window was at the screen edge for use with auto-raise, it should snap to said edge and change its size so the workspaces size in the preview stays the same.
• If it is not at any particular screen edge, Zoom will move it there if the auto-raise feature is enabled. Otherwise, it should only make sure that the window fits into the screen.