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

refactor(ui5-wizard): rename event parameter #8845

Merged
merged 2 commits into from
Apr 26, 2024
Merged

refactor(ui5-wizard): rename event parameter #8845

merged 2 commits into from
Apr 26, 2024

Conversation

hristop
Copy link
Contributor

@hristop hristop commented Apr 25, 2024

Rename changeWithClick to withScroll in the WizardStepChangeEventDetail.

If the application code relied on qchangeWithClic event detail, it should now rely on withScroll event detail instead. The withScroll parameter will be set to true when the event is fired due to user scrolling.

BREAKING CHANGE: changeWithClick was renamed to withScroll in the WizardStepChangeEventDetail.

JIRA: BGSOFUIRILA-3867

Related to: #8461

@simonarangelova
Copy link
Contributor

My suggestion for the commit message: "If the application code relied on changeWithClick event detail, it should now rely on withScroll event instead. The withScroll parameter will be set to true when the event is fired due to user scrolling. ..."

@hristop
Copy link
Contributor Author

hristop commented Apr 25, 2024

My suggestion for the commit message: "If the application code relied on changeWithClick event detail, it should now rely on withScroll event instead. The withScroll parameter will be set to true when the event is fired due to user scrolling. ..."

Done :)

@hristop hristop merged commit 9882144 into main Apr 26, 2024
9 checks passed
@hristop hristop deleted the wizard-rename branch April 26, 2024 09:45
nnaydenow pushed a commit that referenced this pull request Apr 29, 2024
Rename `changeWithClick` to `withScroll` in the `WizardStepChangeEventDetail`.

If the application code relied on `changeWithClic` event detail, it should now rely on `withScroll` event detail instead. The `withScroll` parameter will be set to true when the event is fired due to user scrolling.

BREAKING CHANGE: `changeWithClick` was renamed to `withScroll` in the `WizardStepChangeEventDetail`.

JIRA: BGSOFUIRILA-3867

Related to: #8461
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.

None yet

4 participants