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

Release version 5.4 #2287

Merged
merged 17 commits into from
May 24, 2022
Merged

Release version 5.4 #2287

merged 17 commits into from
May 24, 2022

Conversation

RasmusKjeldgaard
Copy link
Collaborator

Changelog

Bugfixes

dropdown - experimental popover feature: Fix popovers hide() being called twice (#2195)
dropdown - experimental popover feature: Only prevent touchevent outside popover (#2205)

Features

segmented-control - add new option to disable change on swipe (#2237)

jkaltoft and others added 14 commits April 28, 2022 09:41
Clean up the mess

TBC...
Bumps [ejs](https://github.com/mde/ejs) from 3.1.6 to 3.1.7.
- [Release notes](https://github.com/mde/ejs/releases)
- [Changelog](https://github.com/mde/ejs/blob/main/CHANGELOG.md)
- [Commits](mde/ejs@v3.1.6...v3.1.7)

---
updated-dependencies:
- dependency-name: ejs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* 🐛  Only remove popover element if it is showing

* ♻️ Refactor hide() to ensure willHide is fired

* 🔥 Remove backdrop click listener

The event listener was not called because dropdowns onBlur event was
always fired first, closing the dropdown including the popover
component. In fact it was only called when clicking inside the dropdown
because onBlur is not fired then - and thus it is useless at this point.

Components using popover (dropdown only right now) should take care of
calling hide() at the appropriate time themselves, e.g. in onBlur.

* 🔥 Remove redundant isShowing check
* 🐛 Only prevent event (e.g. scroll) outside popover

* ♻️  Rename preventEvent callback for clarity

* ♻️ Make callback static

This is to avoid future problems if 'this' is referred within the
callback down the line.
Co-authored-by: Michael Troelsen <mxt@bankdata.dk>
Co-authored-by: Michael Troelsen <mxt@bankdata.dk>
…module

💄 Add missing import statement
…ol/disable-change-on-swipe

segmented-control - disable change on swipe attribute
@github-actions github-actions bot temporarily deployed to pr-main May 24, 2022 07:02 Inactive
@github-actions github-actions bot temporarily deployed to pr-main May 24, 2022 10:22 Inactive
@RasmusKjeldgaard RasmusKjeldgaard merged commit b6881eb into stable May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants