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

Добавляет баннер вверх сайта. Ссылка на исследование #1250

Merged
merged 16 commits into from
Mar 20, 2024

Conversation

solarrust
Copy link
Member

@solarrust solarrust commented Mar 19, 2024

Заверстала баннер для проведения исследования аудитории. Обозвала его универсально, на случай, если воспользуемся им ещё.

Как выглядит:
CleanShot 2024-03-19 at 13 46 03@2x
CleanShot 2024-03-19 at 13 44 36@2x
CleanShot 2024-03-19 at 13 45 31@2x
CleanShot 2024-03-19 at 13 45 48@2x

Что нужно сделать

  • Добавить логику, чтобы баннер не показывался, если на экране есть хоть один другой баннер (куки, подписка)
  • Скрывать по клику на ссылку и больше не показывать этому пользователю

Добавьте, если что-то забыла.

@solarrust solarrust added кухня Работа и устройство репозитория вёрстка Задачи по вёрстке labels Mar 19, 2024
@TatianaFokina
Copy link
Member

TatianaFokina commented Mar 19, 2024

У меня уточняющий вопрос: а этот элемент не сразу же будет появляться? Если нет, то он, получается, будет двигать другое содержимое страниц, так как он в потоке документа, так?

И второй вопрос: hidden же там временно, пока нет логики появления элемента?

@solarrust
Copy link
Member Author

У меня уточняющий вопрос: а этот элемент не сразу же будет появляться? Если нет, то он, получается, будет двигать другое содержимое страниц, так как он в потоке документа, так?

И второй вопрос: hidden же там временно, пока нет логики появления элемента?

  1. Да, будет двигать. Имхо, не сильно страшно, сдвиг небольшой
  2. Я скопировала это с баннера куков, там атрибут стоит у всего блока сразу.

@TatianaFokina
Copy link
Member

TatianaFokina commented Mar 19, 2024

У меня уточняющий вопрос: а этот элемент не сразу же будет появляться? Если нет, то он, получается, будет двигать другое содержимое страниц, так как он в потоке документа, так?
И второй вопрос: hidden же там временно, пока нет логики появления элемента?

  1. Да, будет двигать. Имхо, не сильно страшно, сдвиг небольшой
  2. Я скопировала это с баннера куков, там атрибут стоит у всего блока сразу.

Оу, я не замечала, что у кукиз есть hidden. Я бы убрала его, на самом деле. Получается, что элемент всем виден визуально, но вспомогательным технологиям нет. Ну и кажется, что браузерам тоже будет ок, если такие элементы ему видны.

@solarrust
Copy link
Member Author

@igsekor кажется, добавленное мною условие для вывода ссылки не работает. Пожалуйста, посмотри этот момент.

Для главной
https://forms.yandex.ru/cloud/65f1aaf35d2a061e4ff460e0/?utm_source=doka_main

Для остальных страниц
https://forms.yandex.ru/cloud/65f1aaf35d2a061e4ff460e0/?utm_source=doka_other

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.

👐

@igsekor
Copy link
Member

igsekor commented Mar 20, 2024

Оу, я не замечала, что у кукиз есть hidden. Я бы убрала его, на самом деле. Получается, что элемент всем виден визуально, но вспомогательным технологиям нет. Ну и кажется, что браузерам тоже будет ок, если такие элементы ему видны.

@TatianaFokina предлагаю тогда сделать это отдельным пиаром. Сотворил отдельное ишью #1252.

@igsekor igsekor self-assigned this Mar 20, 2024
@igsekor igsekor assigned solarrust and unassigned solarrust Mar 20, 2024
Copy link

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

@solarrust solarrust merged commit 0bf1047 into main Mar 20, 2024
4 checks passed
@solarrust solarrust deleted the research-banner branch March 20, 2024 21:32
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