Skip to content

Conversation

@jeremypw
Copy link
Collaborator

@jeremypw jeremypw commented Aug 12, 2025

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.

Screenshot from 2025-08-11 21 07 39 Screenshot from 2025-08-11 21 10 15

@jeremypw
Copy link
Collaborator Author

Note: The "Revert shortcut clash" commit was included in error and reverted. It belongs in #1640.

@jeremypw jeremypw marked this pull request as ready for review August 13, 2025 10:51
@jeremypw jeremypw requested a review from danirabbit August 13, 2025 10:52
@jeremypw
Copy link
Collaborator Author

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.

@jeremypw
Copy link
Collaborator Author

jeremypw commented Aug 13, 2025

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.

Copy link
Contributor

@zeebok zeebok left a 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!

@jeremypw
Copy link
Collaborator Author

@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.

@jeremypw jeremypw merged commit c941d9c into master Oct 16, 2025
6 checks passed
@jeremypw jeremypw deleted the jeremypw/clone/show-progress-in-sidebar branch October 16, 2025 09:11
@teamcons
Copy link

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.

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 ?
You are not using an overlay so the toast can show up above the list ? Is there a specific reason?

@jeremypw
Copy link
Collaborator Author

You are not using an overlay so the toast can show up above the list ? Is there a specific reason?

Not that I can recall - I'll push a PR to use an overlay.

@jeremypw jeremypw added this to the 8.1.2 milestone Oct 18, 2025
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.

Git clone success dialog should be a toast

4 participants