3 ms·
Did a quick browse through the code. A couple of things jumped out: 1. A number of CSS classes have 'furatto' as a prefix ex. '.furrato-block'. This seems a bi
by bushido 13y ago
Did a quick browse through the code. A couple of things jumped out:
1. A number of CSS classes have 'furatto' as a prefix ex. '.furrato-block'. This seems a bit counter-productive as the word itself does not add any value to meaning of the class names.
2. I notice that for the responsive design the class is '.row-fluid', but since responsive is now a norm it may make sense to have '.row' be fluid/default and have a '.row-fixed' class for the few that may want a fixed only.
3. I like the font, it works well, but for the navbar links its a little tough to read. Also the '.motto' class is almost impossible to read, though changing the font color for the class should suffice.
4. I really like the circular images and the inclusion of date picker and toolbars js.
I will definitely try it out and watch it on github.
p.s. you may want to look into the double right scrollbar that is on the github.io pages.
- kurenn88 13y agoHey thanks a lot for the feedback!!...it's really great!...I totally agree with on the row-fluid class name I'll updated that. I want to make a more semantic class names to give more meaning to the markup.... I'll consider the font on the navbar...probably just reduce the font weight would do the trick.... Thanks again for the feedback. I'll be working on it! :)