Conversation
|
The app.css is a generated file and thus must not be modified, only sass files. Remove the change on app.css and add the appropriate changes in sass files. Adding a carousel implies a higher importance on the first project than the following projects, since people that visit websites do not have patience and time to wait for the carousel to advance. Therefore, only the first item matters, to them. Since we do not have many projects, IMO, we should show them directly, without using carousel. I will ask, also, for the final images, otherwise this PR is incomplete. Or ask me to supply them. |
|
About the app.css, i'll change it back and, in fact, i won't need to add anything to sass files. The things i needed changed are not useful now that the carousel will not be used. |
|
My statements are based on several articles and blog posts that defend that the carousel provides a bad UX, so it should be avoided. For this case, carousel hides two projects that have the same importance as the first shown project. |
Added the released projects, displayed in a bootstrap carousel. Had to import that on the beggining, dont know if its the best way to use it but it was a way to show it working. Later the images of the project will be added too.
The app.css shows so many lines added because it was beautified, but i only added 3 or 4 things on the carousel indicators.