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

Don't make the window disappear when the user clicks outside of this window #899

Open
waazdakka opened this issue Aug 28, 2018 · 9 comments · Fixed by #926
Open

Don't make the window disappear when the user clicks outside of this window #899

waazdakka opened this issue Aug 28, 2018 · 9 comments · Fixed by #926
Assignees
Labels
3. to review Waiting for reviews enhancement New feature request

Comments

@waazdakka
Copy link

waazdakka commented Aug 28, 2018

Steps to reproduce

  1. Click to create a new event
  2. Start to enter infos
  3. Click on the time selector (or just go to the 4.)
  4. Click somewhere before clicking on minutes selector (on the whole event window or somewhere on the page) to close the time selector
  5. Rage seeing the whole event's window disappear (full of fresh-new infos)

Expected behaviour

Don't make the event's popup disappear when the user clicks outside of this window : it may results in crisis when you just entered for the third time all informations about the event you want to create in your planning.
Maybe the event could be saved as draft... or even created

Actual behaviour

When an user click outside of the event's window, this window just disappear, while the user had already fill some fields.

Calendar version: (see the apps page)
1.6.1

Client configuration

Firefox / Chrome


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@georgehrke georgehrke added enhancement New feature request 1. to develop Accepted and waiting to be taken care of labels Nov 4, 2018
@georgehrke georgehrke added this to the 2.0.0 beta3 milestone Nov 4, 2018
@weeman1337
Copy link
Member

weeman1337 commented Dec 8, 2018

Jep - this happens for me nearly every time creating an event via the web interface. Sometimes it doesn't happen intentionally. It's more like I didn't hit the create button and clicked out of the window → everything is gone :/

I could imagine it working like this:

  • when clicking another day: keep the inputs and change the date
  • when clicking outside of the calendar, close the window
    • clicking a day again → reopen the create dialog and restore the inputs

What do you think @nextcloud/calendar @nextcloud/designers (I hope designers is right here - somehow for me the group targets usability, too)

@weeman1337 weeman1337 self-assigned this Dec 8, 2018
@weeman1337 weeman1337 added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Dec 8, 2018
@georgehrke
Copy link
Member

@weeman1337 Hey, since you seem to be looking into this issue: Please note that we are currently migrating the app to Vue.js: #926

@georgehrke
Copy link
Member

To answer your actual question ... 😉

when clicking another day: keep the inputs and change the date

I like this idea 👍

when clicking outside of the calendar, close the window

We could also just keep the dialog open and only close it when the user actually clicks the cancel button / the x or presses the escape key.

@jancborchardt
Copy link
Member

We have 2 requirements:

  • Data should not get lost unexpectedly
  • People are used to clicking the outside of dialogs to close them

So clicking outside of the dialog should close it, but also save changes. Changes only don’t get saved when you click the x button (should get a "Cancel" tooltip).

@georgehrke
Copy link
Member

So clicking outside of the dialog should close it, but also save changes. Changes only don’t get saved when you click the x button (should get a "Cancel" tooltip).

So you are saying clicking outside the dialog should have the same effect as clicking the Update / Save button?

@georgehrke
Copy link
Member

Just to be clear about the original issue: When the date-time-picker is open, clicking outside the window should only close the date-time-picker, not the window.

@jancborchardt
Copy link
Member

So you are saying clicking outside the dialog should have the same effect as clicking the Update / Save button?

Yes. Do you disagree? What’s happening in macOS Calendar?

@weeman1337
Copy link
Member

I planned to wait for

Please note that we are currently migrating the app to Vue.js: #926

But that still seems to be some time in the future.

I've looked into the code today (for hours) and couldn't figure out how the event flow is working. So I gave it up. Regarding the issue I think it is intended to keep the event data if another day is clicked, see: https://github.com/nextcloud/calendar/blob/master/js/app/service/eventseditordialogservice.js#L123

Maybe someone who is firm with the code could have a look at this.

@georgehrke georgehrke modified the milestones: 2.0.0 beta3, 2.0.0 beta1 Sep 9, 2019
@georgehrke georgehrke added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 9, 2019
@georgehrke georgehrke mentioned this issue Oct 16, 2019
30 tasks
@georgehrke georgehrke reopened this Oct 19, 2019
@georgehrke georgehrke modified the milestones: 2.0.0 beta1, 2.0.0 beta2 Oct 19, 2019
@georgehrke georgehrke modified the milestones: 2.0.0 beta2, 2.0.0 beta3 Nov 4, 2019
@georgehrke georgehrke modified the milestones: 2.0.0 beta3, 2.0.0 beta4 Dec 9, 2019
@georgehrke georgehrke modified the milestones: 2.0.0 RC, 2.0.1 Jan 15, 2020
@armaccloud
Copy link

To answer your actual question ... wink

when clicking another day: keep the inputs and change the date

I like this idea +1

when clicking outside of the calendar, close the window

We could also just keep the dialog open and only close it when the user actually clicks the cancel button / the x or presses the escape key.

Since the last update, the escape key isn't working to close the window anymore. Will this come back the the resolution of this issue?

@georgehrke georgehrke modified the milestones: 2.0.2, 2.0.3 Feb 27, 2020
@georgehrke georgehrke modified the milestones: 2.0.3, 2.1.0 Apr 6, 2020
@georgehrke georgehrke modified the milestones: 2.1.0, 2.2.0 Sep 4, 2020
@ChristophWurst ChristophWurst modified the milestones: 2.2.0, 2.3.0 Mar 24, 2021
@ChristophWurst ChristophWurst modified the milestones: v2.3.0, v2.4.0 Jun 24, 2021
@ChristophWurst ChristophWurst modified the milestones: v2.4.0, v2.4.1 Nov 25, 2021
@tcitworld tcitworld removed this from the v2.4.1 milestone Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants