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

docs(ui-editable,ui-menu,ui-number-input): functional based examples … #1639

Conversation

ToMESSKa
Copy link
Contributor

…are added to InplaceEdit, Menu and NumberInput

Closes: INSTUI-4173

Copy link

github-actions bot commented Aug 15, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-09-13 09:01 UTC

@ToMESSKa ToMESSKa self-assigned this Aug 15, 2024
@ToMESSKa ToMESSKa marked this pull request as ready for review August 15, 2024 09:17
Comment on lines 283 to 286
// IE11 doesn't support Number.isInteger
function isInteger(number) {
return Math.floor(number) === number
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you please remove this part? We dont care about IE11 anymore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed the IE11 workaround

Copy link
Collaborator

@matyasf matyasf left a comment

Choose a reason for hiding this comment

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

Nice work! Can you please remove the workaround for IE11?

…are added to InplaceEdit, Menu and NumberInput

Closes: INSTUI-4173
@ToMESSKa ToMESSKa force-pushed the INSTUI-4173_write_functional_examples_for_inplaceedit_menu_numberinput branch from 2a1dc9c to 31e6dd9 Compare August 27, 2024 15:07
@ToMESSKa ToMESSKa requested a review from matyasf August 28, 2024 07:22
@ToMESSKa ToMESSKa merged commit 5dfc458 into master Sep 13, 2024
11 checks passed
@ToMESSKa ToMESSKa deleted the INSTUI-4173_write_functional_examples_for_inplaceedit_menu_numberinput branch September 13, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants