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

fix(popover): prevent exiting fullscreen with popover close #551

Merged
merged 4 commits into from
Apr 26, 2023

Conversation

abugraokkali
Copy link
Contributor

@abugraokkali abugraokkali commented Apr 25, 2023

Prevented the browser from exiting fullscreen mode when the popover is closed with the Escape key.
This was accomplished by using the keydown event instead of the keyup event, and making sure the action is only triggered when the Escape key is pressed down.

Fixes #543

@muratcorlu muratcorlu changed the title fix(popover): prevent exiting browser fullscreen with popover close fix(popover): prevent exiting fullscreen with popover close Apr 26, 2023
@muratcorlu muratcorlu merged commit 2d31e3c into Trendyol:next Apr 26, 2023
@muratcorlu
Copy link
Contributor

Thank you @abugraokkali ! 💯

@github-actions
Copy link

🎉 This PR is included in version 2.1.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@muratcorlu muratcorlu mentioned this pull request May 10, 2023
muratcorlu added a commit that referenced this pull request May 12, 2023
This PR includes the features/fixes for the release of v2.1.0

## Features

* **Switch**: CSS property to set custom color for `off` state (#527)
* **Popover**: CSS properties to set `max-width` and `border-size` (#483)

## Fixes

* **Dropdown**: Close dropdown when an item is clicked (#533) 
* **Popover**: Prevent exiting fullscreen with popover close (#551)
* **Dialog**: Prevent exiting fullscreen with dialog close (#548)
* **Select**: Select item separator is back (#552) 
* React wrapper types (#532) 
* Set package type as module (#570)

Thanks to our contributors for this version 🚀 :

@muratcorlu @ogunb @abugraokkali @AykutSarac @ozkersemih
@github-actions
Copy link

🎉 This PR is included in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Closing Popover with Escape should not cause exiting browser fullscreen
2 participants