Saima Ahmed Sial
@saimasial-bitAll comments
- @frontend-devvSubmitted 9 days ago@saimasial-bitPosted 9 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @ysagohhSubmitted 9 days agoWhat challenges did you encounter, and how did you overcome them?
I had a hard time with the social links overlay. Perhaps there is a more elegant solution, but I struggled setting the position manually using position: absolute.
@saimasial-bitPosted 9 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
Marked as helpful0 - @cursebreakersSubmitted 9 days ago@saimasial-bitPosted 9 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @Niti60Submitted 9 days ago@saimasial-bitPosted 9 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @lucas-dslSubmitted 9 days agoWhat challenges did you encounter, and how did you overcome them?
Em um determinado momento próximo ao final do projeto tive que dar uma revisada em meus estudos de flexbox para organizar alguns itens sem bagunça.
@saimasial-bitPosted 9 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
Marked as helpful0 - @Jamzkid2024Submitted 9 days ago@saimasial-bitPosted 9 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @Aline9402Submitted 9 days agoWhat are you most proud of, and what would you do differently next time?
Orgulho de conseguir estilizar boa parte do desafio e entender os erros de alinhamento
What challenges did you encounter, and how did you overcome them?Alinhar a ultima parte, o nome com a imagem do rapaz
What specific areas of your project would you like help with?Estilização
@saimasial-bitPosted 9 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
Marked as helpful0 - @Fernand-GomesSubmitted 9 days ago@saimasial-bitPosted 9 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @haryoariefSubmitted 9 days agoWhat are you most proud of, and what would you do differently next time?
Now I can use hover
@saimasial-bitPosted 9 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @jiangying-97Submitted 9 days ago@saimasial-bitPosted 9 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
1 - @Gabin-HernandezSubmitted 9 days agoWhat are you most proud of, and what would you do differently next time?
It took 2 days to complete this challenge, but I'm proud of never giving up, sometimes we say "I can't do it" and it is just a rock in the way. You can, for real.
What challenges did you encounter, and how did you overcome them?Holy molly with js, it was my first time using js in a real project and it's so different when you are alone with the code. I learned a lot of things in this challenge, some things about queryselectors and nodelist, by the way, you have to be careful with this nodelist.
What specific areas of your project would you like help with?I don't know if it is possible to do cleaner code, feel free to check it out.
@saimasial-bitPosted 9 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @Glen998Submitted 9 days agoWhat challenges did you encounter, and how did you overcome them?
i have trouble puting this on github i dont what happen but it work now
What specific areas of your project would you like help with?maybe i write this in not the best way so please give some suggestion so i can learn and improve thank you!
@saimasial-bitPosted 9 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
Marked as helpful1 - @Mohammed-BataSubmitted 15 days agoWhat specific areas of your project would you like help with?
i would like to have a feedback about responsive
@saimasial-bitPosted 15 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @MOUHAMADOU80Submitted 15 days agoWhat are you most proud of, and what would you do differently next time?
rien
@saimasial-bitPosted 15 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - @samatha2303Submitted 15 days ago@saimasial-bitPosted 15 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
Marked as helpful0 - @SylvainPS78Submitted 15 days ago@saimasial-bitPosted 15 days ago
"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
Marked as helpful0 - P@barriedirkSubmitted 15 days agoWhat are you most proud of, and what would you do differently next time?
I could manipulate better the display grid and the box-shadow.
What challenges did you encounter, and how did you overcome them?the challenges I had were the media query, trying to get the correct font-size and the background.
What specific areas of your project would you like help with?I think the sizes of the testimonial boxes for each media query, and the position of the background quote.
@saimasial-bitPosted 15 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
0 - P@moustafaessamSubmitted 16 days agoWhat are you most proud of, and what would you do differently next time?
added accessibility and learn different things regarding accessible features
What challenges did you encounter, and how did you overcome them?how to make the components accessible but overcomed it by googling
What specific areas of your project would you like help with?any feedback will be highly appreciated
@saimasial-bitPosted 16 days ago"Great job on the solution! The layout looks clean and responsive. However, I noticed that you haven't used semantic HTML elements like <article> and <section>. Adding them would improve accessibility and readability. Also, on smaller screens, the text alignment looks slightly off. Adjusting the padding/margins might help. Keep up the great work!"
1