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

Cross deletion of 'expiration date for user' shows the calendar using Safari as browser #36735

Closed
davitol opened this issue Jan 9, 2020 · 19 comments
Assignees
Labels
Browser:Firefox Browser:Safari p2-high Escalation, on top of current planning, release blocker Type:Bug
Milestone

Comments

@davitol
Copy link
Contributor

davitol commented Jan 9, 2020

Steps to reproduce

  1. Create a file/folder
  2. Share the file/folder with a user
  3. Set an expiration date for user (behavior included in PR [For 10.4] Include expiration option for user and group shares #36573)
  4. Delete the date clicking on the cross

Expected behaviour

Calendar should not be shown

Actual behaviour

Calendar is shown even when finally the expiration date is deleted

safaridate

Server configuration

ownCloud version: 10.4.0 RC1

Browser version

Safari Version 13.0.4 (13608.4.9.1.4)

@felixheidecke @phil-davis JFYI

@phil-davis
Copy link
Contributor

On Chrome 79 and Firefox 72 on Ubuntu 18.04:

  1. a) have an expiration date already entered
    b) click the "x" to remove the expiration date
    c) the calendar coses (good)
    d) the whole share settings "panel" closes
    e) (and the expiration date has been removed, as expected).

  2. a) have no expiration date entered
    b) click the "x" to remove the expiration date
    c) nothing happens (correct expected behaviour)

Item 1 (d) is the weird thing - you have to then open the share settings again to enter a new expiration date. But that is not a really bad thing, just a bit unexpected.

But I can't reproduce the behaviour reported on Safari.

@davitol
Copy link
Contributor Author

davitol commented Jan 10, 2020

But I can't reproduce the behaviour reported on Safari.

Yes. I had also tried in Chrome and Firefox and everything went fine. That's why I set Safari label to the ticket

@micbar
Copy link
Contributor

micbar commented Jan 28, 2020

@davitol I can confirm the behavior on Safari

And I found a new problem on both Firefox and Safari
Expiration-Bug

The dropdown is closing when you are hovering over it.

@micbar
Copy link
Contributor

micbar commented Jan 28, 2020

@felixheidecke @mrow4a @VicDeo
Who could take care?
Thisd is blocking 10.4

@micbar micbar added Browser:Firefox p2-high Escalation, on top of current planning, release blocker labels Jan 28, 2020
@phil-davis
Copy link
Contributor

phil-davis commented Jan 28, 2020

For me, the sharing date/permissions dropdown sometimes quickly opens and closes when I open the sharing panel of a folder. But other times it comes up open and stays that way, and other times it comes up closed and stays that way.

My observations are on Chrome 79 on Ubuntu 18.04

@phil-davis
Copy link
Contributor

Next clue - for a share with a user I see the "quick open and close" behaviour, and it ends up with the dropdown closed.

  • for a share with a group, the dropdown is open

@mrow4a
Copy link
Contributor

mrow4a commented Jan 28, 2020

@phil-davis @micbar is this still present after the expiry fix in #36766 ?

@phil-davis
Copy link
Contributor

Yes, I am trying with current core master.

I did think that the hover thing happened to me twice, but I am struggling to reproduce it.

However there does seem to be a clear difference between user and group shares - so looking for differences in the JS of those might be a good place to start.

@phil-davis
Copy link
Contributor

phil-davis commented Jan 28, 2020

For groups, it starts open. When I hover over the contents of the settings (expire date field, permissions checkboxes) then nothing happens. When I hover over the actual group name (where it says e.g. "Finance (group)") then the settings auto-closes.

@VicDeo
Copy link
Member

VicDeo commented Jan 28, 2020

@micbar For me after clicking the share icon

Chrome
Share details view loaded with expanded share options panel.
After loading options are hidden

Firefox
Share details view loaded with expanded share options panel.
Options are hidden on hover

UPD
I run make clean && make
Now both FF and Chrome load share options panel in the expanded state and hide it after loading is complete

@davitol
Copy link
Contributor Author

davitol commented Jan 28, 2020

And I found a new problem on both Firefox and Safari

@micbar It looks like a regression produced by #36766

I cannot reproduce the behaviour The dropdown is closing when you are hovering over it in oC 10.4.0. RC1 but it happens applying the PR #36766

@VicDeo

This comment has been minimized.

@phil-davis
Copy link
Contributor

Note: release-10.4.0 branch is old and does not contain the latest code.
We are working from master, and some day when master is "ready" it will become "the new 10.4.0"

@mrow4a
Copy link
Contributor

mrow4a commented Jan 28, 2020

@phil-davis make sure also to run in incognito mode. You might have a js cached in browser that causes problems. @micbar please add me in relevant reviews to fix it. I can help also investigating, will have a look later.

@VicDeo
Copy link
Member

VicDeo commented Jan 28, 2020

Checked with a clean master install. Both FF and Chrome display share options and then hide this block

@micbar
Copy link
Contributor

micbar commented Jan 28, 2020

@VicDeo Can confirm this too on Safari.

Safar is in addition showing the second bug if you click on the cross, the calendar pops up.

@phil-davis
Copy link
Contributor

With chrome 79 incognito.

For a share with a user: opening the sharing panel, the detailed expiration/permissions block opens and quickly closes.

For a share with a group: opening the sharing panel, the detailed expiration/permissions block opens and stays open. Hover over the group name, the detailed expiration/permissions block closes itself.

output

@mrow4a
Copy link
Contributor

mrow4a commented Jan 28, 2020

Fixes for both problems in #36847. Please make sure while testing to logout and open new tab to clear css/js caches.

@micbar
Copy link
Contributor

micbar commented Feb 20, 2020

closing

@micbar micbar closed this as completed Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Browser:Firefox Browser:Safari p2-high Escalation, on top of current planning, release blocker Type:Bug
Projects
None yet
Development

No branches or pull requests

5 participants