search

codicille / edsa

Web-based and multi-devices e-book reader.
Other
7 stars 0 forks source link

Readability options #8

Closed EtienneLem closed 11 years ago

EtienneLem commented 11 years ago

CSS

j15e commented 11 years ago

Awesome, juste un commentaire pour les icônes, je pense qu'on serait mieux avec une font d'icônes pour les avoir en vectoriel et que ce soit par fait même "retina-ready". Nous on utilise http://icomoon.io/app/ (aux dernières nouvelles) pour générer nos icons fonts.

Et juste pour savoir, pourquoi binder les events dans le dom avec href="javascript:event.trigger()"?

EtienneLem commented 11 years ago

Ouais on utilise http://icomoon.io aussi pour Conference Badge, on updatera ça en conséquence éventuellement!

Bonne question, je ne fais normalement jamais ça pour un “site web”, ceci dit dans une webapp je trouve ça somewhat logique. Ça permet d’avoir moins d’event on('click') au niveau des classes. Les classes sont ainsi plus génériques et dépendent beaucoup moins du DOM. Ça permet aussi de bien classer publique/privé, j’y tiens pas particulièrement si vous êtes pas chaud à l’idée ;)

j15e commented 11 years ago

Parfait pour les icônes, on verra plus tard comme tu dis, c'est pas le nerd de la guerre.

Pour le JS in the dom, ça semble logique et bon à essayer vu le contexte "app-like", pas de contre indication pour ma part!

EtienneLem commented 11 years ago

Que fais-je pour le contraste? Y a-t-il des “thèmes” prédéfinis?

j15e commented 11 years ago

Oui on voulait faire ex. 5 thèmes prédéfini pour les différents types d'éclairage.

Le choix des couleurs existents pas, tu peux mettre des sets de couleurs qu'on ajustera...

j15e commented 11 years ago

cc @L-A peux tu regarder ça pour les couleurs?

EtienneLem commented 11 years ago

C’est pas “final”, mais plus que raisonnable pour une présentation!

j15e commented 11 years ago

Parfait merci! Je vais le déployer et informer René du statut

L-A commented 11 years ago

@j15e Je m'y met bientôt.

@EtienneLem Préfères-tu que je t'envoie les paramètres d'une façon barbare (en texte) pour que tu les intègre, ou bien je te crée une branche et je pull request ça?

j15e commented 11 years ago

Tu peux les intérgrer dret ici je pense : https://github.com/hooktstudios/edsa/blob/2848347bbbef2f1ca3450447cd8d64d8ab729670/app/assets/stylesheets/application.css.scss#L52

EtienneLem commented 11 years ago

Libre à toi. Chose sûr, avec le changement de couleur du background il y aura très fortement autres modifications à l’ensemble de l’app. (Le fade-out header/footer entre autre).

C’est pourquoi les classes de theme sont sur <body> et non <div role="main">, pour pourvoir être capable de tout modifier au besoin.

EtienneLem commented 11 years ago

Yep et si tu veux pas changer la couleur de tout, tout, tout, ça va comme suit:

.theme-2 {
  background-color: #fff;

  [role="main"] {
    color: $mainTextColor;
  }
}