Mobile webpage for a refugee organization











up vote
7
down vote

favorite












For a school project, I've to create a website for a refugee foundation in Haarlem. But designing is one part, coding the actual design is a whole other world. The hamburger menu went pretty well and I'm very proud of that piece of code, but the mosaic tiles just don't feel right. It is designed mobile first, so the desktop version is completely forgotten at this point of the project. This is how it should look:



How it should look like



And well, it does look like it. But the code is a big mess in my opinion:






main {
display: grid;
grid-template-areas: 'one one one two two' 'three three four four four';
grid-template-columns: repeat(5, 1fr);
grid-gap: 45px;
height: 70%;
width: 87.5%;
margin: 2.5% auto;
}

.one,
.two,
.three,
.four {
display: grid;
grid-template-rows: 1fr 1fr 1fr;
font-family: functionbold;
font-size: 4em;
text-transform: uppercase;
color: white;
text-align: center;
}

.one {
grid-area: one;
background: url("../images/mozaik1.jpg") no-repeat center;
background-size: 175%;
}

.text {
grid-row: 2;
align-self: center;
}

.two {
grid-area: two;
background: url("../images/mozaik2.jpg") no-repeat center;
background-size: 250%;
}

.three {
grid-area: three;
background: url("../images/mozaik3.jpg") no-repeat left;
background-size: 275%;
background-position: -100px 0;
}

.four {
grid-area: four;
background: url("../images/mozaik4.jpg") no-repeat center;
background-size: 150%;
}

<main>
<div class="one"><span class="text">Samen Praten</span></div>
<div class="two"><span class="text">Samen Shoppen</span></div>
<div class="three"><span class="text">Samen Doen</span></div>
<div class="four"><span class="text">Samen Eten</span></div>
</main>





Could someone please help me with simplifying and beautifying this piece of code?










share|improve this question















migrated from stackoverflow.com Jun 15 at 8:36


