Skip to content

Conversation

@luka-nextcloud
Copy link
Contributor

Signed-off-by: Luka Trovic luka@nextcloud.com

  • Resolves: #
  • Target version: master

Summary

text-align: left;
max-height: 0;
transition: max-height 0.5s cubic-bezier(0, 1, 0, 1);
z-index: 110;
Copy link
Member

Choose a reason for hiding this comment

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

The cypress test failure seems expected due to this z-index being larger than the one of the file creation menu which would cause some overlapping like this:

image

In addition it might be good to add a position: relative; here as well to make sure the z-index always applies.

Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

The cypress failure actually catched a case that where it has some visual regression 😉

@juliusknorr
Copy link
Member

@luka-nextcloud Could you check that again for the comment above?

@juliusknorr
Copy link
Member

Needs a rebase and build with latest bundles 😉

@juliusknorr
Copy link
Member

/rebase

Signed-off-by: Luka Trovic <luka@nextcloud.com>
@juliusknorr juliusknorr force-pushed the fix-text-idle-message-overlap-collabora branch from 052b4c4 to d59cc78 Compare October 21, 2021 10:14
@juliusknorr juliusknorr merged commit a470211 into master Oct 21, 2021
@juliusknorr juliusknorr deleted the fix-text-idle-message-overlap-collabora branch October 21, 2021 10:25
@juliusknorr
Copy link
Member

/backport to stable22

@juliusknorr
Copy link
Member

/backport to stable21

@juliusknorr
Copy link
Member

/backport to stable20

@backportbot-nextcloud
Copy link

The backport to stable22 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable21 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable20 failed. Please do this backport manually.

@skjnldsv skjnldsv mentioned this pull request Oct 25, 2021
25 tasks
max-nextcloud pushed a commit that referenced this pull request Jan 13, 2022
* Backports: #1860
* Target version: stable22

Signed-off-by: Luka Trovic <luka@nextcloud.com>
max-nextcloud pushed a commit that referenced this pull request Jan 13, 2022
* Backports: #1860
* Target version: stable22

Signed-off-by: Luka Trovic <luka@nextcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants