ipepe - whole page refactor #30
Conversation
|
Zrobcie code review: @kwarpechowski @kamilkowalski |
| @@ -1,2 +1,3 @@ | |||
| source 'https://rubygems.org' | |||
| gem 'github-pages' | |||
| source 'https://rubygems.org' do | |||
There was a problem hiding this comment.
Nie trzeba wrzucać tego do bloku, to jedyny source i pewnie tak zostanie.
|
Ogólnie fajnie, że wypadło tyle kodu zbędnego. Trzeba by dodać trochę kolorów uczelni i zmienić zdjęcia w tle na bardziej związane z tym co robimy, może coś mniej biznesowego i bardziej na luzie. Nie jestem też szczególnie za użyciem Bootstrapa, bo jest wszędzie, ale rozumiem że dużo ułatwia. Można wrzucić go chyba w wersji Sass, wtedy można nadpisać zmienne Bootstrapa swoimi kolorami jeżeli trzeba, precompiler powinien dać sobie radę. |
|
Czy mógłbyś też ujednolicić wcięcia? Raz widzę tabulatory, raz dwie spacje, raz cztery. |
|
Poprawiłem formatowanie i dodałem sass cache. Niestety w najbliższym czasie nie będę miał czasu na pomoc w temacie stronki więc jakbyś mógł kamil to przemij temat. |
Zrobilem refaktor calej strony na nowy template, mamy tez i18n