Skip to content

Conversation

@nfebe
Copy link
Contributor

@nfebe nfebe commented Dec 8, 2023

  • It's unexpected that click outside a tooltip would proceed with the action that could be carried out inside the tooltip.
  • Allow user to finish typing date before formatting

Resolves : #42046, #42114

@nfebe nfebe marked this pull request as draft December 8, 2023 16:24
@nfebe nfebe marked this pull request as ready for review December 8, 2023 17:06
@nfebe nfebe force-pushed the click-outside-should-abort-share branch from c591790 to 98ee9d6 Compare December 8, 2023 17:07
@nfebe nfebe linked an issue Dec 8, 2023 that may be closed by this pull request
8 tasks
Copy link
Contributor

@artonge artonge left a comment

Choose a reason for hiding this comment

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

Some files seem unrelated

It's unexpected that click outside a tooltip would proceed with the
 action that could be carried out inside the tooltip.

Signed-off-by: fenn-cs <[email protected]>
@nfebe nfebe force-pushed the click-outside-should-abort-share branch from 98ee9d6 to 41061dd Compare December 11, 2023 20:03
Debounce `onExpirationChange` to avoid calling `formatDateToString`
on invalid on uncompletely inputed date strings.

Signed-off-by: fenn-cs <[email protected]>
@nfebe nfebe force-pushed the click-outside-should-abort-share branch from 41061dd to 04e8733 Compare December 11, 2023 20:06
@nfebe nfebe requested a review from artonge December 11, 2023 20:08
@nfebe
Copy link
Contributor Author

nfebe commented Dec 11, 2023

/compile

Signed-off-by: nextcloud-command <[email protected]>
@susnux susnux merged commit 6c482bc into master Dec 12, 2023
@susnux susnux deleted the click-outside-should-abort-share branch December 12, 2023 19:29
@nfebe
Copy link
Contributor Author

nfebe commented Dec 12, 2023

/backport to stable28

@nfebe
Copy link
Contributor Author

nfebe commented Dec 12, 2023

/backport to stable27

@nfebe
Copy link
Contributor Author

nfebe commented Dec 13, 2023

/backport to stable28

@nfebe
Copy link
Contributor Author

nfebe commented Dec 13, 2023

/backport to stable27

@skjnldsv
Copy link
Member

/skjnldsv-backport c179408 04e8733 to stable28

@skjnldsv
Copy link
Member

/skjnldsv-backport c179408 04e8733 to stable27

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[Bug]: Click outside dropdown still create public share [Bug]: Year cannot be typed correctly in sharing flow

6 participants