Elena Barden• 170
@metehnay
Submitted
i dont know why screenshot didn't render input range button. :l
Harmanjot Singh
@arkharman12
All comments
- Harmanjot Singh• 985
@arkharman12
Posted
A few things I have noticed:
- Looks like "Start my trial" button is missing a hover state?
- Mobile layout is not matching with the given design. Especially the billing area.
- There is no margin around content on 375px.
Good luck!
0 I've been studying all week on how to use API's and manipulate json-data. So this exercise was a nice test to see where I am.
Harmanjot Singh• 985@arkharman12
Posted
You might have to use
fetch("url", {cache: "no-cache"})for quickly getting the new advice.Marked as helpful
0- Shashree Samuel• 9,260
@shashreesamuel
Submitted
Feel free to leave any feedback on my solution, I don't have a certain part of the solution that need s feedback so always happy to improve and get better in this journey.
Cheers Happy coding 👍
Harmanjot Singh• 985@arkharman12
Posted
Add cursor to pointer over the whole accordion, you never know where the user is going to start opening! 😁
0 - Erick F. Felix• 265
@efs0-cod3
Submitted
Your feedback is appreciated. Thanks in advance!
Harmanjot Singh• 985@arkharman12
Posted
I would say sizing, spacing and positioning of the overall site could be improved to better match with the given design.
Nice work overall and good luck!
1 - Kenn Bach• 140
@kennbach
Submitted
Hello Mentors,
Thank you for taking the time to review my project. Please feel free to make any comments and suggestions. All feedback is welcome.
Cheers!
Harmanjot Singh• 985@arkharman12
Posted
A few things I have noticed in your solution:
- When the mobile menu is opened, it's not ideal to let the user scroll. Also, if the window width is increased while the menu is still open, half of the logo disappears.
- Change the color of the input field. I cannot see what I am typing.
Nice work overall and good luck!
Marked as helpful
0 - PGraeme Byrne• 390
@grmbyrn
Submitted
I think it looks okay, but let me know if you spot anything!
Harmanjot Singh• 985@arkharman12
Posted
Success and error message is not properly placed on desktop screens. Otherwise, looks good!
0 - Mohamed Saleh• 110
@MohamedSalehjr
Submitted
Hi any suggestions on how to create blur affect around button when user is hovering on it? Also any other feedback would be much appreciated.
Harmanjot Singh• 985@arkharman12
Posted
- Use drop-shadow for the hover effect
- Advice delivery rate is quite slow
- Fixed width and height of the box is more appropriate for UX
Good luck!
1 - Rashmi Raj• 30
@rashmiraj513
Submitted
I think I have implemented this correctly.
Harmanjot Singh• 985@arkharman12
Posted
- Button is not centered in the box.
- You might have to use
fetch("url", {cache: "no-cache"})for quickly getting the new advice.
0 - liezl• 190
@liezsm
Submitted
Hi all just finished this challenge any feedbacks tips suggesstions advices will do 🥰. I'm curious tho how can I achieve that glowing box shadow when button is hovered, I tried it myself but my btn is clip-path so it doesn't show when hovered but when i removed clip-path it works, any idea how to do it?
Harmanjot Singh• 985@arkharman12
Posted
Make the container with fixed width and height because jumping around of text is not best for UX. The position of the button should also be fixed in one place. Good luck!
Marked as helpful
0 - Mariem Bchir• 140
@Mabchir
Submitted
It took a long time to set up this initial page which barely matched the given design. I have a feeling that I am approaching this in a very non-strategic way :
- I determine the big elements and the smaller ones then I start building...
I would love to hear some really critical feedback on this non-responsive desktop version!
Harmanjot Singh• 985@arkharman12
Posted
Try to make the page responsive.
0 - Mohamed Dwedar• 40
@mohameddwedar689
Submitted
All feedback is welcome 😍
Harmanjot Singh• 985@arkharman12
Posted
Your button requires two clicks to get to the next advice.
Marked as helpful
0 - Karishma Garg• 1,010
@krrish105
Submitted
Hey Guys, I just finished the Rock, Paper, Scissors Game Challenge. I would like you all to see it and let me know how I can improve it. Let me know my mistakes. Thank you!.
Harmanjot Singh• 985@arkharman12
Posted
- Scoreboard should not go in negative. Keep it at 0 if the player loses in the first game.
- Bottom half of the viewport is cut out at 1440px.
Marked as helpful
0 @Adriana2710
Submitted
Hello everyone! I've just completed the "QR code component" challenge and to be honest I don't know what I did wrong but looks like the size of the page is different. I'm a newbie in Front End, just finished one course of HTML5 and CSS basics, if someone has some time could you please give some tips on what I did wrong, any feedback and suggestions on how I can improve are very welcome!
Harmanjot Singh• 985@arkharman12
Posted
Impressive work! 👏
0- Pstephmunez• 435
@stephmunez
Submitted
Hey everyone!
I just completed another challenge. 🤘🏽
Feedback is always welcome. Please let me know of any issues you may find and/or how I can improve my code :)
Happy coding!
Harmanjot Singh• 985@arkharman12
Posted
Great solution! I wish some of these challenges were also available for free users. It's a nice and sleek-looking one.
Good job! 👏
1 - Genesis Gran• 270
@GenesisGran
Submitted
This is my solution to the Base Apparel Coming Soon Page. Feel free to give any suggestion or pointing out any mistakes I made.
Harmanjot Singh• 985@arkharman12
Posted
- Double-check your email validation e.g. abc@abc format is not a valid email address.
- Give the user some kind of feedback if the input field is empty or validation is passed.
Good luck!
0 - Nnenna Miriam Udefi• 300
@Nnenna-udefi
Submitted
Any remark on the grid code would be highly appreciated.
Harmanjot Singh• 985@arkharman12
Posted
Areas of improvement:
- Media query breakpoint
- Sizing
- Spacing
Good luck!
Marked as helpful
1 - PDarrick Fauvel• 340
@DarrickFauvel
Submitted
Hi everybody, I would definitely love to receive any feedback you have. Did I miss anything? Could I do something better? Thanks :D
Harmanjot Singh• 985@arkharman12
Posted
This is great! 👏
1 - Kamasah-Dickson• 5,610
@Kamasah-Dickson
Submitted
Hello Frontendmentors. I underestimated this design, I thought it is so so Trivia but it drained me. It made me learn CSS grid so that I can finish It off quickly. Trying this solution over and over again was so much boring and stressful. besides am able to at least do my best I could. But please do not forget to comment and like I really need your suggestions on this :)
Harmanjot Singh• 985@arkharman12
Posted
This still quite needs a lot of work in terms of positioning:
- Fix the spacing for the second section. Buttons are also not as same as the design. The individual boxes create spacing between each other when going responsive. They should all stick together with each other.
- The content overflows for the cherry and orange section when also going responsive.
- Client testimonial section is not centered horizontally.
- If I try to increase the window width while the mobile menu is open. The contact button goes in a weird position.
Good luck!
0 - Jakub Radziejewski• 120
@jakubinhoo
Submitted
Any feedback appreciated :)
Harmanjot Singh• 985@arkharman12
Posted
Nice work!
1 - AditNovadianto• 1,040
@AditNovadianto
Submitted
I will be happy, to hear any feedback and suggestion
Harmanjot Singh• 985@arkharman12
Posted
Use multiple media queries for a better content transition. Snippets section could be improved positioning-wise.
Marked as helpful
1 - Nisha Jha• 100
@Nisha213-mar
Submitted
Harmanjot Singh• 985@arkharman12
Posted
Try solving this problem with CSS grid instead. You will achieve better layout and responsiveness.
0 - Ashraf Elshaer• 120
@AshrafElshaer
Submitted
please give a feedback about my code and how to improve it , I'll appreciate it Thnaks
Harmanjot Singh• 985@arkharman12
Posted
That mobile menu animation is very interesting!
1 - Anosha Ahmed• 9,340
@anoshaahmed
Submitted
Would appreciate any advice on where and how to get better. Thank you x
Harmanjot Singh• 985@arkharman12
Posted
Tested on iPhone 13 with Safari, the card flip animation seems to be having trouble. The card is already flipped and the text appears backward.
Marked as helpful
1 - Zybartas• 480
@ZybartasRingys
Submitted
Hello, this was a really interesting and tough challenge to do it took a while to finish, this challenge was my first big project for which i used Bootstrap as a CSS framework and also a first project where i did mobile first development so it was really challenging at the beginning of project. This challenge helped me to learn a lot about bootstrap and responsive development , JS also was a big challenge since i have tried to make it on my own, still need to figure out some functionality.
Any feedback would be appreciated !
Harmanjot Singh• 985@arkharman12
Posted
- Positioning of the hamburger icon needs an adjustment.
- Mobile menu should automatically close if the screen width keeps increasing.
- "Our Collection" section messes up at 1390px width.
- "Our Commitment" section image keeps moving which makes the person's face hidden.
- "Free Shipping" truck icon also seems squished.
- Accordions don't open or close if I click on the icon itself.
- Create my plan should only work if all of the criteria are selected.
Otherwise, nice work!
Marked as helpful
0
