Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make collabora on top of text idle message #1860

Merged
merged 1 commit into from
Oct 21, 2021

Conversation

luka-nextcloud
Copy link
Contributor

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

  • Resolves: #
  • Target version: master

Summary

@@ -175,6 +175,7 @@ export default {
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

@juliushaertl juliushaertl 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 😉

@juliushaertl
Copy link
Member

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

@juliushaertl
Copy link
Member

Needs a rebase and build with latest bundles 😉

@juliushaertl
Copy link
Member

/rebase

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

/backport to stable22

@juliushaertl
Copy link
Member

/backport to stable21

@juliushaertl
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