-
-
Notifications
You must be signed in to change notification settings - Fork 112
Cloning: Show progress in sidebar #1641
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
Conversation
|
Note: The "Revert shortcut clash" commit was included in error and reverted. It belongs in #1640. |
|
Just noticed the issue also mentioned sending a notification if the window is not focused. This PR does not implement that at present - it is targeted on the toast aspect. |
|
Now implemented notification if not focused. Do we need to withdraw notification (like Terminal) if Code window becomes active? It did not seem necessary to add an action to the notification - when clicked the sending app was focused anyway. |
zeebok
left a comment
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.
Looks good to me, and functions as advertised. I can't speak to the styling, it feels odd to be right aligned but I don't know what would be better. I do hope to see a future PR for having a progress bar in the dock!
|
@zeebok Thanks for the review! I'll merge and hope that will stimulate further suggestions for appearance improvement. I'll look into progress indication in the new dock. |
I cant test this, but from the screenshot and at a cursory glance through the code : The list of folders pads down to leave space for the toaster ? |
Not that I can recall - I'll push a PR to use an overlay. |
Fixes #1628
During cloning the dialog hides and a spinner appears in the sidebar. On a successful clone a toast appears in the sidebar. If the clone is unsuccessful the behaviour is as before.