Fr33dan / GPSaveConverter

Utility to transfer save files between the games installed from the Xbox app and other storefronts. https://ko-fi.com/fr33dan
295 stars 33 forks source link

Replace non-Xbox location error prompt with flashing error icon #89

Open FlaminSarge opened 1 month ago

FlaminSarge commented 1 month ago

When non-Xbox location has an error/is missing, instead of interrupting the app with a popup warning, this creates an error icon next to the "Select non-Xbox Location" button. The icon flashes for about 3 seconds whenever it gets a new error (if the error message remains the same as before it does not flash). Hovering over the icon shows the reason for the error (which was previously the text inside the popup box).

Tested setting and clearing the error with various games, it seems to work fine.

The only thing I can think of that would be a concern about this is potential accessibility issues for screen readers/tiny text.

image image

FlaminSarge commented 2 days ago

@Fr33dan any concerns with this impl? should the error be moved to be more prominent around the non-Xbox location box?