Closed bmlancien closed 3 years ago
@henhuy
error
and warning
classes on the nav-item
containers (not nav-link
). Since I don't know which element you are targeting, I commented out a part of the code (see _graphs.scss
) in order to target nav-link
instead. But if it's nav-item
, you can replace it with the right code (but you need to compile the scss into css...). errors
, warnings
and I now added an infos
class as well.Thanks @bmlancien. Error and warnings are working now. But for refresh button I get some unintended behavior, as sometimes button gets disabled (though I don't see disabled flag). Steps to reproduce:
Could you fix/check one more time, @bmlancien ? sorry
@henhuy Do you want to keep using the .update
class for the refresh buttons if they will always be displayed in green anyway?
Now you can:
error
orwarning
to theli
items with thenav-item
classes. Theerror
class will display a red color andwarning
a brown color. I did two classes in the case there would not be any error but only warnings. If it's too complicated to implement, just use theerror
class, it's totally fine for me.disabled
class from the refresh buttons when they need to be active. It should switch from grey and not active and clickable to green and clickable.PS: I didn't merge the
main
branch since there was some conflicts, so I'd rather let you solve them.