Skip to content

Conversation

VilppeRiskidev
Copy link
Collaborator

Closing the modal while importing is in progress used to cause confusing UX, as the data state changes unexpectedly. This change prevents closing the modal.

@anttimaki
Copy link
Collaborator

@VilppeRiskidev this still leaves the close modal icon visible but broken on top right corner of the screen.

@VilppeRiskidev VilppeRiskidev force-pushed the prevent-closing-profile-import-progress-modal branch from 0fce794 to 58e408d Compare September 5, 2024 12:30
@VilppeRiskidev
Copy link
Collaborator Author

@VilppeRiskidev this still leaves the close modal icon visible but broken on top right corner of the screen.

Fixed!

Copy link
Collaborator

@anttimaki anttimaki left a comment

Choose a reason for hiding this comment

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

👌🏻

@anttimaki anttimaki merged commit e57cd78 into develop Sep 10, 2024
5 of 7 checks passed
@anttimaki anttimaki deleted the prevent-closing-profile-import-progress-modal branch September 10, 2024 14:55
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.

2 participants