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

some elements in the sharing dropdown are not visible and no way to scroll down in certain screen resolutions #33839

Closed
Tracked by #1280
szaimen opened this issue Sep 2, 2022 · 10 comments · Fixed by nextcloud-libraries/nextcloud-vue#3445 or #35179

Comments

@szaimen
Copy link
Contributor

szaimen commented Sep 2, 2022

See
image

@szaimen
Copy link
Contributor Author

szaimen commented Sep 5, 2022

I think this should be fixed with the next vue 7.0 release as well...

@szaimen szaimen added the 2. developing Work in progress label Sep 7, 2022
@szaimen
Copy link
Contributor Author

szaimen commented Sep 14, 2022

No, isn't yet

@szaimen szaimen removed the 2. developing Work in progress label Sep 14, 2022
@thomasgg23
Copy link

thomasgg23 commented Oct 19, 2022

Version: Nextcloud Hub 3 Version 25.0.0
Browser: 106.0.1370.47 (Offizielles Build) (64-Bit) - cache and cookies cleared

Bug still exists.
Sharing of folder can't be removed on smaller screen sizes, because the button is not displayed and menu can't be scrolled down.

image

Screen resolution
image

@szaimen
Copy link
Contributor Author

szaimen commented Oct 19, 2022

cc @jancborchardt @nimishavijay @nextcloud/server-frontend

@szaimen szaimen added the high label Oct 20, 2022
@PVince81
Copy link
Member

PVince81 commented Nov 2, 2022

still happening on master, we're still on vue 7.0.0-beta.4 though

I need to zoom to 50% to see the "Note to recipient" and "+ Add another link" at the bottom of the dropdown.

@szaimen
Copy link
Contributor Author

szaimen commented Nov 8, 2022

@nextcloud/frontenders any ideas how to solve this? I guess the popper element would need som scrolling so that it doesnt get out of view?

@szaimen
Copy link
Contributor Author

szaimen commented Nov 8, 2022

found a solution

@PVince81
Copy link
Member

nextcloud-libraries/nextcloud-vue#3445 is merged

requires a nextcloud-vue release + lib update here

@PVince81 PVince81 reopened this Nov 14, 2022
@szaimen
Copy link
Contributor Author

szaimen commented Nov 15, 2022

Needs:

  • backport to vue 7
  • new release of vue 7
  • merge of new vue 7 in master
  • backport of new vue 7 to stable25

@szaimen
Copy link
Contributor Author

szaimen commented Nov 15, 2022

Reopening for the last missing step

@szaimen szaimen reopened this Nov 15, 2022
@szaimen szaimen closed this as completed Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment