I look forward to your comments 💚
César Mejias
@CesarGMEFAAll comments
- @CesarGMEFASubmitted about 2 years ago
- @matias-userSubmitted over 2 years ago
greetings dear community
@CesarGMEFAPosted over 2 years agoI'm doing another job and the selectButton.js code was very helpful, thank you
0 - @LucaTwentySevenSubmitted over 2 years ago
I think I can shrink my code at least about 20%. What do you think?
@CesarGMEFAPosted over 2 years agoGood job, this is a challenge that I have not yet started. I would like to know what JavaScript is needed?
I want to know after doing it. If you want you can review my work so that later you can do it.
0 - @victoriacheng15Submitted almost 3 years ago
feedback is always welcome and appreciated.
@CesarGMEFAPosted almost 3 years agoI think my screen is 1366px and the view I receive from your page is this. https://prnt.sc/26b794j and this https://prnt.sc/26b79cv then it gives me the impression that it is not ok.
But when I test the responsive from the element inspector everything seems to be fine. Look https://prnt.sc/26b7ac9
0 - @victoriacheng15Submitted almost 3 years ago
feedback is always welcome and appreciated.
@CesarGMEFAPosted almost 3 years agoHi bro, from my 14-inch laptop your page does not see well :(
0 - @ouuoussamaSubmitted about 3 years ago
i want your opinions
@CesarGMEFAPosted about 3 years agoYou could accommodate the paragraphs of the buyers' comments, putting a (font-size: 15px;) and (line-height: 21px;) and in the list of styles it says that the font size should be 15px.
0 - @hamilton-i7Submitted about 3 years ago
This is my first challenge using React. Any constructive feedback is welcome
@CesarGMEFAPosted about 3 years agoI do not understand that. Could you explain me please
0 - @mewebbieSubmitted over 3 years ago
Feedbacks are most welcome.
@CesarGMEFAPosted about 3 years agoHi! I want to tell you that you can apply this to the rem measure. html { font-size: 62.5%; } In this way 1rem would be the same at 10px. so 1.8rem is 18px, 3rem is 30px.
the size of the browser is 16px and if you pass that code it is 10px.
0 - @JakeyikappSubmitted over 3 years ago
Please let me know if there is anyway I can structure my HTML or CSS better or if there are any better ways to code this challenge.
@CesarGMEFAPosted over 3 years agoGood sibling code, you could place the .attribution inside the main so that it doesn't overflow on top of the card and add a margin-top to it. For instance.
The other thing is that you can put a link to your name in the attribution.
The other thing you could accommodate is the width in the .card-container in the @media. It's at 330px and the size of an iphone 5 would be 320px, so it looks weird.
I do those things and send you the proposed changes on github. and check the size of the box because in the body you put a height of 100vh and it still produces a horizontal scroll.
Marked as helpful1