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

[🚩esl-modal]: beta release version of esl-modal module #1969

Draft
wants to merge 92 commits into
base: main
Choose a base branch
from

Conversation

NastaLeo
Copy link
Collaborator

@NastaLeo NastaLeo commented Oct 2, 2023

Closes: #1376

HenadzV and others added 30 commits March 17, 2023 04:55
Description:

Remove the full version of the polyfill for ESL UI site renderer
Remove shims and targets =ES5 in the site codebase
Remove ES5-related OOTB ESL polyfills except es5-target-shim.ts
Drop support for IE11 and ES5 target

BREAKING CHANGES:

ESL UI site renderer and ESL polyfills no longer support IE11 and ES5 target.
Description:

Remove the medium version of the polyfill for ESL UI site renderer
Drop support for Edge old versions and ES6 polyfils

BREAKING CHANGES:

ESL UI site renderer and ESL polyfills no longer support Edge old versions and ES6 polyfils.
Co-authored-by: ala'n (Alexey Stsefanovich) <astsefanovich@exadel.com>

BREAKING CHANGE: `ESLEventUtils.descriptors` alias of `ESLEventUtils.getAutoDescriptors` is no longer supported
BREAKING CHANGE: `EventUtils` alias of `ESLEventUtils` is no longer supported
BREAKING CHANGE: `esl-media-quey` module no longer supports `addListener` and `removeListener` shorthand
BREAKING CHANGE: `SynteticEventTarget` no longer supports `addListener` and `removeListener` shorthand
BREAKING CHANGE: 'fallback-duration' is no longer in the JSX shape of ESLPanel and ESLPanelGroup
BREAKING CHANGE: `ToggleableActionParams` alias of `ESLToggleableActionParams` is no longer supported
BREAKING CHANGE: `TraversingQuery` alias of `ESLTraversingQuery` is no longer supported
BREAKING CHANGE: `RTLUtils` retired use separate methods instead
BREAKING CHANGE: `TraversingUtils` retired use separate methods instead
BREAKING CHANGE: `deepCompare` alias of `isEqual` is no longer supported
BREAKING CHANGE: `generateUId` alias of `randUID` no longer supported
Deprecation: use `Array.prototype.flat` instead of `flat` utility
# Conflicts:
#	package-lock.json
#	package.json
BREAKING CHANGE: `hasEventListener` no longer accepts min number value use `this.getEventListeners(type).length` to make extended checks
BREAKING CHANGE: `dispatchEvent` does not accepts target argument
BREAKING CHANGE: `createZIndexIframe` and `is-fixes` module no longer available due to drop of IE11 support
Comment on lines 62 to 65
if (!this.cache[selector]) {
this.cache[selector] = this.$fragment.querySelector(selector);
}
return this.cache[selector];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess you missed clearing the cache in the clear() method (in row number 52).

expect(ESLModalStack.store[1]).toBe(TEMPLATE.$modal2);
expect(ESLModalStack.store.length).toBe(2);
});
test('Closing and removing parent modal from store leads to hide all nested modals in case their are opened before target modal closes and its removing from store', () => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it can be simplified

Suggested change
test('Closing and removing parent modal from store leads to hide all nested modals in case their are opened before target modal closes and its removing from store', () => {
test('Removing the parent modal leads to hiding all nested modals if opened', () => {

dshovchko and others added 18 commits January 4, 2024 15:44
fix: popup has wrong positioning leads to a broken popup layout
refactor: small optimization/unification of `Rect` usage in popup
# [5.0.0-beta.4](v5.0.0-beta.3...v5.0.0-beta.4) (2024-01-05)

### Bug Fixes

* **esl-utils:** fix getting viewport sizes ([c17d2e3](c17d2e3))

### Features

* **esl-toggleable:** introducing a base trigger class in purpose to simplify and make Trigger-Toggleable constructions more API strict in ESL ([4ea1565](4ea1565))
…ic/esl-modal

# Conflicts:
#	site/src/localdev.ts
#	site/views/examples/modal.njk
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[🚩esl-modal]: beta release version of esl-modal module
6 participants