@hoangtuanou
Submitted
@Sidd5arth
@hoangtuanou
Submitted
@Sidd5arth
Posted
Great! solution, but the plan selection is not working properly, it is not updating the UI on changing the plan. To reproduce it - Go the plan selection step and try to select a plan then try to toggle the switch to yearly-> see its not updating the price there.
@SuxrobOktamov
Submitted
@Sidd5arth
Posted
great! solution I like that you implemented that languages selection there!
What I can suggest for now is to improve upon the UI, there is lot of room for improvement there, also try to make component yourself without using third party library, if you really want to learn front-end development otherwise you did great!
Marked as helpful
@dannygorgon
Submitted
Do you think I have approached the problem properly?
What could I improve?
Is the README well written?
@Sidd5arth
Posted
@dannygorgon Hi there, overall its a good solution!, well if I look into the details you can decrease the width the card and increase the spread and opacity of the vertical shadow on the card.
Marked as helpful
@akshita1812-alt
Submitted
@Sidd5arth
Posted
A good start I guess, make use of good font I would suggest for now!
Marked as helpful
@SaumyaG1318
Submitted
@Sidd5arth
Posted
Great! but I think you should watch for padding and margin, the solution is lacking proper proportion, but you did a good job!
@MrSharma619
Submitted
@Sidd5arth
Posted
Great solution! but I think you should work on css a bit more! like you should give gap between the image and form input 😊
@nicky202
Submitted
@Sidd5arth
Posted
nice UI almost similar but you had not implemented you modal logic correctly, you must revert back to register page as you click on dismiss button also you have to show the entered email on the subscribed modal.
Marked as helpful
@hyeo151
Submitted
@Sidd5arth
Posted
hey nice work! but you had not implemented the validations correctly, you should use regex expression to check if the entered input is actual in email format or not