LiquidBounce icon indicating copy to clipboard operation
LiquidBounce copied to clipboard

[BUG] toggling a module off shows a green checkmark

Open Cypphi opened this issue 3 years ago • 61 comments

Describe the bug When you toggle off a module it shows a popup with a green checkmark instead of a red cross or similar to that.

To Reproduce Steps to reproduce the behavior:

  1. Toggle a module on and then off

Cypphi avatar May 11 '21 06:05 Cypphi

I'm not sure if this is intentional behavior or not. I've definitely wondered the same myself but maybe @SenkJu can give more insight into this since he worked on the HUD.

NurMarvin avatar May 11 '21 06:05 NurMarvin

Yeah I also wasn't sure. And if it's intended I'm suggesting to change it to a red cross because otherwise it's very hard to see what you did without reading.

Cypphi avatar May 11 '21 06:05 Cypphi

Currently, it is intended. We might change it.

SenkJu avatar May 11 '21 06:05 SenkJu

I would say a red cross is more likely to be used for an error. Green indicates something has been successfully done.

1zun4secondary avatar May 11 '21 09:05 1zun4secondary

I think that green tick should be for on, Red Cross should be for off, and yellow/orange explaination mark (!) should be for errors

ghost avatar May 11 '21 10:05 ghost

image something like this

ghost avatar May 11 '21 16:05 ghost

This looks so wrong. Of course, it makes sense to show something "red" when you turn off but ... I don't know. It doesn't fit.

1zun4secondary avatar May 11 '21 16:05 1zun4secondary

image something like this

i like that idea, exactly what i meant

Cypphi avatar May 11 '21 16:05 Cypphi

The problem is that notifications are intended to be as generic as possible. We want to use them for other things as well so adding specific icons for module toggle notifications would be a little strange.

SenkJu avatar May 11 '21 16:05 SenkJu

so are there only going to be ticks for icons?

ghost avatar May 11 '21 16:05 ghost

No. There are other icons, but we probably won't use red-X icons for "Disabling module" which is basically a successful task (so it has to be Green)

1zun4secondary avatar May 11 '21 16:05 1zun4secondary

so a green cross?

ghost avatar May 11 '21 16:05 ghost

It must be a green cross whether the module is disabled or enabled. The X icon should be for errors, and the ! icon should be for notification messages.

Examples: X: Failed to load entities. !: Speed module is disabled due to setback detection.

mems01 avatar May 11 '21 16:05 mems01

what about icons being for what type of notification and colours for the status of that thing in the category?

ghost avatar May 11 '21 16:05 ghost

image this is what i meant by the comment above btw

ghost avatar May 16 '21 08:05 ghost

so the colours are for what happened, and the icon is for the category of that thing (like binds, settings, modules ect)

ghost avatar May 16 '21 08:05 ghost

the icons dont need to be exactly like this, this is just showing off the idea

ghost avatar May 16 '21 11:05 ghost

the icons dont need to be exactly like this, this is just showing off the idea

i think they look very good, i'd love if this got added

Cypphi avatar May 16 '21 12:05 Cypphi

image this is what i meant by the comment above btw

that looks so wrong

mems01 avatar May 16 '21 13:05 mems01

what, the icons? like i said, its just showing off the idea, they dont have to look like this

ghost avatar May 16 '21 14:05 ghost

i mean, i personally like the icons, but thats just my opinion

ghost avatar May 16 '21 14:05 ghost

yeah i like the icons too, what they indicate is what i dont like.

mems01 avatar May 16 '21 15:05 mems01

wdym?

ghost avatar May 16 '21 17:05 ghost

image this is what i meant by the comment above btw

this looks cool, btw do you use gimp to edit cause i do too

TheAwesome98-Real avatar May 31 '21 06:05 TheAwesome98-Real

Thank you! And yes, I did

ghost avatar May 31 '21 09:05 ghost

No.

1zun4secondary avatar Jun 22 '21 18:06 1zun4secondary

So wait, why is this issue open then?

ghost avatar Jun 22 '21 19:06 ghost

No.

why

TheAwesome98-Real avatar Jun 23 '21 07:06 TheAwesome98-Real

No.

why

The icons don't really match its purpose.

mems01 avatar Jun 23 '21 09:06 mems01

Like I said, the icons don’t have to be exactly the same, they were just an example

No.

why

The icons don't really match its purpose.

ghost avatar Jun 23 '21 14:06 ghost