Skip to content
OverviewSolutions
4
Comments
5
Hatamovich
@Hatamovich

All comments

  • Hatamovich•93
    @HatamovichPosted over 4 years ago

    Hello I think you are doing well if you are a newbie :) firstly you should add cursor: pointer; to the button. secondly set background of the body to the image that is on the left side now so that it covers whole body, then create two columns within the container. which will be left and right columns on the left side put the content and on the right side create image tag and set its src attribute to the image that is on the right side now. I hope you understand me and it will help you :)

    0
  • Hatamovich•93
    @HatamovichPosted over 4 years ago

    Looks pretty nice until you click on the arrow buttons. I think you should change the way it's animated or remove the animation at all :) in my personal opinion a flash animation would be a nice choice for this particular design.

    1
  • Hatamovich•93
    @HatamovichPosted over 4 years ago
    This comment was deleted over 4 years ago
    0
  • esraagamal•600
    @EsraaGamal-22Submitted over 4 years ago

    Rate my design please, Any feedback and suggestions on how I can improve are very welcome!

    using flexbox

    3
    Hatamovich•93
    @HatamovichPosted over 4 years ago

    Hello Esra Gamal 😊 I would recommend you to wrap your content inside a container and center it and also always consider adding cursor: pointer; to you buttons. I also highly recommend you to use meaningful class names. try not to use names like "cont-1, cont-2, cont-1-2 , cont-2-2".

    If you do not know what that container (wrapper) does then I strongly recommend you to go and do some research and learn how to center your content on the page, and learn why and how to use percentage units :)

    http://css-tricks.com is a nice place to learn some cool and useful stuff.

    Sorry for my English if you find it terrible :D I hope my advice will help you to get more comfortable 😊

    1
  • Hatamovich•93
    @HatamovichPosted over 4 years ago

    Nice try 👍🏻 In my view it will look much better if you increase the font sizes 😊

    2.5em for primary heading and maybe 1 or 1.1em for paragraphs :)

    0