Duyen Nguyen• 70
@kyduyennguyen
Submitted
To Dat
@tltd0807
All comments
- To Dat• 290
@tltd0807
Posted
it seems like the font and letter spacing in the button are a little bigger than the design. Also, the space between the subheader, header, and paragraph needs a little bit closer
Marked as helpful
1 - Irtaza Hyder• 30
@SyedMIrtazaHyder
Submitted
Placing the quotation marks was difficult, as i used the relative absolute feature. I think there is a better way. The quotation (which has the tag '#design' in the CSS file) moves in a weird way but works. Also I used flexbox as I have not done CSS grid yet, so is a solution in Grid more easier and smaller in comparison to my FlexBox solution?
To Dat• 290@tltd0807
Posted
hi IRTAZA HYDER’S. First of all, congratulation on your challenge, you did it very nice and i have learned from it a lot And here is my thought: -Grid will be easier for 2 dimensions layout you can watch here -The quotation you can set it as background with the position top left etc... link Also you can check my solution if you want And have a nice day 😁😁
1 - Travolgi 🍕• 31,480
@denielden
Submitted
Hi there! 😀
This is my solution for the Advice Generator App. This app generates Advice and change the backgroud color when you click to the button generate.
Any feedback and advice are welcome!
Happy coding 😁
To Dat• 290@tltd0807
Posted
hi Deniel Den, congratulation on your challenge, Your app doesn't work properly on firefox To fix that you can try: https://stackoverflow.com/questions/71255840/fetch-api-return-same-data-always/71256248#71256248
Marked as helpful
2 - Ahmed Alaa• 10
@Ahmed-Alaa-2001
Submitted
To Dat• 290@tltd0807
Posted
Hi Ahmed Alaa, congratulation for your first project challenge. About the solution, this is my opinion:
- First, it looks pretty nice
- Submit button border:none;
- You can use tabindex for the focus of div tag, or set the focus for the button
Marked as helpful
1 - Elena• 60
@ElenaKondrashova
Submitted
To Dat• 290@tltd0807
Posted
Hi Elena, congrats on completing your project here! It looks very nice, In my opinion, you should use display: flex for the ".description" instead of position: absolute for the h1 and p tag. It is just my opinion, by the way, I have learned from your solution a lot. Have a nice day 😁😁
Marked as helpful
1 - Kelvin• 40
@dripping-code
Submitted
- What's the easiest way to get a page to be responsive to mobile and desktop.
- I hope its better now.
To Dat• 290@tltd0807
Posted
Hello Kelvin, congratulations on your new solution! -for the content in the middle, try to set the width: 100% for the .place or the body
Marked as helpful
0
