Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Réorganisation du code SASS #1738

Closed
sandhose opened this issue Nov 12, 2014 · 4 comments

Comments

Projects
None yet
4 participants
@sandhose
Copy link
Member

commented Nov 12, 2014

La partie SASS du front doit être réorganisée. Je propose une séparation par module, et par page, pour les styles spécifiques. Ca donnerait un truc comme ça:

assets/scss/
 |- style.scss
 |- _variables.scss
 |- mixins/
 |   |- _display-flex.scss
 |   \- _sprite-pos.scss
 |- components/
 |   |- _base.scss
 |   |- _typography.scss
 |   |- _layout.scss
 |   |- _icons.scss 
 |   |- _pygments.scss 
 |   |- _buttons.scss 
 |   |- _modals.scss
 |   |- _editor.scss
 |   |- _nav.scss
 |   |- _header.scss
 |   |- _footer.scss
 |   |- _sidebar.scss
 |   |- _lists.scss
 |   |- _dropdowns.scss
 |   \- _forms.scss
 \- pages/
     |- _home.scss
     |- forum/
     |   |- _topics-list.scss
     |   |- _topic-view.scss
     |   |- _topic-create.scss
     |   \- _topic-edit.scss 
     |- messages/
     |   |- _messages-list.scss
     |   |- _message-view.scss
     |   |- _message-create.scss
     |   \- _message-edit.scss
     |- tutorials/
     |   |- _tutorials-list.scss
     |   |- _tutorial-view.scss
     |   |- _tutorial-create.scss
     |   \- _tutorial-edit.scss
     |- ...
     \- ...

Ce qu'il y a dans "scss/pages/" n'est pas encore tout à fait clair, et risque de changer au fur et à mesure que j'avance dans la réorganisation. Je suis en train de la faire en ce moment même

@firm1

This comment has been minimized.

Copy link
Contributor

commented Nov 12, 2014

Comme ça, à première vue il manque les modules suivants :

  • breadcrumb
  • accordion
  • pagination
  • topbar (différent de dropdown)
  • label (ou tag)
  • ...

Je ne vois pas non plus le sens de scss/pages/ car on a pas mal de composant qui se retrouvent a plusieurs niveau.

@sandhose

This comment has been minimized.

Copy link
Member Author

commented Nov 12, 2014

La topbar, je la pensais dans la nav, et/ou dans le header. A voir

Pour les autres, je vais voir, je sais pas si ça a réellement un sens d'avoir des fichiers de 5 lignes (je pense notamment aux labels), alors qu'ils peuvent peut-être se caser autre part. Je vais voir ça au fur et à mesure

Pour le dossier pages, c'est pour les pages spécifiques. Je pense notamment à la home, mais aussi à la vue grille des galeries... Je pensais mettre la dedans tout ce qui n'est pas réutilisable ; si j'ai mis forum/messages/tutos la dedans, c'était surtout pour montrer le principe, mais je pense bien qu'il n'y aura que peu de style spé pour certaines pages.

@firm1

This comment has been minimized.

Copy link
Contributor

commented Nov 12, 2014

Pour les autres, je vais voir, je sais pas si ça a réellement un sens d'avoir des fichiers de 5 lignes (je pense notamment aux labels)

Ah j'ai écrits label, mais je pensais plutôt à alert (pour les erreurs, warning, etc.) qui apparaissent plus haut.

Pour le dossiers pages, oui, on verra, je ne pense pas qu'on aura grand chose dedans. Car même sur la home, pour moi ce sont plutôt de composants "box" qu'on devrait avoir.

@sandhose sandhose self-assigned this Nov 15, 2014

@SpaceFox SpaceFox added this to the "Futur proche" (v1.x) milestone Nov 21, 2014

@sandhose sandhose added the En cours label Jan 14, 2015

@SpaceFox SpaceFox modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 3, 2015

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Feb 3, 2015

C'est corrigé et mergé, encore merci !

@SpaceFox SpaceFox closed this Feb 3, 2015

@SpaceFox SpaceFox removed the En cours label Feb 3, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.