Skip to content

Conversation

@Samq64
Copy link
Member

@Samq64 Samq64 commented May 22, 2025

Changes

  • Shows the upload error messages that were already in the code but weren't displayed
  • Dims the buttons while loading and adds a spinner to the clicked one
  • Adds a border to the thumbnail

Reason for changes

It makes it easier to understand what's going on.

2025-05-22_10-05-24

When there is no internet connection:
2025-05-22_10-05-35

Tests

Tested on Brave.

@Samq64 Samq64 added scope: addon Related to one or multiple addons scope: user experience Related to the user experience (UX) aspect of the extension type: enhancement New feature for the project labels May 22, 2025
@Samq64 Samq64 changed the title Thumb setter UX animated-thumb: Improve UX May 22, 2025
@Samq64 Samq64 changed the title animated-thumb: Improve UX animated-thumb: Better status indicators May 22, 2025
Samq64 and others added 2 commits May 23, 2025 07:59
Co-authored-by: Maximouse <brestl1c@outlook.com>
Co-authored-by: Maximouse <brestl1c@outlook.com>
@Samq64 Samq64 merged commit d28d3f3 into ScratchAddons:master May 23, 2025
2 checks passed
@Samq64 Samq64 deleted the thumb-setter-ux branch May 23, 2025 12:00
WorldLanguages pushed a commit that referenced this pull request Jul 21, 2025
* Fix thumbnail upload erorrs not displaying

* Add border to thumbnail

* Add spinner

* Remove line for old localStorage item

* remove overridden style

* Update addons/animated-thumb/userscript.js

Co-authored-by: Maximouse <brestl1c@outlook.com>

* Update addons/animated-thumb/userscript.js

Co-authored-by: Maximouse <brestl1c@outlook.com>

---------

Co-authored-by: Maximouse <brestl1c@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: addon Related to one or multiple addons scope: user experience Related to the user experience (UX) aspect of the extension type: enhancement New feature for the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants