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

bug: ion-datetime mode="ios" swipe locks when swiping next/previous on desktop #24482

Closed
4 of 6 tasks
StevieApp opened this issue Dec 23, 2021 · 5 comments
Closed
4 of 6 tasks
Labels
package: core @ionic/core package type: bug a confirmed bug report

Comments

@StevieApp
Copy link

StevieApp commented Dec 23, 2021

Prerequisites

Ionic Framework Version

  • v4.x
  • v5.x
  • v6.x

Current Behavior

try date time swiping left or right and the months don't switch but it presents a new set of dates with min and max implemented . But the dates are not clickable. I'm not sure if I'm the only one facing this issue

<ion-card class="ion-no-padding">
<ion-card-content class="ion-no-padding">
<ion-datetime>
<div slot="title"> {{name | translate}} </div>
</ion-datetime>
</ion-card-content>
</ion-card>

Expected Behavior

The months should switch. or at least the date should be clickable then it can switch.

Steps to Reproduce

Just display a simple date time then display try swiping left and right

Code Reproduction URL

Ionic Info

No response

Additional Information

No response

@ionitron-bot ionitron-bot bot added the holiday triage issues that were created during holiday period label Dec 23, 2021
@ionitron-bot
Copy link

ionitron-bot bot commented Dec 23, 2021

Thanks for the issue! This issue has been labeled as holiday triage. With the winter holidays quickly approaching, much of the Ionic Team will soon be taking time off. During this time, issue triaging and PR review will be delayed until the team begins to return. After this period, we will work to ensure that all new issues are properly triaged and that new PRs are reviewed.

In the meantime, please read our Winter Holiday Triage Guide for information on how to ensure that your issue is triaged correctly.

Thank you!

@StevieApp StevieApp changed the title bug: try date time swiping left or right and the months don't switch bug: try date time swiping left or right and the months don't switch (when size = "cover") Dec 23, 2021
@StevieApp StevieApp changed the title bug: try date time swiping left or right and the months don't switch (when size = "cover") bug: try date time swiping left or right and the months don't switch (when size = "cover", ). with ngx-translate Dec 23, 2021
@StevieApp StevieApp changed the title bug: try date time swiping left or right and the months don't switch (when size = "cover", ). with ngx-translate bug: try date time swiping left or right and the months don't switch (when size = "cover", ). with ngx-translate. Dec 23, 2021
@StevieApp StevieApp changed the title bug: try date time swiping left or right and the months don't switch (when size = "cover", ). with ngx-translate. bug: try date time swiping left or right and the months don't switch (when size = "cover", ). with ngx-translate and ion-card (mode='ios') Dec 23, 2021
@sean-perkins
Copy link
Contributor

Hello @StevieApp thanks for reporting this issue.

I've created a reproduction here: https://github.com/sean-perkins/gh-24482-date-swipe. I'll update the title to best describe the issue.

The problem is that the ios approach for datetime is built around limitations with Safari's Intersection Observer implementation. We didn't really intend for developers to use the ios display mode for desktop touch devices.

The intersection observer fires at a 70% threshold, which accounts for some oddities in Safari. We can likely explore additional platform checks to only do this behavior on a mobile device.

@sean-perkins sean-perkins changed the title bug: try date time swiping left or right and the months don't switch (when size = "cover", ). with ngx-translate and ion-card (mode='ios') bug: ion-datetime mode="ios" swipe locks when swiping next/previous on desktop Jan 5, 2022
@sean-perkins sean-perkins added package: core @ionic/core package type: bug a confirmed bug report labels Jan 5, 2022
@ionitron-bot ionitron-bot bot removed the holiday triage issues that were created during holiday period label Jan 5, 2022
@StevieApp
Copy link
Author

StevieApp commented Jan 5, 2022 via email

@liamdebeasi
Copy link
Contributor

Thanks for the issue. This has been resolved via #24421, and a fix was released as part of Ionic 6.0.5.

@ionitron-bot
Copy link

ionitron-bot bot commented Mar 4, 2022

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Mar 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
package: core @ionic/core package type: bug a confirmed bug report
Projects
None yet
Development

No branches or pull requests

3 participants