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

Добавляет рецепт «Плиточная раскладка» #5344

Merged
merged 17 commits into from
May 17, 2024

Conversation

alexTaraskina
Copy link
Contributor

Описание

Добавлен рецепт «Плиточная раскладка»

Closes #4361

Чек-лист

  • Текст оформлен согласно руководству по стилю
  • Ссылки на внутренние материалы начинаются со слеша и заканчиваются слэшем либо якорем на заголовок (/css/color/, /tools/json/, /tools/gulp/#kak-ponyat)
  • Ссылки на картинки, видео и демки относительные (images/example.png, demos/example/, ../demos/example/)

@github-actions github-actions bot added статья Расширенный материал рецепт Контент для Рецептов labels May 13, 2024
Copy link
Member

@solarrust solarrust left a comment

Choose a reason for hiding this comment

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

Спасибо! Долгожданный рецепт 🥰
Я быстро пробежалась и предложила несколько мелких правочек по тексту.
Обрати, пожалуйста, внимание на заголовки внутри демок и на title у <iframe>. Кажется, сейчас они чуть не соответствуют действительности.

Призывают @TatianaFokina чтобы она отсыпала доступности и @skorobaeus за красотищей =)

recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
@alexTaraskina
Copy link
Contributor Author

@solarrust спасибо за ревью! Всё поправила

skorobaeus and others added 4 commits May 15, 2024 18:03
Исправляет тайтлы; убирает скругления углов; убирает красный; меняет рамку на сплошную заливку
Co-authored-by: Alena Batitskaia <solarrust@users.noreply.github.com>
Copy link
Member

@skorobaeus skorobaeus left a comment

Choose a reason for hiding this comment

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

Починила превью, ай да я 😁

Демки замечательные! Я убрала скругление углов и красный цвет как средства выразительности, которые в демках не используются <3

Copy link
Member

@TatianaFokina TatianaFokina left a comment

Choose a reason for hiding this comment

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

Е-е-е, как клёво. Поредачила ещё немного текст и оставила два комментария, что ещё можно упомянуть.

recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
recipes/masonry/index.md Outdated Show resolved Hide resolved
Copy link

Превью контента из c0165c8 опубликовано.

@TatianaFokina
Copy link
Member

Да думаю уже можно мёрджить с тремя апрувами.

@solarrust solarrust merged commit 758e268 into doka-guide:main May 17, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
рецепт Контент для Рецептов статья Расширенный материал
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Плиточная раскладка на чистом CSS
4 participants