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

Scroller scrolls up automatically while trying to select an expiry date for public link #7513

Closed
sushmita56 opened this issue Aug 24, 2022 · 5 comments · Fixed by #8909
Closed
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@sushmita56
Copy link
Contributor

sushmita56 commented Aug 24, 2022

Steps to reproduce

  1. First of all reduce the screen resolution
  2. Log in to the WebUI and create a resource
  3. Then, create a public link for the resource
  4. And click on the public link edit button and then click on the Add expiration date option
  5. Leave year and month as it is and now try to set the expiration date to the very last date of the month like 28/29/30/31

To select the last date of the month you'll have to scroll down. Whenever you scroll down and are about to click on the date, say 30, the scroller scrolls up automatically.

Discovered such behavior while working on the issue: #7320

Actual behaviour

The scroller moves up instantly as we hover over the date.

Expected behaviour

Unless we select the date the scroller was not assumed to shift upward.

Screenshots

web_calendar.mp4

Environment general

Operating system: Ubuntu

Backend (ownCloud Core or Infinite Scale): Core

Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...

@sushmita56 sushmita56 added the Type:Bug Something isn't working label Aug 24, 2022
@JammingBen
Copy link
Contributor

Phew, at a first glance, the v-calendar lib which is used here does not provide any options to prevent this from happening. One easy solution would be to place the calendar always at the top. What do you think @tbsbdr ?

image

@SwikritiT
Copy link
Contributor

@grgprarup
Copy link
Contributor

@SwikritiT
Copy link
Contributor

This test failure is also because of this issue #8902

@SwikritiT
Copy link
Contributor

Unskip the test skipped in #8905 after this bug is fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants