Saiful Islam• 390
@saifuldt
Submitted
iamporrrrrrrrr
@iamporrrrrrrrr
All comments
- iamporrrrrrrrr• 190
@iamporrrrrrrrr
Posted
May be change the font size and the flex layout a little bit, but, other than that, this is excellent
0 - sung92• 450
@sung92
Submitted
iamporrrrrrrrr• 190@iamporrrrrrrrr
Posted
Nice Solution
0 - Mario Christofell• 180
@mariotbg11
Submitted
What are you most proud of, and what would you do differently next time?
What challenges did you encounter, and how did you overcome them?
What specific areas of your project would you like help with?
iamporrrrrrrrr• 190@iamporrrrrrrrr
Posted
That looks perfect! Great Job!
0 - Randomgituser69• 230
@Randomgituser69
Submitted
What are you most proud of, and what would you do differently next time?
At least the container is on the center. Though next time i may do it without following a tutorial as i did follow a tutorial but failed since the styling did not work to a button for some reason and other stuff.
What challenges did you encounter, and how did you overcome them?
Not sure if i did but even if i did, i might've forgotten or not understood properly how did i overcome them or
What specific areas of your project would you like help with?
How can i fix the button styling and success modal styling? It just didn't work
iamporrrrrrrrr• 190@iamporrrrrrrrr
Posted
I believe the styling problem is from the HTML code. The div tag has no closing bracket. Maybe you could also do some more styling to make
0 - Luis• 220
@luisv79
Submitted
What are you most proud of, and what would you do differently next time?
I was able to write the code by myself from scratch, with more practice I will do it faster.
What challenges did you encounter, and how did you overcome them?
how to make the css element disappear and appear through javascript
What specific areas of your project would you like help with?
I'm always open to constructive criticism.
iamporrrrrrrrr• 190@iamporrrrrrrrr
Posted
The alignments and colors seem a little bit off. Other then that, it is a great design! For the hiding element using JS, I use a class name hidden to handle this by setting display: none in style.css. Then, in JS I toggle the class .hidden to make the elements appear and disappear. You can use my code as a reference. :)
https://github.com/iamporrrrrrrrr/Article-Preview-Component
Marked as helpful
0 - Stefan Grobler• 150
@StefanGrobler78
Submitted
What are you most proud of, and what would you do differently next time?
This was a good grid challenge, I proud that I remembered how to use grid for layout and got it to work without look up.
What challenges did you encounter, and how did you overcome them?
It was fine had think a bit here and there.
What specific areas of your project would you like help with?
Controlling the high in the grid layout. Please Have a look at my CSS and suggest what you would have done differently!
iamporrrrrrrrr• 190@iamporrrrrrrrr
Posted
I think the ratio of the width of each box is quite different from the design. From what I see, the columns are divided into four equal parts. Since grid-row-templates is set to auto, the height of the grid is adjusted to fit the content.
0 - Stefan Grobler• 150
@StefanGrobler78
Submitted
What are you most proud of, and what would you do differently next time?
Got It done next time only flexbox
What challenges did you encounter, and how did you overcome them?
Different colors for top border made use of utility classes
What specific areas of your project would you like help with?
no help thank you
iamporrrrrrrrr• 190@iamporrrrrrrrr
Posted
Well done! :)
0 - Imerzion• 120
@Imerzion
Submitted
What are you most proud of, and what would you do differently next time?
Happy with the overall outcome on this one. Feel like I'm improving overall. Would advise myself in the future to take a little more time with the HTML structure as I had to re-jig it a little bit to make it work at desktop.
What challenges did you encounter, and how did you overcome them?
As stated above, had some trouble with my HTML structure when moving from mobile over to desktop. Had to re-write some of my HTML, but was a pretty simple fix overall.
Had some trouble with a gap displaying between the image at desktop, but Googled a fix for that pretty quickly.
What specific areas of your project would you like help with?
Still feel like I'm having trouble when it comes to declaring widths of my containers.
Design appears to have a more vertical feel as opposed to my landscape outcome. Not sure what is causing such the drastic difference.
iamporrrrrrrrr• 190@iamporrrrrrrrr
Posted
The solution included semantic HTML. Maybe the dimensions of the boxes, texts, and paddings on the desktop version could be changed to more closely resemble the original design. The mobile version looks great. Overall, it is an excellent solution, and you have done well in creatin g a responsvie web design.
0
