@gabrielfelipeeeSubmitted almost 2 years ago
Leave your feedback!
Matheus Viana de Souza
@vianaV19All comments
- @vianaV19Posted over 1 year ago
I like it. The card has a little big dimension than the design, but it's ok. Very good job! Have a nice day!
"Always Learning..."
0 - @PatrykPorabikSubmitted over 1 year ago
I will probably make some fixes to the project, but I decided to push it to the review. So far, the most tricky part was to handle menu. I've tried to work on the same nav elements on every device, however I was struggling with its appearance. I decided to make two independent elements, but I know that it may not be the best solution in the bigger projects. This part has potential to be improved.
Same with the sidebar menu, I've made it, however I don't like the behavior that on reload, we can see the transforming (translateX) element. It should be hidden from the beginning.
@vianaV19Posted over 1 year agoHello Patryck, Nice to meet you,
I've seeing your solution and I noticed that the last list of elements are overflowing the page width. You can create a class called container defining the width and set it up in each webpage section.
This is my solution in the case you want to see and do a constructive criticism. Thank you!.
Github news-homepage solution: https://github.com/vianaV19/news-homepage Link to solution website: https://news-homepage-murex.vercel.app/
Greetings!, Have a nice day!.
0 - @crisanrimorSubmitted almost 2 years ago
First project using SASS, feedback welcome.
@vianaV19Posted over 1 year agoHello cristian rincon, Nice to meet you,
I've seeing your solution and I noticed that there is one line more than expected on news aside. I hope you can fix this. If you want I made a solution without this inexpected error. I'm open for constructive criticism.
Github news-homepage solution: https://github.com/vianaV19/news-homepage Link to solution website: https://news-homepage-murex.vercel.app/
Greetings!, Have a nice day!.
0
