Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Layout + styles #1

Open
PNKBizz opened this issue Dec 15, 2019 · 2 comments
Open

Layout + styles #1

PNKBizz opened this issue Dec 15, 2019 · 2 comments

Comments

@PNKBizz
Copy link

PNKBizz commented Dec 15, 2019

  1. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/styles/style.css#L122
    Вот этот стиль перебивает стиль основного контейнера и у вас слетает центровка блока.

  2. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/index.html#L45
    Хорошее переиспользование стилей, но sub - неприемлемое имя для класса. Совершенно непонятно что оно означает. Лучше всего обозначить как btn-transparent или типа того

  3. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/index.html#L47-L51
    Следите за форматированием кода.

  4. https://github.com/Viklysx/testrep/blob/588e806661f3d063ab0320f65d554e0326a4b45f/index.html#L27
    Сделайте телефон ссылкой

@PNKBizz
Copy link
Author

PNKBizz commented Jan 2, 2020

  1. https://github.com/Viklysx/testrep/blob/060e2c69e7f8ed5d52d3137165712959f6d864f0/index.html#L120-L133
    Вот этот блок нужно перенести в секцию controls, поскольку это собственно заголовок секции. Чтоб все осталось на своих местах - всей секции задайте отрицательный margin-top. И для заголовков используем h1-h5, а не p

  2. https://github.com/Viklysx/testrep/blob/060e2c69e7f8ed5d52d3137165712959f6d864f0/styles/style.css#L290-L295
    Уберите абсолютное позиционирование. Используйте его только в крайних случаях и для небольших элементов.

  3. https://github.com/Viklysx/testrep/blob/060e2c69e7f8ed5d52d3137165712959f6d864f0/index.html#L165-L170
    Вот эта конструкция должна выглядеть как

<label>
  Название лейбла
  <input>
</label>
  1. https://github.com/Viklysx/testrep/blob/060e2c69e7f8ed5d52d3137165712959f6d864f0/index.html#L194
    Высота этих элементов должна быть реальной, у вас сейчас она равна 326px, по высоте
    .example-item-1. Так как текст абсолютно позиционирован. Избавьтесь от абсолютного позиционирования. Я бы сделал фоновую картинку у всего блока, а текст отбил вниз через padding-top

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants