P
@miedzygalaktycznygitSubmitted 4 months ago
Latest solutions
grid testimony with grid!
PSubmitted 5 months agoIf you have experience creating projects with the cube methodology, any tips on structuring my code to adhere to it would be appreciated :) but any help is appreciated!
Product preview responsive design using media query
PSubmitted 5 months agoCould you please review my CSS code and provide feedback on the following aspects:
Code structure and organization Cross-browser compatibility Responsive design Flexbox usage Performance Styling consistency Best practices
I'm particularly interested in any suggestions for improving the layout and ensuring it works consistently across different browsers and devices."
Social Media solution using flexbox
PSubmitted 6 months agoI would like advice on good coding practice for both html and css. Thank you! :) *any other criticism is accepted!
Qr-code-component using flexbox
PSubmitted 6 months agoI would like to know if I have good coding or if I should change some things for future solutions. Thank you :).
Latest comments
- P@Ramirez-ChristopherPosted 4 months ago
everything looks great! some styles are used multiple times, maybe consider making them variables.
0 - P@christianb3llSubmitted 5 months agoWhat specific areas of your project would you like help with?
I got really stuck on getting the share icon to work for both mobile and desktop. I thought of doing two separate components and hiding one but that felt a bit hacky. Maybe I'm overthinking it. Curious to see how other people did it.
- P@Makc240305Submitted 5 months ago
- @CSENGGGEEKSubmitted 5 months agoP@Ramirez-ChristopherPosted 5 months ago
The line-height for both .tagline and h1 is set to 25%, which is unusual and might cause readability issues. Consider using a more conventional line height (e.g., 1.2 to 1.5 for headings and 1.5 to 1.8 for paragraphs).
Marked as helpful0 - @Shaharyar16Submitted 5 months agoP@Ramirez-ChristopherPosted 5 months ago
Consolidate Common Styles: Move common styles outside of media queries to avoid redundancy.
Sizing and Margin Reset: The margin: 0% in the universal selector is unusual. It should be margin: 0 to remove all margins.
Marked as helpful0 - P@anamaydevSubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
I’m proud that I completed this project in under an hour and that I’m becoming increasingly comfortable with Flexbox. Additionally, I made a conscious effort to avoid using px units, as suggested by my peers, and instead opted for more responsive units. I’m also really pleased with how the transition effect on the buttons turned out.
What challenges did you encounter, and how did you overcome them?I didn’t encounter any challenges during this project. Everything went smoothly.
What specific areas of your project would you like help with?I would appreciate feedback on my CSS code and any tips for improving it.
P@Ramirez-ChristopherPosted 6 months agoTo improve maintainability, especially with repeating colors and spacings, you might consider using CSS variables:
Marked as helpful0
