@dimitrisdr
@samir-Deve
Submitted
What are you most proud of, and what would you do differently next time?
I am proud of getting tougher day by day in frontend pro and my consistency, I may code less but don't cut it completely !
What challenges did you encounter, and how did you overcome them?
If you get used to using Docs you don't encounter challenges !
What specific areas of your project would you like help with?
any suggestion is appreciated
@dimitrisdr
Posted
Nice work!! It's just like the original! I think that if you put some transitions to your hover states the UI experience will be improved a lot! Keep up!
What are you most proud of, and what would you do differently next time?
I was able to do the mobile version but due to inexperience I am not able to do the desktop version, will need to retry.
What challenges did you encounter, and how did you overcome them?
I did the sliders using only js and css classes. But I realized I could do it another way which allows for animations.
What specific areas of your project would you like help with?
I realised my planning with Grid isn't so good, so if there are any tips with grid I can use please inform me.
My slider is made with only js and css but it does not allow for animations. Maybe I can have tips on how to do that ?
For people who are reading the code Header = first row
Header__section2 = "Discover innovate ways to decorate page'
header__section3 = image + "About our furniture"
picture__section4 = image
@dimitrisdr
Posted
You could do a basic layout by learning grid-template-areas. Search in youtube for videos related to this! keep up the good work!
@Sazid99246
Submitted
@dimitrisdr
Posted
Nice work! Keep up the good work!
Marked as helpful
@dimitrisdr
Posted
Nice work! It is so close to the original design! Here is some suggestions: 1)You could also set justify-content: center to your card__footer in order to center horizontally the elements. 2) Also for very small screen sizes you can set the card_body--subsection to flex-direction:column and text-align: center to your hole card for a better ui experience! Keep up the good work!!
@sudhanshusingh-g
Submitted
@dimitrisdr
Posted
Nice work! It is almost identical to the original! I think that if you remove the width:970px from .product-container you will prevent undesired overflows. Also on your media-query(max-width:768px) you could set a max width: 500px to the main content for preventing the image being stretched a lot. Hope this helps! Keep up the good work!
Marked as helpful
@TheBeyonder616
Submitted
What are you most proud of, and what would you do differently next time?
I added lazy-loading, smooth scrolling, and popup using Observer API
What challenges did you encounter, and how did you overcome them?
picking between background image or image of the Hero section
What specific areas of your project would you like help with?
Any advice would do
@dimitrisdr
Posted
Nice work! It is very close to the original design! I think that if you put some transitions for your hover states, the UI experience will be drastically improved. Moreover you should try to use the grid layout for your trending articles! It will save you some lines of code! Keep up the good work! Happy coding!
Marked as helpful
@TheBeyonder616
Submitted
What are you most proud of, and what would you do differently next time?
I was able to create a valid form
What challenges did you encounter, and how did you overcome them?
I had to remove the required from the html input, hence my code can function properly
What specific areas of your project would you like help with?
Any advice would do
@dimitrisdr
Posted
Very Nice work! Love the styling and the functionality Some Suggestions:
Marked as helpful
@tannguyenk3
Submitted
@dimitrisdr
Posted
Nice work! Love the animations when the answers appear and disappear! I think that the font size of the answers as well as the line-height could be a bit bigger! Keep up the good work! #happy coding!
@konradbaczyk
Submitted
What challenges did you encounter, and how did you overcome them?
Work with radio buttons was great experience. For styling them I had to read some MDN.
@dimitrisdr
Posted
Very nice work! It's really close to the original design! I think that you should consider increasing the space between the star and the other content for perfection!! Keep up the good work! #happy coding
@antimatterhut
Submitted
What are you most proud of, and what would you do differently next time?
I guess I am proud of the fact that I enjoyed writing js for the first time doing this project.
What challenges did you encounter, and how did you overcome them?
I had alot of issues with integrating the custom button, along with the other 5%, 10% tip buttons.
What specific areas of your project would you like help with?
I would like help with any and everything if possible. My JS is quite long and messy. Theres probably a much better way to write what I did.
@dimitrisdr
Posted
Congratulations for your design! Love the style! It's almost identical to the original one! I think that you should consider updating your inputs not to include negative values!! hope this helps! #happy coding!
Marked as helpful
@Kapkrazy
Submitted
What are you most proud of, and what would you do differently next time?
It was a struggle. I made it out alive.
What challenges did you encounter, and how did you overcome them?
The css tricked me up a bunch as well the Javascript.
@dimitrisdr
Posted
Congratulations for getting this project as close as the original one! I have some suggestions for taking this project to perfection. Since you decided to make the entire background of each section with the color of its respective category, It could be helpful using: linear-gradient(180deg, var(--clr-of-this-timeCat),var(-- --dark-blue) 50%) for hiding the undesired background color of the cropped edges. Also you could use li tags for giving your .button-group semantic meaning. Hope this helps :) ! Very nice work!
Marked as helpful
@AzerJradi
Submitted
@dimitrisdr
Posted
Nice work! well done for completing the project. Keep up the good work! I have some suggestions: I think that you should increase the space around your .bigPart tags. This will make text a lot cleaner around. I would also suggest making a list like:
<ul class='bigPart-container> <li class="bigPart">.......</li> <li class="bigPart">.......</li> <li class="bigPart">.......</li> </ul>and then apllying to your list: .bigPart-container {list-style:none}. This would give semantic meaning to you html and make it readable for screen readers.
hope this helps! #happyCoding
Marked as helpful
