Skip to content

Conversation

@pulsejet
Copy link
Member

Fixes a regression caused by the header being variable height: see #40958 (comment)

cc @juliushaertl

@pulsejet pulsejet added 3. to review Waiting for reviews files2vue labels Oct 20, 2023
@pulsejet pulsejet added this to the Nextcloud 28 milestone Oct 20, 2023
@pulsejet pulsejet requested review from skjnldsv and susnux October 20, 2023 16:06
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AH, yes, makes more sense indeed :)

@juliusknorr
Copy link
Member

/compile

@juliusknorr juliusknorr added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 20, 2023
@juliusknorr
Copy link
Member

Thanks for the quick fix 💙

Signed-off-by: nextcloud-command <[email protected]>
@susnux susnux merged commit 14bc7d3 into master Oct 20, 2023
@susnux susnux deleted the fix-files-list-header branch October 20, 2023 18:24
@juliusknorr
Copy link
Member

juliusknorr commented Oct 24, 2023

Sorry to bug again, unfortunately this still seems to cause issues with the popover menu that text might show which is cut off and no longer overlaying the content.

Screenshot 2023-10-24 at 15 34 03

Any chance you could have another look at that as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants