@pavanbone
domdem-dev
@domdem-devAll comments
- @domdem-dev
Something's wrong with the preview site
- @SebastianJast@domdem-dev
you can use multiple breakpoints to adapt the layout for multiple devices widths
- P@GeraASMWhat are you most proud of, and what would you do differently next time?
I have problems with the radius I couldn't do the design exactly
@domdem-devI tried compiling the form. It shows the success message despite the email is incorrect and despite showing some errors in some of the form fields
- P@GeraASMWhat are you most proud of, and what would you do differently next time?
This project is more easy to anothers I don't have many problems just in the height that how to center the container fqa
@domdem-devWhen clicking a faq, all the others should close. Try implementing a "reset function" in your js
- P@GeraASMWhat are you most proud of, and what would you do differently next time?
The next time I tried to start with the font styles and then the colors, and studied the event keydown
@domdem-devnice job
- P@GeorgeGit1@domdem-dev
Nice job
- P@JeronimoCardu@domdem-dev
A little inaccurate but good job
- P@jull20@domdem-dev
Breakpoints need to be fixed
- P@orrest@domdem-dev
Nice job
- @TroyJose@domdem-dev
The content is not centered properly. Put all the cards inside a container, and than modify the body display to flex, justify-content center and align-content center
- @LIFONGHANG@domdem-dev
Some details like text color or the box-shadows are inaccurate. Then the "Our Artificial Intelligence powered..." part can be centered with margin: 0 auto
- @hastigholami@domdem-dev
My suggestion is to include both the image and the description card inside a new component. Then modify the CSS with display: flex, justify-content: center, and align-items: center
Marked as helpful - @sixwordxiong@domdem-dev
All color fonts are wrong. You can find it in the style-guide of the starter download files
- @lh62863@domdem-dev
Inside button's css, border: 0 should make it more similar to the original
- @Mawuawoe@domdem-dev
maybe react was too much for this simple task
- @Majzen11What are you most proud of, and what would you do differently next time?
I'm proud of managing to get the design looking right mostly on my own. I didn't use the Figma files as I haven't learned to use Figma yet, but I think that might be next on my to-do list.
I felt unsure as to whether I got the sizing correctly, as I did it mostly by looking at the design and testing things out. It looked alright in the end, but I would like to learn more ways of doing this to provide a more accurate result.
I'm pround of figuring out how to use flexbox for this project and problem solving along the way.
What challenges did you encounter, and how did you overcome them?My most difficult challenge was to get the design centered on the page. It would center horizontally, but not vertically, despite me using the proper coding for flexbox.
However, after quite a bit of googling, I came across a forum post of someone who had struggled with the same issue and one of the responses helped me out. I needed to add a height to my component. So I learned a bit about viewport units and added "height: 100vh" to my body and that solved the issue. I still need to learn more about this to completely understand it.
What specific areas of your project would you like help with?I was unsure if I needed to add a <section> tag within my <main> for correct semantic html since this was all one section. I decided not to because I needed to use a <div> anyway, but should I have used a <section>?
Am I missing any important parts of the semantic html?
How should I have managed the sizing? Is there a better way than eyeballing it? The project does come with sizing, but I was unsure of how to use these, as they seemed to be the screen sizes and not the size of the elements.
Is there a better way to center an element than the way I used? Would grid have been better or was flexbox a good option?
@domdem-devok
