Reset alert colors to bootstrap default #19
Conversation
|
|
||
| @state-info-text: #26929c; | ||
| @state-info-bg: #eef8fc; | ||
| @state-info-border: #d0f2f7; |
There was a problem hiding this comment.
ça va être tellement moche
There was a problem hiding this comment.
Ça va être tellement plus lisible surtout.
There was a problem hiding this comment.
pas d'accord, subjectif les deux, et c t un choix assumé dés le départ, je ne vois pas pourquoi on change ça mnt
There was a problem hiding this comment.
on a toujours utilisé ces couleurs dans des éléments sur fond blanc et pour le coup ça fonctionnait très bien.
On s'est rendu compte aujourd'hui de 2 choses :
- si on colle plusieurs de ces éléments les uns aux autres, leur couleur perçue est beaucoup plus pâle : le fond blanc perd en importance et l'oeil n'a pas d'élément de contraste
- sur les vieux écrans ou écrans mal calibrés, les couleurs sont difficiles à voire quand le fond blanc est moins important. Le vert devient presque invisible par exemple.
There was a problem hiding this comment.
ok ça me va t'inquiète, j'aimai bien ces couleurs c tout
There was a problem hiding this comment.
@youknowriad moi aussi, sur mon retina, elle rendent bien...
Sinon, autre solution à envisager : essayer de les rendre paramétrable au niveau du thème (comme les trois couleur principales). @youknowriad tu penses que c'est jouable sans trop galérer ?
There was a problem hiding this comment.
@jubianchi ça me parait beaucoup de travail, au fait à part utiliser des classes commes les -one -two etc... je ne vois pas trop comment (les css variables ne sont pas encore là :( ).
donc peut être que c'est trop de boulot pour pas grand chose, autant changer les couleurs pas grave
No description provided.