This question came from our site for professional and enthusiast programmers.



















    up vote
    7
    down vote

    favorite












    For a school project, I've to create a website for a refugee foundation in Haarlem. But designing is one part, coding the actual design is a whole other world. The hamburger menu went pretty well and I'm very proud of that piece of code, but the mosaic tiles just don't feel right. It is designed mobile first, so the desktop version is completely forgotten at this point of the project. This is how it should look:



    How it should look like



    And well, it does look like it. But the code is a big mess in my opinion:






    main {
    display: grid;
    grid-template-areas: 'one one one two two' 'three three four four four';
    grid-template-columns: repeat(5, 1fr);
    grid-gap: 45px;
    height: 70%;
    width: 87.5%;
    margin: 2.5% auto;
    }

    .one,
    .two,
    .three,
    .four {
    display: grid;
    grid-template-rows: 1fr 1fr 1fr;
    font-family: functionbold;
    font-size: 4em;
    text-transform: uppercase;
    color: white;
    text-align: center;
    }

    .one {
    grid-area: one;
    background: url("../images/mozaik1.jpg") no-repeat center;
    background-size: 175%;
    }

    .text {
    grid-row: 2;
    align-self: center;
    }

    .two {
    grid-area: two;
    background: url("../images/mozaik2.jpg") no-repeat center;
    background-size: 250%;
    }

    .three {
    grid-area: three;
    background: url("../images/mozaik3.jpg") no-repeat left;
    background-size: 275%;
    background-position: -100px 0;
    }

    .four {
    grid-area: four;
    background: url("../images/mozaik4.jpg") no-repeat center;
    background-size: 150%;
    }

    <main>
    <div class="one"><span class="text">Samen Praten</span></div>
    <div class="two"><span class="text">Samen Shoppen</span></div>
    <div class="three"><span class="text">Samen Doen</span></div>
    <div class="four"><span class="text">Samen Eten</span></div>
    </main>





    Could someone please help me with simplifying and beautifying this piece of code?










    share|improve this question















    migrated from stackoverflow.com Jun 15 at 8:36


    This question came from our site for professional and enthusiast programmers.

















      up vote
      7
      down vote

      favorite









      up vote
      7
      down vote

      favorite











      For a school project, I've to create a website for a refugee foundation in Haarlem. But designing is one part, coding the actual design is a whole other world. The hamburger menu went pretty well and I'm very proud of that piece of code, but the mosaic tiles just don't feel right. It is designed mobile first, so the desktop version is completely forgotten at this point of the project. This is how it should look:



      How it should look like



      And well, it does look like it. But the code is a big mess in my opinion:






      main {
      display: grid;
      grid-template-areas: 'one one one two two' 'three three four four four';
      grid-template-columns: repeat(5, 1fr);
      grid-gap: 45px;
      height: 70%;
      width: 87.5%;
      margin: 2.5% auto;
      }

      .one,
      .two,
      .three,
      .four {
      display: grid;
      grid-template-rows: 1fr 1fr 1fr;
      font-family: functionbold;
      font-size: 4em;
      text-transform: uppercase;
      color: white;
      text-align: center;
      }

      .one {
      grid-area: one;
      background: url("../images/mozaik1.jpg") no-repeat center;
      background-size: 175%;
      }

      .text {
      grid-row: 2;
      align-self: center;
      }

      .two {
      grid-area: two;
      background: url("../images/mozaik2.jpg") no-repeat center;
      background-size: 250%;
      }

      .three {
      grid-area: three;
      background: url("../images/mozaik3.jpg") no-repeat left;
      background-size: 275%;
      background-position: -100px 0;
      }

      .four {
      grid-area: four;
      background: url("../images/mozaik4.jpg") no-repeat center;
      background-size: 150%;
      }

      <main>
      <div class="one"><span class="text">Samen Praten</span></div>
      <div class="two"><span class="text">Samen Shoppen</span></div>
      <div class="three"><span class="text">Samen Doen</span></div>
      <div class="four"><span class="text">Samen Eten</span></div>
      </main>





      Could someone please help me with simplifying and beautifying this piece of code?










      share|improve this question















      For a school project, I've to create a website for a refugee foundation in Haarlem. But designing is one part, coding the actual design is a whole other world. The hamburger menu went pretty well and I'm very proud of that piece of code, but the mosaic tiles just don't feel right. It is designed mobile first, so the desktop version is completely forgotten at this point of the project. This is how it should look:



      How it should look like



      And well, it does look like it. But the code is a big mess in my opinion:






      main {
      display: grid;
      grid-template-areas: 'one one one two two' 'three three four four four';
      grid-template-columns: repeat(5, 1fr);
      grid-gap: 45px;
      height: 70%;
      width: 87.5%;
      margin: 2.5% auto;
      }

      .one,
      .two,
      .three,
      .four {
      display: grid;
      grid-template-rows: 1fr 1fr 1fr;
      font-family: functionbold;
      font-size: 4em;
      text-transform: uppercase;
      color: white;
      text-align: center;
      }

      .one {
      grid-area: one;
      background: url("../images/mozaik1.jpg") no-repeat center;
      background-size: 175%;
      }

      .text {
      grid-row: 2;
      align-self: center;
      }

      .two {
      grid-area: two;
      background: url("../images/mozaik2.jpg") no-repeat center;
      background-size: 250%;
      }

      .three {
      grid-area: three;
      background: url("../images/mozaik3.jpg") no-repeat left;
      background-size: 275%;
      background-position: -100px 0;
      }

      .four {
      grid-area: four;
      background: url("../images/mozaik4.jpg") no-repeat center;
      background-size: 150%;
      }

      <main>
      <div class="one"><span class="text">Samen Praten</span></div>
      <div class="two"><span class="text">Samen Shoppen</span></div>
      <div class="three"><span class="text">Samen Doen</span></div>
      <div class="four"><span class="text">Samen Eten</span></div>
      </main>





      Could someone please help me with simplifying and beautifying this piece of code?






      main {
      display: grid;
      grid-template-areas: 'one one one two two' 'three three four four four';
      grid-template-columns: repeat(5, 1fr);
      grid-gap: 45px;
      height: 70%;
      width: 87.5%;
      margin: 2.5% auto;
      }

      .one,
      .two,
      .three,
      .four {
      display: grid;
      grid-template-rows: 1fr 1fr 1fr;
      font-family: functionbold;
      font-size: 4em;
      text-transform: uppercase;
      color: white;
      text-align: center;
      }

      .one {
      grid-area: one;
      background: url("../images/mozaik1.jpg") no-repeat center;
      background-size: 175%;
      }

      .text {
      grid-row: 2;
      align-self: center;
      }

      .two {
      grid-area: two;
      background: url("../images/mozaik2.jpg") no-repeat center;
      background-size: 250%;
      }

      .three {
      grid-area: three;
      background: url("../images/mozaik3.jpg") no-repeat left;
      background-size: 275%;
      background-position: -100px 0;
      }

      .four {
      grid-area: four;
      background: url("../images/mozaik4.jpg") no-repeat center;
      background-size: 150%;
      }

      <main>
      <div class="one"><span class="text">Samen Praten</span></div>
      <div class="two"><span class="text">Samen Shoppen</span></div>
      <div class="three"><span class="text">Samen Doen</span></div>
      <div class="four"><span class="text">Samen Eten</span></div>
      </main>





      main {
      display: grid;
      grid-template-areas: 'one one one two two' 'three three four four four';
      grid-template-columns: repeat(5, 1fr);
      grid-gap: 45px;
      height: 70%;
      width: 87.5%;
      margin: 2.5% auto;
      }

      .one,
      .two,
      .three,
      .four {
      display: grid;
      grid-template-rows: 1fr 1fr 1fr;
      font-family: functionbold;
      font-size: 4em;
      text-transform: uppercase;
      color: white;
      text-align: center;
      }

      .one {
      grid-area: one;
      background: url("../images/mozaik1.jpg") no-repeat center;
      background-size: 175%;
      }

      .text {
      grid-row: 2;
      align-self: center;
      }

      .two {
      grid-area: two;
      background: url("../images/mozaik2.jpg") no-repeat center;
      background-size: 250%;
      }

      .three {
      grid-area: three;
      background: url("../images/mozaik3.jpg") no-repeat left;
      background-size: 275%;
      background-position: -100px 0;
      }

      .four {
      grid-area: four;
      background: url("../images/mozaik4.jpg") no-repeat center;
      background-size: 150%;
      }

      <main>
      <div class="one"><span class="text">Samen Praten</span></div>
      <div class="two"><span class="text">Samen Shoppen</span></div>
      <div class="three"><span class="text">Samen Doen</span></div>
      <div class="four"><span class="text">Samen Eten</span></div>
      </main>






      html css






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited Jun 15 at 13:49









      200_success

      127k15148412




      127k15148412










      asked Jun 15 at 8:34







      Melvin Idema











      migrated from stackoverflow.com Jun 15 at 8:36


      This question came from our site for professional and enthusiast programmers.






      migrated from stackoverflow.com Jun 15 at 8:36


      This question came from our site for professional and enthusiast programmers.
























          1 Answer
          1






          active

          oldest

          votes

















          up vote
          1
          down vote













          This question is difficult to answer because art is subjective but if this was my site, I would limit the gutters between the images to be slightly closer together (perhaps 2 maybe 3 ems) and being sure to keep them extending out to the width of the boxed layout.



          If I was a user viewing this site, I would be so confused because there is no indication of actions to use. For example, the website doesn't tell the user anything about what actions to take. I would add some content to the top, maybe some rollover or pop-overs on my images to encourage user interactions.



          Last thing, if this site is a non profit, it is pretty common to re-enforce the mission and/or vision statement on the homepage for the user to understand the commitment of the nonprofit. Foundations are typically trying to raise money for a cause and without the user knowing or understanding the cause, they will likely not take action. My college professors all said that the user should be able to find information immediately, if they have to look they are going to leave.



          Without seeing the actual site, I am going off of my design knowledge so I hope this helps. If you want to share your HTML to show how you are placing your code and what is actually happening on the site, I will try my best to help you clean up the CSS.






          share|improve this answer























            Your Answer





            StackExchange.ifUsing("editor", function () {
            return StackExchange.using("mathjaxEditing", function () {
            StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
            StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
            });
            });
            }, "mathjax-editing");

            StackExchange.ifUsing("editor", function () {
            StackExchange.using("externalEditor", function () {
            StackExchange.using("snippets", function () {
            StackExchange.snippets.init();
            });
            });
            }, "code-snippets");

            StackExchange.ready(function() {
            var channelOptions = {
            tags: "".split(" "),
            id: "196"
            };
            initTagRenderer("".split(" "), "".split(" "), channelOptions);

            StackExchange.using("externalEditor", function() {
            // Have to fire editor after snippets, if snippets enabled
            if (StackExchange.settings.snippets.snippetsEnabled) {
            StackExchange.using("snippets", function() {
            createEditor();
            });
            }
            else {
            createEditor();
            }
            });

            function createEditor() {
            StackExchange.prepareEditor({
            heartbeatType: 'answer',
            convertImagesToLinks: false,
            noModals: true,
            showLowRepImageUploadWarning: true,
            reputationToPostImages: null,
            bindNavPrevention: true,
            postfix: "",
            imageUploader: {
            brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
            contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
            allowUrls: true
            },
            onDemand: true,
            discardSelector: ".discard-answer"
            ,immediatelyShowMarkdownHelp:true
            });


            }
            });














            draft saved

            draft discarded


















            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f196552%2fmobile-webpage-for-a-refugee-organization%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown
























            1 Answer
            1






            active

            oldest

            votes








            1 Answer
            1






            active

            oldest

            votes









            active

            oldest

            votes






            active

            oldest

            votes








            up vote
            1
            down vote













            This question is difficult to answer because art is subjective but if this was my site, I would limit the gutters between the images to be slightly closer together (perhaps 2 maybe 3 ems) and being sure to keep them extending out to the width of the boxed layout.



            If I was a user viewing this site, I would be so confused because there is no indication of actions to use. For example, the website doesn't tell the user anything about what actions to take. I would add some content to the top, maybe some rollover or pop-overs on my images to encourage user interactions.



            Last thing, if this site is a non profit, it is pretty common to re-enforce the mission and/or vision statement on the homepage for the user to understand the commitment of the nonprofit. Foundations are typically trying to raise money for a cause and without the user knowing or understanding the cause, they will likely not take action. My college professors all said that the user should be able to find information immediately, if they have to look they are going to leave.



            Without seeing the actual site, I am going off of my design knowledge so I hope this helps. If you want to share your HTML to show how you are placing your code and what is actually happening on the site, I will try my best to help you clean up the CSS.






            share|improve this answer



























              up vote
              1
              down vote













              This question is difficult to answer because art is subjective but if this was my site, I would limit the gutters between the images to be slightly closer together (perhaps 2 maybe 3 ems) and being sure to keep them extending out to the width of the boxed layout.



              If I was a user viewing this site, I would be so confused because there is no indication of actions to use. For example, the website doesn't tell the user anything about what actions to take. I would add some content to the top, maybe some rollover or pop-overs on my images to encourage user interactions.



              Last thing, if this site is a non profit, it is pretty common to re-enforce the mission and/or vision statement on the homepage for the user to understand the commitment of the nonprofit. Foundations are typically trying to raise money for a cause and without the user knowing or understanding the cause, they will likely not take action. My college professors all said that the user should be able to find information immediately, if they have to look they are going to leave.



              Without seeing the actual site, I am going off of my design knowledge so I hope this helps. If you want to share your HTML to show how you are placing your code and what is actually happening on the site, I will try my best to help you clean up the CSS.






              share|improve this answer

























                up vote
                1
                down vote










                up vote
                1
                down vote









                This question is difficult to answer because art is subjective but if this was my site, I would limit the gutters between the images to be slightly closer together (perhaps 2 maybe 3 ems) and being sure to keep them extending out to the width of the boxed layout.



                If I was a user viewing this site, I would be so confused because there is no indication of actions to use. For example, the website doesn't tell the user anything about what actions to take. I would add some content to the top, maybe some rollover or pop-overs on my images to encourage user interactions.



                Last thing, if this site is a non profit, it is pretty common to re-enforce the mission and/or vision statement on the homepage for the user to understand the commitment of the nonprofit. Foundations are typically trying to raise money for a cause and without the user knowing or understanding the cause, they will likely not take action. My college professors all said that the user should be able to find information immediately, if they have to look they are going to leave.



                Without seeing the actual site, I am going off of my design knowledge so I hope this helps. If you want to share your HTML to show how you are placing your code and what is actually happening on the site, I will try my best to help you clean up the CSS.






                share|improve this answer














                This question is difficult to answer because art is subjective but if this was my site, I would limit the gutters between the images to be slightly closer together (perhaps 2 maybe 3 ems) and being sure to keep them extending out to the width of the boxed layout.



                If I was a user viewing this site, I would be so confused because there is no indication of actions to use. For example, the website doesn't tell the user anything about what actions to take. I would add some content to the top, maybe some rollover or pop-overs on my images to encourage user interactions.



                Last thing, if this site is a non profit, it is pretty common to re-enforce the mission and/or vision statement on the homepage for the user to understand the commitment of the nonprofit. Foundations are typically trying to raise money for a cause and without the user knowing or understanding the cause, they will likely not take action. My college professors all said that the user should be able to find information immediately, if they have to look they are going to leave.



                Without seeing the actual site, I am going off of my design knowledge so I hope this helps. If you want to share your HTML to show how you are placing your code and what is actually happening on the site, I will try my best to help you clean up the CSS.







                share|improve this answer














                share|improve this answer



                share|improve this answer








                edited Nov 25 at 5:27









                Sᴀᴍ Onᴇᴌᴀ

                7,88561750




                7,88561750










                answered Nov 25 at 4:00









                Nikimaria87

                72




                72






























                    draft saved

                    draft discarded




















































                    Thanks for contributing an answer to Code Review Stack Exchange!


                    • Please be sure to answer the question. Provide details and share your research!

                    But avoid



                    • Asking for help, clarification, or responding to other answers.

                    • Making statements based on opinion; back them up with references or personal experience.


                    Use MathJax to format equations. MathJax reference.


                    To learn more, see our tips on writing great answers.





                    Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


                    Please pay close attention to the following guidance:


                    • Please be sure to answer the question. Provide details and share your research!

                    But avoid



                    • Asking for help, clarification, or responding to other answers.

                    • Making statements based on opinion; back them up with references or personal experience.


                    To learn more, see our tips on writing great answers.




                    draft saved


                    draft discarded














                    StackExchange.ready(
                    function () {
                    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f196552%2fmobile-webpage-for-a-refugee-organization%23new-answer', 'question_page');
                    }
                    );

                    Post as a guest















                    Required, but never shown





















































                    Required, but never shown














                    Required, but never shown












                    Required, but never shown







                    Required, but never shown

































                    Required, but never shown














                    Required, but never shown












                    Required, but never shown







                    Required, but never shown







                    Popular posts from this blog

                    Сан-Квентин

                    8-я гвардейская общевойсковая армия

                    Алькесар