GithubHelp home page GithubHelp logo

air_planet's People

Contributors

evdrug avatar

Watchers

avatar

air_planet's Issues

index.html (header, footer, aside, content)

  1. Лучше будет, если шапка и футер тебя будут тянуться на всю страницу)
  2. Внутренние элементы у header лучше отталкивать за счет внутренних паддингов у header, а не за счет внешних паддингов у внутренних элементов (logo, contacts). Тоже самое касается и footer и его внутренних элементов
  3. header - высоту у элемента лучше подгонять не через min-height, а через внутренние отступы
  4. .menu-nav_position - лучше margin-top заменить на padding-top у sidebar. Внешние отступы нужны чтобы отталкивать элементы друг от друга, а внутренние чтобы отталкивать внутренние элементы от границы) В данном случае нужен внутренний отступ у главного родителя, а не у внутреннего элемента
  5. menu-nav__item - паддинг слева тогда лучше перенести внешнему отступу слева .menu-nav_position
  6. .menu-nav__text_position - а вот здесь как раз наоборот, слева лучше сделать внутренний отступ (padding-left), а нижние внешние отступы перенести элементу, который должен отталкиваться - menu-nav__item. При этом у последнего menu-nav__item не должно быть нижнего отступа
  7. .info_padding - телефон внутри этого блока лучше сделать ссылкой)
  8. partners_position - верхний отступ лучше убрать, вместо этого добавь нижний к info_padding. Правильнее выталкивать элементы друг от друга снизу.
  9. .partners__list_position - тоже самое, что и в предыдущем замечании
  10. partners__link - добавь свойство display: inline-block;. Старайся для всех ссылок добавлять свойство display.
  11. Я помню, ты спрашивал про выравнивание партнеров) К элементам списка я добавила следующие свойства из тех, которые вы уже изучили:
    vertical-align: middle;, display: inline-table;, text-align: center;, width: 169px;
    Для ссылок внутри добавила vertical-align: middle;, display: table-cell;
    Практически получилось подогнать) Дело в том, что здесь по сути самый эффективный способ выравнивания по вертикале и горизонтале - микс табличной верстки и свойства vertical-align: middle
    Потом вы еще несколько способов выравнивания по вертикале, скажем так более современных, это один из первоначальных способов)
  12. В футер можно уже свое имя вставить в “автора работ”
  13. .social-link__item - у последнего элемента не должно быть отступа вправа
  14. social-link__link - добавь свойство display: block;
  15. Логотип компании лучше сделать кликабельным и пусть он ведет на главную страницу (и в шапке, и в подвале)

reviews.html (сontent)

  1. .breadcrumbs_position - опять таки лучше тогда убрать верхний маржин у этого элемента и задать тогда внутренний верхний отступ .page
  2. article - свойство width: 877px; лучше тогда уж перенести родителю .page__reviews_position
  3. review_position - лишняя обертка
  4. .review-photo_position - нижний внешний отступ лучше перенести во внутренний нижний к article. Соответственно у последнего элемента не должно быть отступа, его лучше перенести во внешний нижний для .page__reviews_position
  5. .form__botton - рядом с этим классом нужно добавитькласс clearfix, так как у внутренних элементов указано свойство float и родитель их “не видит”. clearfix нужно задавать в том случае, если непосредственные дети блока имеют float, как например у тебя в этом блоке.
  6. Для .form__btn необходимо добавить свойство cursor: pointer;
  7. В консоле есть ошибка - не может найти какую-то картинку)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.