Skip to content

Conversation

@karrtikr
Copy link

@karrtikr karrtikr commented Aug 2, 2022

For #19621

Change refresh icon when interpreter list is refreshing:
spinner

Add a back button in Enter Interpreter path... options: image

@karrtikr karrtikr added the feature-request Request for new features or functionality label Aug 2, 2022
@karrtikr karrtikr marked this pull request as ready for review August 2, 2022 21:01
karthiknadig
karthiknadig previously approved these changes Aug 3, 2022
@karrtikr
Copy link
Author

karrtikr commented Aug 3, 2022

Checked with Miguel who recommended using the spinning loading octicon:

loadingspinner

@karrtikr karrtikr requested a review from karthiknadig August 3, 2022 18:19
karthiknadig
karthiknadig previously approved these changes Aug 3, 2022
@karrtikr
Copy link
Author

karrtikr commented Aug 3, 2022

@karthiknadig Sorry, CI failed, please re-review.

@karrtikr karrtikr merged commit 93fd4e8 into microsoft:main Aug 4, 2022
@karrtikr karrtikr deleted the refreshing branch August 4, 2022 21:06
wesm pushed a commit to posit-dev/positron that referenced this pull request Mar 28, 2024
…n#19611)

* Change refresh icon when interpreter list is refreshing

* Add tests

* Minor tweaks

* Fix situation if dialog box is cancelled

* Fix tests

* Improve ignoreErrors() typing

* Update to not use custom svg, instead use built VSCode icon

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

Labels

feature-request Request for new features or functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants