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

Добавляет рецепт с написанием попапа и демку к нему #4805

Merged
merged 34 commits into from
Nov 21, 2023
Merged

Conversation

AnnaBaraulina
Copy link
Contributor

@AnnaBaraulina AnnaBaraulina commented Oct 24, 2023

Описание

Добавляет рецепт с реализацией попапа на и демку к нему

Closes #2733

https://content-4805.dev.doka.guide/recipes/popup/

Чек-лист

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

@github-actions github-actions bot added the статья Расширенный материал label Oct 24, 2023
@AnnaBaraulina
Copy link
Contributor Author

@TatianaFokina привет! Я хотела посоветоваться насчет демки, мне она не очень нравится в нынешнем варианте. Если не открывать ее в отдельной вкладке, не очевидно, что там вообще что-то есть, так как до кнопки приходится скроллить вниз. При этом мне хотелось показать в статье, как с помощью модалки заблокировать скролл страницы. Может быть мне сделать две демки: в начале статьи оставить нынешнюю , но удалив у body высоту, а в том месте, где пишу про блокировку скролла добавить демку как у меня была изначально?

@solarrust
Copy link
Member

Привет! Подкину идею. Можно добавить на «страницу» демки какой-то контент. Например, несколько фотографий котиков. А кнопку открытия попапа, например, фиксированным позиционированием повесить в нижнем правом углу. Тогда и страница будет скроллиться и кнопка открытия всегда будет «под рукой».

@AnnaBaraulina
Copy link
Contributor Author

Привет! Подкину идею. Можно добавить на «страницу» демки какой-то контент. Например, несколько фотографий котиков. А кнопку открытия попапа, например, фиксированным позиционированием повесить в нижнем правом углу. Тогда и страница будет скроллиться и кнопка открытия всегда будет «под рукой».

Это без проблем, только Таня перед этим наоборот попросила спозиционировать кнопку по центру (изначально была в верхнем левом углу). Делаю в правом нижнем тогда?

@AnnaBaraulina
Copy link
Contributor Author

Фиксированным будет симпатичнее всего,конечно, так и сделаю, спасибо

@TatianaFokina
Copy link
Member

Это без проблем, только Таня перед этим наоборот попросила спозиционировать кнопку по центру (изначально была в верхнем левом углу). Делаю в правом нижнем тогда?

Таня перед этим тупанула 😅 Мне нравится идея Алёны.

@AnnaBaraulina
Copy link
Contributor Author

Каждый раз я не прохожу тест на форматирование. При этом ошибка, которую он выдает - recipes/popup/demos/popup-demo/index.html:
Wrong line endings or no final newline - я не могу обнаружить на глаз, а prettier не помогает. Как мне с этим бороться, подскажите мне, пожалуйста.

@AnnaBaraulina
Copy link
Contributor Author

Привет, всем! От меня нужны какие-то еще доработки?
Вижу конфликт файлов, но не могу понять, откуда взялся .yaspeller.json

@TatianaFokina
Copy link
Member

@AnnaBaraulina, я там слегка переделала демку. Пока, наверное, лучше не заменять примеры в тексте на более актуальные, Света может что-то добавить и изменить.

Пока предлагаю выбрать топ-4 самых мемных котика, а остальные картинки удалить.

@skorobaeus
Copy link
Member

Ребята, вы такие классные 🥺

Поддерживаю Таню, четырёх котиков достаточно, чтобы скролл был даже на исполинском десктопе.

От себя предлагаю на мобилках кнопку выровнять по центру по горизонтали, всё-таки. А то при малой ширине экрана такой сдвиг выглядит странно:

image

@TatianaFokina
Copy link
Member

Я добавила папочку автора, можешь что-то написать о себе в файлике, если хочешь.

@AnnaBaraulina
Copy link
Contributor Author

Я добавила папочку автора, можешь что-то написать о себе в файлике, если хочешь.

Я написала там чуть, спасибо!:)

@TatianaFokina
Copy link
Member

@solarrust, посмотришь этот пиар?

Copy link
Contributor

@StarHamster StarHamster left a comment

Choose a reason for hiding this comment

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

Турбо-пушка! Можно только мемы через tinypng прогнать и вообще будет супер

Copy link

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

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.

🤩

@solarrust solarrust merged commit e2090df into doka-guide:main Nov 21, 2023
6 of 7 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.

Попап
5 participants