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 mobile version of calendar close date field on 2nd button click #1114

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

lemondevxyz
Copy link
Contributor

Fixes #1084

In the Calendar app on mobile, if you click on the calendar date button it shows a date input field and that input field disappears if you set a date. If you don't, the input field stays unless you change the date by clicking on the calendar date button and change the current date.

This fix makes it so that when the calendar date button is clicked and the input field is visible, it makes the input field invisible.

Here's a mobile demo and a desktop demo:
mobile demo and desktop demo.

In the Calendar app on mobile, if you click on the calendar date button
it shows a date input field and that input field disappears if you set
a date. If you don't, the input field stays unless you change the date
by clicking on the calendar date button and change the current date.

This fix makes it so that when the calendar date button is clicked and
the input field is visible, it makes the input field invisible.
@ghost ghost added Calendar Related to CryptPad internal calendar app UI/UX Interfaces & user experience labels Jul 7, 2023
@ghost ghost requested a review from yflory July 7, 2023 07:05
@ghost ghost changed the base branch from main to staging July 7, 2023 07:05
@mathilde-cryptpad
Copy link
Contributor

Hello @cryptpad/maintainers,

Hasn't this been already fixed recently?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Calendar Related to CryptPad internal calendar app UI/UX Interfaces & user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Date selector is hard to close on mobile
2 participants