@BrianUribe6
Posted
hi, I liked the animations it looks pretty cool. A small bug is that at screen sizes < 318 the background dissappears. You should consider adding the background starting from the smallest screen size. Also add a padding on the whole container to prevent the card from hitting the screen edges as the screen gets smaller.
Finally, I noticed you are using using "xs" breakpoints, I'm assuming this is for targetting the smallest screen size. Although there are places where this is useful, in general you should start building from the smallest screen size, and as the screen gets bigger you override or add new properties. if you follow this approach, you'll notice you will use fewer CSS classes, and have an easier time overall.
Marked as helpful
@dimar-hanung
Posted
@BrianUribe6
Hi, Thanks for feedback and review my code solution!, "in general you should start building from the smallest screen size" ah yes, i forgot if tailwind is mobile first, This is a new insight to try slicing with a mobile first approach.
"I'm assuming this is for targetting the smallest screen size." you are right, at that time I was confused whether it is better to reduce the padding or remove the background
thanks for insight!