@Tasin269Submitted 2 months ago
Ferdiansyahmulyadi
@FerdiansyahmulyadiAll comments
- @FerdiansyahmulyadiPosted 2 months ago
Hi @Tasin269,
I think your solution looks good, except the size of each box differs from the design, other than that, everything is okay, good luck.
0 - P@whiteriver-devSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
Proud of how quickly I grasped React. I really enjoyed the modular aspect of React, and the organized structure of Sass.
Next time I will try implement Redux.
What challenges did you encounter, and how did you overcome them?The project took me quite a while as I was learning React on the go. Some of the main problems I encountered were UseStates and passing props through different components. However, by solving these solutions I've gotten a much better grasp of React.
What specific areas of your project would you like help with?What I can do better in React. Including hot tips and tricks. I really enjoy React and want to become proficient at it.
@FerdiansyahmulyadiPosted 2 months agoHi Igor,
Your solution looks good, I like it!
0 - @kassahunAmdieSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
In this project, I am delighted to have taken the plunge into learning Sass and implementing it in this challenge. I still have more to learn about it, although I did an in-depth study about it. I am surprised by how much it helped me organize my workflow.
What challenges did you encounter, and how did you overcome them?The challenge I encountered and haven't yet is the output from sass. It seems especially the media queries seem to be everywhere and certain sections are not where I want them to be. For example the utilities section is not where I want it to be. I will keep on working on it.
@FerdiansyahmulyadiPosted 3 months agoHi @kassahunAmdie,
Your solution look good, keep the good work.
0 - P@Bobstyle23Submitted 3 months agoWhat are you most proud of, and what would you do differently next time?
Getting used to more helpful features of sass/scss.
What challenges did you encounter, and how did you overcome them?I guess the provided design file and solutions design files have certain differences that are hard to match. If you guys see some differences or mismatches you can leave a comment or just check the deployed page.
What specific areas of your project would you like help with?I see my sass/scss code is a bit messy, I will refactor and reduce of course, but I would love to get some suggestions regarding cleaner sass/scss styles from someone who has a real grasp of it.
@FerdiansyahmulyadiPosted 3 months agoHi @Bobstyle23,
Your solution looks good, keep up the good work.
0 - @LouckoomSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
Proud of : Using custom grid for the desktop part. Next time : Improve on intermediate media query tablet size maybe.
What challenges did you encounter, and how did you overcome them?Not much, just some text that aren't properly aligned. I kind of didn't fix this issue.
What specific areas of your project would you like help with?Not much, i'm improving on the responsive side, that cool !
@FerdiansyahmulyadiPosted 3 months agoHi @Louckoom,
The solution looks great, the grid is responsive and fits nicely in mobile and desktop views. I suggest making the grid template size with a fixed minimum size so that the cards do not look big in desktop view, thus making your solution look more closely as the design. Good luck!
Marked as helpful1 - @BaloaCSubmitted over 1 year ago@FerdiansyahmulyadiPosted 3 months ago
Hi @BaloaC,
Your solution almost looks like the design, I think you can make it look better by setting the maximum width for each card. Good luck!
0 - @FahatmahSubmitted almost 2 years ago@FerdiansyahmulyadiPosted 3 months ago
Hi @fahatmah,
Excellent!, the design and the solution look same, keep up the good work!
Marked as helpful0 - @MuftiBinHabibSubmitted 3 months ago@FerdiansyahmulyadiPosted 3 months ago
The solution looks so different from the design, that I think it would be better if you write the HTML to use semantic HTML for code clarity. try to adopt a mobile-first design, then move up to tablet and desktop screen.
0 - P@SclataSubmitted 4 months agoWhat are you most proud of, and what would you do differently next time?
This was a fun one. Hadn't worked with transitions before, and modifying the transition-duration for different states/events was an interesting challenge. Learned some more about the @use directive in SASS/SCSS and how the "importing" of other modules interacts with code written in a file itself (how all this works relative to the cascade). Need to give some more thought to how this structuring will affect the end product and move all styling declarations out to modules if I'm going to approach the solution that way.
What challenges did you encounter, and how did you overcome them?The background-color and color transition duration (for the buttons) being different for different events/states. Very interesting and neat effect. Took me a while to figure out how to properly apply an overriding transition duration with the :not(:hover) pseudo-selector. Took much googling and looking through the MDN docs.
What specific areas of your project would you like help with?I think I accomplished everything I was trying to with this project.
@FerdiansyahmulyadiPosted 4 months agoHi @sclata,
this looks good, I think it would be better if you could set the margin to go down slightly. Other than that, good job!
0 - P@mykola-pyrohSubmitted 4 months ago@FerdiansyahmulyadiPosted 4 months ago
Hi @mykola-pyroh,
This looks good, the blog preview card looks the same as the design. I think we can improve it in some ways:
- Make a container to wrap the blog preview card
- Code the container with max-width and max-height to match the width and height of the preview card.
- Set the margin of the container, so that the top, bottom, left, and right stay center.
Here is my .container code, by the way, I use SCSS to complete this challenge: .container { max-width: rem-calc(384); max-height: rem-calc(522); padding: 0 0; margin: rem-calc(219) auto; }
Good luck!
0 - @Katte18Submitted about 2 years ago@FerdiansyahmulyadiPosted 4 months ago
It looks good, but I think you can set the background color from the body tag, create a main container, and set the margins top and bottom with the same value, left-right auto. Good luck!
0
