Fix taking sidebar position in fullscreen mode

[ShGKme]

🐞 Resolves

• Kind of regression of fix: use left position of sidebar to set viewer width #1640

Steps to reproduce:

1. Open files
2. Open a file sidebar
3. On this moment Viewer handle sidebar state change and updates the size
4. Open a file
5. Sidebar goes full-screen mode
6. Viewer is opened (with the old size)
7. There is a gap

🖼️ Changes

Take the sidebar position in beforeOpen

Before

master.mp4

After

FIX.mp4

Details

Old description

A bug

How Viewer react on sidebar opening now in Files:

1. Viewer is open, sidebar is closed:
   
2. Click "Open Sidebar"
3. Sidebar is opened in the default layout
   
   • File info is loading
   • Viewer takes it's position to set its width
4. Loading is finished. Sidebar goes fullscreen. But viewer still use old position for width
   
5. There is a gap between Viewer and Sidebar

Exactly the same happens when Viewer is opened having Sidebar already opened.

It wasn't a problem before #1640 because incorrect width early was correct for fullscreen mode.

Expected behavior

No gap

Solution in this PR

Retake Sidebar's position when file opening (including transition animation) is finished.

Drawbacks

Because of sub-loading and 100ms transition on width, for a short time there is a gap anyway and animation if it's disappearing...

gap.mp4

Alternative solutions

Alternative solutions could be:

1. Always use Sidebar in the fullscreen mode with Viewer. Then we can just always use width.
2. Change Sidebar, make it Fullscreen immediately, not after all data is loaded
3. Add a ghost element directly in DOM inside Sidebar and use ResizeObserver
4. Partially revert fix: use left position of sidebar to set viewer width #1640 and keep Files accurate having overlap in other apps without fullscreen sidebar.

[ShGKme]

Rebased onto master + compiled in prod mode

[juliusknorr]

Just noticed this existed after working on an alternative approach at nextcloud/server#39378, but it is basically alternative 1 you considered. Would appreciate input on that one as it seems to have the same effect without the mentioned drawback from above.

[ShGKme]

PR is updated after nextcloud/server#39378

[ShGKme]

@juliushaertl @skjnldsv @artonge
Sorry for mentions :)
I'll check Cypress tests if everything looks fine by code/solution.

[blizzz]

Please don't forget to set the milestone when opening a PR 😃