Space tourism mobile-first solution using CSS Grid and Flexbox
Design comparison
Solution retrospective
- I wasn't able to figure out how I can push the images on the "Technology" page to the very right of the page to make it touch the border of the browser. How can I achive that?
- I have some weird bugs on the tabs in the "Technology" page and I had to make some minor adjustments in the code to make it work. Does anybody have an idea about how can I tackle that? (Sometimes it doesn't update the content and the image)
Community feedback
- @ArtequePosted 9 months ago
Hello @alonsovzqz, congratulations on finishing the challenge though π I had a hard time with this one too π
First Question: the picture alignment
The
100%
width in theindex.css
file on line 797 takes up all the available space, which is whyjustify-self: flex-end
does not trigger. Changing the width tofit-content
solves the problem, but you need to check the behavior in the mobile view. You can also add this as a media query though:@media screen and (max-width: /*your desired width*/) { .grid-container--technology > picture { grid-area: image; max-width: 100%; /* width: 100%;*/ justify-self: flex-end; width:fit-content; } }
Second question: "the bugs"
I think the problem with this project is the fact that there are 3 "Sliders" Style and as a dev, I tried to use as less Js as possible and css. That's why I opted for self-contained class names, at least I tried, with no nesting with the parents' element. Like so, I could place the element of the sliders where I wanted after displaying a grid element. I was too lazy and didn't take that much time analyzing the template designs... π
You did a good job though. Just take time and read your code again, try to use global reusable classes.
Marked as helpful0@alonsovzqzPosted 9 months agoThanks a lot @Arteque !! I made the change and it actually worked!! Just like magic. And yes I must admit this is a quite tricky one and given the structure of the page I decided to go with another solution that was to wrap the indicators and the "content" (or description/text) in a wrapper. But that brought another different problem with the javscript file and the navigation. I think so far it works but I need to take my time to take a look into it really deep since need to fix it but without lose the current functionality from the other pages. Once again, thanks man π
0@ArtequePosted 9 months ago@alonsovzqz. Oh perfect. I'm happy that I could help you solve this problem. Keep up the good work though
1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord