Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Space tourism mobile-first solution using CSS Grid and Flexbox

P
Alonso Vazquezβ€’ 160

@alonsovzqz

Desktop design screenshot for the Space tourism multi-page website coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

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

P
Ahmed Lemssiahβ€’ 360

@Arteque

Posted

Hello @alonsovzqz, congratulations on finishing the challenge though πŸ˜‰ I had a hard time with this one too πŸ˜•

The 100% width in the index.css file on line 797 takes up all the available space, which is why justify-self: flex-end does not trigger. Changing the width to fit-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;
  }
}

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 helpful

0

P
Alonso Vazquezβ€’ 160

@alonsovzqz

Posted

Thanks 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
P
Ahmed Lemssiahβ€’ 360

@Arteque

Posted

@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 GitHub
Discord logo

Join 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