Skip to content

Conversation

@backportbot-nextcloud
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented May 4, 2023

  • ⚠️ This backport's changes differ from the original and might be incomplete

backport of #4117

This broke the viewer resize and is not needed anymore.

In order to confirm this is not needed
i replaced the `beforeMount` handler with a `console.info` call
opened a text in the viewer and resized the window a few times.
The provided info only showed up once.
So the `ViewerComponent` was only mounted once as well.

Fixes #4099.

Signed-off-by: Max <[email protected]>
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 25.0.7 milestone May 4, 2023
@max-nextcloud
Copy link
Collaborator

/compile

Signed-off-by: nextcloud-command <[email protected]>
@max-nextcloud
Copy link
Collaborator

confirmed locally that this does not lead to reloading of the viewer component.

@cypress
Copy link

cypress bot commented May 4, 2023

Passing run #9655 ↗︎

0 100 0 0 Flakiness 0

Details:

Merge 0193b4b into 88a8c1b...
Project: Text Commit: 30c4f031a0 ℹ️
Status: Passed Duration: 06:20 💡
Started: May 4, 2023 10:14 AM Ended: May 4, 2023 10:20 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@juliusknorr juliusknorr merged commit 1495958 into stable25 May 4, 2023
@delete-merged-branch delete-merged-branch bot deleted the backport/4117/stable25 branch May 4, 2023 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants