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

[calendar] Multiple calendar popups can be opened simultaneously. #5532

Closed
1 of 4 tasks
allen138 opened this issue Jul 19, 2022 · 1 comment · Fixed by #5603
Closed
1 of 4 tasks

[calendar] Multiple calendar popups can be opened simultaneously. #5532

allen138 opened this issue Jul 19, 2022 · 1 comment · Fixed by #5603
Assignees
Labels
bug This issue is a bug in the code Medium Prio SAP SF TOPIC B

Comments

@allen138
Copy link

allen138 commented Jul 19, 2022

Bug Description

Hi team,
When there are multiple date pickers on a page its possible for each one to have the calendar popups opened at the same time. If the date pickers are stacked in a row, this could cause an overlapping issue. Please see the following screen shot:
image

The calendar popups only seem to close on 'mousedown' events, PopoverRegistry.js#L40.

Note: This is not reproducible with the sap.m.DatePicker. Please see, sap.m.sample.DatePicker and try the same replication steps below (steps 2-4)

Thanks,
Allen

Expected Behavior

Only one calendar popup should be opened at a time. Possible solution, close the popup on blur as long as the focus is not on the input

Steps to Reproduce

  1. Go to ui5-webcomponents playground - DatePicker
  2. Click on one date picker icon to open the calendar popup
  3. Click on the date pickers input
  4. Press tab to focus on the next input
  5. (first date picker's calendar popup is still open. Shouldn't it be closed at this point since the focus is not on the input or the calendar popup?)
  6. Press f4 to open the second calendar popup. Notice two popups are open at the same time. Depending on screen size and placement of the two datepickers, the calendars can easily overlap.

Isolated Example

If possible, please provide an isolated example by forking CodeSandbox.

Context

  • UI5 Web Components version: all
  • OS/Platform: all
  • Browser: all
  • Affected component: date picker components

Priority

  • Low
  • Medium
  • High
  • Very High

Stakeholder Info (if applicable)

  • Organization: SF
  • Business impact: Medium
@allen138
Copy link
Author

@tsanislavgatev , I see there has been a PR linked to this issue for some time now. Any updates as when this will be resolved?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug in the code Medium Prio SAP SF TOPIC B
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

4 participants