-
Notifications
You must be signed in to change notification settings - Fork 159
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
[full-ci] Show local loading spinner in sharing button #9423
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
...es/src/components/SideBar/Shares/Collaborators/InviteCollaborator/InviteCollaboratorForm.vue
Outdated
Show resolved
Hide resolved
...es/src/components/SideBar/Shares/Collaborators/InviteCollaborator/InviteCollaboratorForm.vue
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 things I noticed in the UI:
- The button needs to be disabled when in loading mode (it was like this already before).
- I actually liked the way it looked before a bit more. The loading spinner is very hard to see on that blue background now.
...es/src/components/SideBar/Shares/Collaborators/InviteCollaborator/InviteCollaboratorForm.vue
Outdated
Show resolved
Hide resolved
done, now its outlined passive button on save |
SonarCloud Quality Gate failed. 0 Bugs 25.0% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
* Add spinner / loading prop to oc-button * Add delayed spinner, set min-width * Add changelog, increase spinner size, cleanup code * Update snapshots (because of one v-if change in oc-button) * Address PR issues * Replace min-width with padding * Fix infinite loading / missing unref * Update snapshots * Address PR issues improve button look
* Add spinner / loading prop to oc-button * Add delayed spinner, set min-width * Add changelog, increase spinner size, cleanup code * Update snapshots (because of one v-if change in oc-button) * Address PR issues * Replace min-width with padding * Fix infinite loading / missing unref * Update snapshots * Address PR issues improve button look
Description
We've added a loading spinner to the share button in the sharing panel
to indicate that its still processing if action takes longer.
See #9425
Related Issue
Types of changes
Checklist: