Skip to content

Добавляет сайдбар с навигацией #49

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

Merged
merged 7 commits into from
Jan 13, 2021
Merged

Добавляет сайдбар с навигацией #49

merged 7 commits into from
Jan 13, 2021

Conversation

extendsnull
Copy link
Contributor

Основано на изменениях #28

Дополнительно:

  • Убрал полифилл для position: sticky;
  • Актуализировал навигацию, заменил aside на nav;
  • Rem'изировал паддинги для сайдбара на десктопе и немного уменьшил размеры текста;
  • Заменил глобальное переопределение box-sizing.

@nikolai-shabalin
Copy link
Contributor

В h3 заменить 60px на 20px, чтобы скролилось нормально


<pre class="language-js language-correct"><code>val = getVal();
<pre class="language-js language-correct"><code>val = getVal();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Меня смущает, что не указано ключевое слово для объявления переменной. Это нормально?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это пример, тут пояснять откуда переменная не нужно. Представим, что она определена

@nikolai-shabalin nikolai-shabalin merged commit 6075210 into htmlacademy:master Jan 13, 2021
nikolai-shabalin pushed a commit that referenced this pull request Jan 13, 2021
* Добавляет сайдбар с навигацией

* Возвращает исходные размеры текста.

* Удаляет ненужные стили

* Изменяет scroll-margin-top у заголовков

* Изменяет адрес ссылки с котиком

* Форматирует отступы для листингов кода

* Форматирует отступы в листингах кода

(cherry picked from commit 6075210)
@nikolai-shabalin
Copy link
Contributor

Спасибо

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

Successfully merging this pull request may close these issues.

2 participants