To Dat
@tltd0807All comments
- @kyduyennguyenSubmitted 12 months ago@tltd0807Posted 12 months ago
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 helpful1 - @SyedMIrtazaHyderSubmitted about 2 years ago
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?
@tltd0807Posted about 2 years agohi 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 - @denieldenSubmitted about 2 years ago
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 😁
@tltd0807Posted about 2 years agohi 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 helpful2 - @Ahmed-Alaa-2001Submitted about 2 years ago@tltd0807Posted about 2 years ago
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 helpful1 - @ElenaKondrashovaSubmitted about 2 years ago@tltd0807Posted about 2 years ago
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 helpful1 - @dripping-codeSubmitted over 2 years ago
- What's the easiest way to get a page to be responsive to mobile and desktop.
- I hope its better now.
@tltd0807Posted over 2 years agoHello 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 helpful0