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

p-calendar: Doesn't close on date selection in range mode #14558

Closed
lm-swaroop opened this issue Jan 16, 2024 · 5 comments
Closed

p-calendar: Doesn't close on date selection in range mode #14558

lm-swaroop opened this issue Jan 16, 2024 · 5 comments

Comments

@lm-swaroop
Copy link

Describe the bug

hideOnDateTimeSelect doesn't work when selectionMode="range"

Environment

Chrome Version 120.0.6099.109
Ubuntu 22.04

Reproducer

https://stackblitz.com/edit/cte5uw?file=src%2Fapp%2Fdemo%2Fcalendar-range-demo.html

Angular version

17.0.0

PrimeNG version

17.1.0

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

20.10.0

Browser(s)

Chrome 120

Steps to reproduce the behavior

  1. Select start date
  2. Select end date

Expected behavior

Datepicker overlay should close

@lm-swaroop lm-swaroop added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Jan 16, 2024
@mertsincan
Copy link
Member

Thanks a lot for your report! We'll check and get back to you.

@mertsincan mertsincan added Status: Pending Review Issue or pull request is being reviewed by Core Team and removed Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible labels Jan 16, 2024
@mertsincan mertsincan added this to the 17.4.0 milestone Jan 16, 2024
@mehmetcetin01140 mehmetcetin01140 self-assigned this Jan 23, 2024
@cetincakiroglu cetincakiroglu added Type: Bug Issue contains a bug related to a specific component. Something about the component is not working and removed Status: Pending Review Issue or pull request is being reviewed by Core Team labels Jan 24, 2024
cetincakiroglu added a commit that referenced this issue Jan 24, 2024
Fixed #14558 - p-calendar: Doesn't close on date selection in range mode
@amonsour
Copy link

amonsour commented Feb 21, 2024

Sorry to open this back up, but now the hideOnDateTimeSelect does not do anything if range is selected.
Should add && this.hideOnDateTimeSelect at the end for those who want the Calendar to stay open:

if ((this.isSingleSelection() && this.hideOnDateTimeSelect) || (this.isRangeSelection() && this.value[1])) {

@Xylios13
Copy link

Due to this change, as @amonsour mentioned, the property hideOnDateTimeSelect is no longer respected when in range selection mode.

Please take a look at this :)

@cetincakiroglu
Copy link
Contributor

This issue is closed by a typo of another PR comment. Re-opening the issue

@github-actions github-actions bot added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Mar 14, 2024
@cetincakiroglu cetincakiroglu modified the milestones: 17.4.0, 17.12.0 Mar 14, 2024
@cetincakiroglu
Copy link
Contributor

Hi,

Couldn't replicate the same behavior in the latest version (17.11.0) https://stackblitz.com/edit/cte5uw-mrpz6a?file=package.json

This issue was probably solved with a PR of another issue between versions 17.0.0 and 17.11.0 . Closing the issue since it doesn't exist anymore.

@cetincakiroglu cetincakiroglu removed Type: Bug Issue contains a bug related to a specific component. Something about the component is not working Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible labels Mar 20, 2024
@cetincakiroglu cetincakiroglu removed this from the 17.12.0 milestone Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants