feat: Add office switcher with feature comparison

[juliusknorr]

Implements the office switcher from #7198

I'm unable to test with the actual containers, only managed to get the master container itself running locally. @szaimen Can you help with that?

Patch for myself to remember how to develop locally

[marcoambrosini]

Very nice @juliusknorr. I actually like the styling of the cards here better. would it be possible to apply it also to nextcloud/server#57555 ?

[szaimen]

Hey @juliusknorr , thanks a lot for the PR! ☺️

I'm unable to test with the actual containers, only managed to get the master container itself running locally. @szaimen Can you help with that?

Usually I would recommend to follow https://github.com/nextcloud/all-in-one/blob/main/develop.md#how-to-locally-build-and-test-changes-to-mastercontainer. You might be able to also combine this with #6817.

[szaimen]

BTW @juliusknorr can you please fix the Playwright test?

[marcoambrosini]

a few more comments from a layout perspective. I think we should:

• Reduce the vertical spacing between the list items?
• Make all cards the same height
• In general it would be nice to widen the main container that wraps all the AIO interface a bit. @szaimen what do you think?

[szaimen]

• In general it would be nice to widen the main container that wraps all the AIO interface a bit. @szaimen what do you think?

yeah would be possible to widen it a bit. In general we limited the width for best readability

[marcoambrosini]

Just tested with the --max-width variable set to 580px and it looks pretty good. We're still below 80 characters per line and I think it looks a bit better. These cards would also benefit from it.

[szaimen]

I just fixed some details and tested this locally but this still needs some css fixes for when the elements are disabled to correctly look like they are disabled and not clickable (they should be disabled when the containers are running).

@juliusknorr can you please take care of this? Thanks in advance :)

[marcoambrosini]

@juliusknorr in the last screenshot from @szaimen, the disable button that should be underneath the cards is not visible anymore, was it decided to remove that?

[szaimen]

@juliusknorr in the last screenshot from @szaimen, the disable button that should be underneath the cards is not visible anymore, was it decided to remove that?

I removed it in the case the containers are running because it does not make sense to show it while you cannot change your office selection. It gets shown again after you stop the containers

[emberfiend]

just a simple 50% opacity :)

[emberfiend]

looks good!

[szaimen]

LGTM now :)

[szaimen]

Thanks everyone ☺️

[szaimen]

This is now released with v12.6.0 Beta. Testing and feedback is welcome! See https://github.com/nextcloud/all-in-one#how-to-switch-the-channel