Gradience icon indicating copy to clipboard operation
Gradience copied to clipboard

feat: yet another ui polish

Open daudix opened this issue 3 years ago • 10 comments

Feature description

UI improvements that need to be done before v0.3.0

  • [x] Add quick preset switcher back, it was pretty useful
  • [x] Warning dialog when closing Gradience and preset are not saved
  • [x] Don't show "Preset renamed" toast in preset manager
  • [x] Repo indicator, e.g Official and Curated, useful when preset have official and unofficial variant
  • [ ] Hide preset immediately after pressing remove and not after closing toast
  • [ ] Warning dialog when resetting applied preset

Additional Context

Quick preset switcher (this will require a "Favourite" button in preset manager)

image

🔗 Mockup

Repo indicator, useful in case like this

изображение

🔗 We already discussed this

Did you read the Code of Conduct?

daudix avatar Sep 13 '22 20:09 daudix

Where do you want to add the quick switcher ?

0xMRTT avatar Sep 13 '22 20:09 0xMRTT

Where do you want to add the quick switcher ?

I think it could be just under Presets button as before.

tfuxu avatar Sep 13 '22 20:09 tfuxu

Yes :+1:

0xMRTT avatar Sep 13 '22 20:09 0xMRTT

Where do you want to add the quick switcher ?

I think it could be just under Presets button as before.

Yup 👍

daudix avatar Sep 13 '22 20:09 daudix

image

0xMRTT avatar Sep 13 '22 21:09 0xMRTT

Reopened because I don't want to open a bunch of other UI tweak issues

daudix avatar Sep 13 '22 23:09 daudix

Ok 😎

0xMRTT avatar Sep 14 '22 05:09 0xMRTT

image

0xMRTT avatar Sep 16 '22 06:09 0xMRTT

Wow 🔥

daudix avatar Sep 16 '22 08:09 daudix

Another thing to add to the list: Screenshot from 2022-09-24 20-36-40 This explanation box doesn't have top and bottom paddings

tfuxu avatar Sep 24 '22 19:09 tfuxu

Updated to-do

daudix avatar Sep 25 '22 20:09 daudix

Tone isn't working ?

0xMRTT avatar Sep 25 '22 20:09 0xMRTT

Yes, switching tone and pressing apply does nothing visually

daudix avatar Sep 25 '22 20:09 daudix

It's change the palette :D

0xMRTT avatar Sep 25 '22 20:09 0xMRTT

Screenshot from 2022-09-28 16-11-50

0xMRTT avatar Sep 28 '22 14:09 0xMRTT

Why would we need to show badges here? Wouldn't it be better to put a 'Remove preset' button here instead?

tfuxu avatar Sep 28 '22 15:09 tfuxu

Or in the upper row instead of being in a submenu

0xMRTT avatar Sep 28 '22 15:09 0xMRTT

Screenshot from 2022-09-30 23-47-04 We should make this page more clear for a user, as there isn't any mention of it configuring Flatpak overrides, and System configuration can be confusing.

tfuxu avatar Sep 30 '22 21:09 tfuxu

Yes, also Flatpak overriding isn't working

0xMRTT avatar Sep 30 '22 22:09 0xMRTT

But in same time we need to keep it simple for the regular user

daudix avatar Sep 30 '22 22:09 daudix

Screenshot from 2022-10-02 21-48-32

0xMRTT avatar Oct 02 '22 19:10 0xMRTT

.Devel icon shows now 🎉

daudix avatar Oct 03 '22 20:10 daudix

Lol how 😅😀

0xMRTT avatar Oct 04 '22 05:10 0xMRTT

I don't know, maybe one of tfuxu commits?

daudix avatar Oct 04 '22 10:10 daudix

I didn't noticed that before but a fix for https://github.com/GradienceTeam/Gradience/issues/406#issuecomment-1257049286, added too much padding to other boxes: Screenshot from 2022-10-08 15-26-41

tfuxu avatar Oct 08 '22 13:10 tfuxu

0xMRTT is offline today so I will try to fix this

daudix avatar Oct 08 '22 13:10 daudix

Alright, I think the padding is OK

daudix avatar Nov 17 '22 15:11 daudix

Alright, I think the padding is OK

I don't think so... Screenshot from 2022-11-17 18-45-03 Screenshot from 2022-11-17 18-45-20

tfuxu avatar Nov 17 '22 17:11 tfuxu