-
Notifications
You must be signed in to change notification settings - Fork 20
Conversation
Good job!!! So much better 😁 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Not sure it's fully done...? It looks like a few things are missing at first and there's a few nitpicks too.
Hope that helps!
.travis.yml
Outdated
@@ -0,0 +1,4 @@ | |||
language: ruby | |||
rvm: | |||
- 2.2.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is an old version, but meh...we don't really care :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fixed, we're now using 2.4.2 👍
@@ -0,0 +1,9 @@ | |||
emileber: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we will need to add everyone who ever did a talk in that file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exact
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could even use github avatar with a valid github
key by using this plugin: https://github.com/benbalter/jekyll-avatar 🎉
Edit: See #84
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On parle en français partout ailleurs dans ce repo, bien que je préfère l'anglais sur github, c'est assez restreint comme projet.
Ceci dit, je prévois faire une page des auteurs avec photos et plus de détails éventuellement et avec la nouvelle structure, ça va être vraiment facile à ajouter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En effet, c'est l'habitude
_includes/footer.html
Outdated
</footer> --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove -->
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Faudrait que je cleanup, j'ai laissé une couple d'affaire du vieux site comme référence durant que je remplaçais dans le nouveau.
<div id="main" class="container"> | ||
|
||
<!-- Image --> | ||
<!-- <a href="#" class="image featured"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove commented code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nah, lui il va finir par servir, j'ai juste pas eu le temps d'intégrer l'image des event facebook ou l'image de l'event tout court.
<li><a href="{{ next_io.event_url }}" class="button style1 big" target="_blank">Je participe</a></li> | ||
{% endif %} | ||
|
||
<!-- <li><a href="#" class="button style2 big">More Info</a></li> --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove commented code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'aimerais ça qu'il serve aussi, mais je ne lui ai pas encore trouvé de fonctionnalité 😐
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Peut-être un lien vers l'archive des événements?
@@ -0,0 +1,37 @@ | |||
<div class="wrapper style3"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is dummy text FYI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cette section n'est pas encore intégré, mais elle va contenir les témoignages, ou les derniers événements (mais c'est moins pertinents).
_includes/top_nav.html
Outdated
<ul> | ||
<li class="current"><a href="{{ site.baseurl }}">Accueil</a></li> | ||
|
||
<!-- <li><a href="/openhack">OpenHack</a></li> --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove instead of comment IMHO.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça va probablement revenir, donc j'ai seulement commenté ce que je ne veux pas qui apparaisse pour l'instant.
_includes/top_nav.html
Outdated
<!-- For large screens --> | ||
<div id='top-nav'> | ||
<!-- <div id='top-nav'> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove instead of comment IMHO.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça faut effectivement l'enlever.
index.html
Outdated
<!-- Logo --> | ||
<div id="logo"> | ||
<img class="logo" src="{{ '/images/logo_transparent_cropped.png' | relative_url }}" /> | ||
<!-- <h1>Saglac IO</h1> --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this commented?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parce que c'est le logo qui est affiché, pas le titre en texte. Il y a surement un moyen facile d'afficher les 2 pour le référencement, mais pour l'instant, j'y vais pour la fonctionnalité, après, on fera une PR référencement au besoin.
index.html
Outdated
</div> | ||
|
||
|
||
<!-- <div id='event-panels'> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove instead of comment IMHO
Gardez en tête que c'est l'alpha, j'ai seulement mis la landing page et j'ai commenté beaucoup de section dans le but d'avoir de quoi de fonctionnel pour demain. |
J'ai aussi retravaillé la structure des datas pour avoir une plus grande souplesse et moins de répétition lors d'ajout d'événement. Il faudra éventuellement migrer les vieilles données. |
* Using ruby 2.4.2 stable version for CI ✌️ * Specify 2.4.2 in Gemfile ✌️ Should fix the following message from CI: > Your Ruby version is 2.4.2, but your Gemfile specified 2.2.2 * Specify 2.4.2 in .ruby-version for rbenv
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GabLeRoux ouais j'ai remarqué mais j'ai pas encore trouvé pourquoi. Un affaire avec l'overlay qui chi quand le menu mobile est la... Si tu trouve, je te paye la bière 🍺 demain, je m'occupe de la FAQ en ce moment et des autres pages. |
@GabLeRoux Ouais le menu est vide parce que j'ai temporairement commenté la main nav, mais il ne sera pas vide aussitôt que j'ajoute les autres pages. Je vais faire un dernier commit ici, et je propose qu'on fasse plusieurs PR sur la branch |
* Feature/new website template (#82) * Filled the footer with the right information * Added sass styles * Add Travis build image * Add next io * Add new saglac io logo * Using ruby 2.4.2 stable version for CI ❤ (#83) * Fix responsive breakpoints (#88) * Fix header-wrapper::before overlay using negative z-index (#89) * Fix broken responsive breakpoints * Fix header-wrapper::before overlay using negative z-index * Fix font-awesome missing icons font (#95) * Removes manually tracked font-awesome * Adds font-awesome using bootstrapcdn * Initial FAQ page layout and questions data (#97) * Stylize FAQ page * Moves manual fixes back to custom css & fix logo positionning with bigger header * Adds editorconfig so we don't mess with withespace * Fixes z-index * Fix archives page (#105) * Disable travis email notifications (#104) * Add anchor links to faq questions (#103) * Add about page (#102)
Voici la version 2.0 alpha avec seulement la landing page.