Lo más difícil de construir el proyecto fue hacer que sea responsive. Estuve mucho tiempo para adaptarlo en las medidas que me pedía, y que el contenido no se saliera de los márgenes, y demás. El uso de Medias Queries fue lo más complejo, pero fue un aprendizaje hermoso. Y también lo más complejo, pero divertido, fue el tema de estilos. No soy tan creativo para el diseño, pero aprendí mucho sobre eso, y quiero seguir aprendiendo. Cabe destacar que es mi primera página terminada, y subida al internet. Es un gran paso para mí el haber empezado y terminado este proyecto. ¡Obviamente cualquier comentario es más que bienvenido! Acepto consejos, críticas, lo que sea para seguir aprendiendo y mejorando. Gracias.
Vishal Biradar
@vishalbrdrAll comments
- @Josue-code-lanchSubmitted 9 months ago@vishalbrdrPosted 9 months ago
Se verá mucho mejor si aumenta el ancho del contenedor para la pantalla del escritorio. pruebe con ancho máximo: 600 px; por lo demás luce genial. buen trabajo👍
0 - @ashishmohanty10Submitted 9 months ago@vishalbrdrPosted 9 months ago
hey buddy, nice job! here are some improvements that will make your solution better:
- increase the padding for content
div
- use
<button>🛒Add to cart</button>
instead of<div>
- use
object-fit:cover;
property for image, it will fix the stretching of image.
Marked as helpful0 - increase the padding for content
- @AdrielMurraySubmitted over 2 years ago
Any thoughts? Where did I go wrong? What can I improve?
@vishalbrdrPosted over 2 years agoset img height to 100%. set product-info height same as the product-image height. or don't specify a height for any, just set the height of the container and let flex-box take care of that.
Marked as helpful0 - @EYRAM565Submitted over 2 years ago
feedback on how to improve the ui and responsiveness are welcome
@vishalbrdrPosted over 2 years agoHey, your solution works well! you can fix that dice with this code:
button { border: none; background-color: var(--NeonGreen); border-radius: 50%; margin-top: 30px; /* padding: 1rem; */ position: absolute; top: 85%; height: 3rem; width: 3rem; display: flex; align-items: center; justify-content: center; }
And wrap all your code into
<main></main>
.Marked as helpful0 - @waldekglazSubmitted over 2 years ago
Another part of my React education
Problem
Couldn't figure out how display only one related popup div after mouse click 🥺, so after click all "amount" divs are visible.
@vishalbrdrPosted over 2 years agoI used CSS for this one. use data attribute and access it in css, design amount box using ::before element, and change the opacity of ::before element on mouse hover on the bar-graph.
1 - @Jem0Submitted over 2 years ago
I'm having trouble creating the hover effect in the design. Any advice would be appreciated!
@vishalbrdrPosted over 2 years agoHello there, great job on the solution. You can use the box-shadow property for the hover effect.
2 - @vishalbrdrSubmitted almost 3 years ago
Any feedback is appreciated.
@vishalbrdrPosted almost 3 years agoThank you very much for your advice, really appreciated
0 - @shuree0331Submitted about 3 years ago
I would like to have feedback on my entire code and styling (semantic HTML and CSS elements). Thank you
@vishalbrdrPosted about 3 years agoIt's showing your README.md on Github pages link, something's wrong.
0 - @HuKu06Submitted about 3 years ago
Can Anyone Suggest Me other ways to optimize the site for different screen other than using @media... in CSS?
@vishalbrdrPosted about 3 years agouse em, rem, vw, vh rather than px.
Marked as helpful1 - @grtrspwSubmitted over 3 years ago
I got a problem to make a footer stay in center for mobile display (375px), so i decide to make it hidden. any solutions?
*sorry for my grammar mistakes because english isnt my first languange
@vishalbrdrPosted over 3 years agoHey! Great job, but it can be better, check out my video for solution of this challenge https://youtu.be/FIl8b9hVW54
0 - @chillcodemaoSubmitted over 3 years ago
Had some difficulty with finding the icons for the social media links. Eventually I just used png icons and used a transition to highlight them when focused. The solution works but I'm sure there has to be a more efficient way of writing the code. Suggestions?
@vishalbrdrPosted over 3 years agoyou can you fontawesome for icons, btw great job.
Marked as helpful0 - @mundevuxSubmitted over 3 years ago@vishalbrdrPosted over 3 years ago
Hey great job, but it can be better, check out my video https://youtu.be/FIl8b9hVW54
0