-
Notifications
You must be signed in to change notification settings - Fork 0
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
CSS Meme Slider #1
Open
Dmitriy-Frostoff
wants to merge
17
commits into
main
Choose a base branch
from
gh-pages
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…cture (Sun, Sep 17, 2023 01:27:48 AM)
…17, 2023 04:13:44 AM)
… (Sun, Sep 17, 2023 06:15:01 AM)
…17, 2023 08:59:48 AM)
…n and footer); feat: implement basic structure of the slider without slider logic for 1024px and 500px device widths (Mon, Sep 18, 2023 06:10:34 AM)
… Sep 18, 2023 07:04:28 AM)
…f the task (Mon, Sep 18, 2023 07:32:00 AM)
…, Sep 18, 2023 07:38:21 AM)
…ask TR (Mon, Sep 18, 2023 09:53:38 AM)
Dmitriy-Frostoff
temporarily deployed
to
github-pages
September 18, 2023 06:54
— with
GitHub Pages
Inactive
… task (Mon, Nov 06, 2023 07:09:19 PM)
… Nov 06, 2023 07:19:36 PM)
Dmitriy-Frostoff
temporarily deployed
to
github-pages
November 6, 2023 16:20
— with
GitHub Pages
Inactive
Dmitriy-Frostoff
temporarily deployed
to
github-pages
November 6, 2023 16:26
— with
GitHub Pages
Inactive
Dmitriy-Frostoff
force-pushed
the
gh-pages
branch
from
November 6, 2023 16:30
7609d75
to
7e821c6
Compare
Dmitriy-Frostoff
temporarily deployed
to
github-pages
November 6, 2023 16:30
— with
GitHub Pages
Inactive
…n, Nov 06, 2023 07:46:19 PM)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Выполнено всё из секции Требования к репозиторию и как сабмитить задание max +30
В текущем требовании предусматривается выполнение 6 пунктов задания => 30баллов / 6 = +5 за каждый пункт.
=>
Создайте личный публичный репозиторий cssMemeSlider - Выполнено, +5
Создайте gh-pages ветку (если таковой не имеется) - Выполнено, +5
Перейдите в эту (gh-pages) ветку и создайте папку cssMemeSlider. Ваша работа будет доступна по ссылке,
например: https://${YOUR_GITHUB_NAME}.github.io/cssMemeSlider/cssMemeSlider/index.html. Выполнено, +5
Выполняете таску в этой папке (cssMemeSlider). В процессе выполнения задания должно быть не менее 5
коммитов. Выполнено, +5
Названия коммитов должны быть согласно гайдлайну + каждый ваш
коммит должен содержать тайм-стемп.
После завершения работы откройте Pull Request из ветки gh-pages в ветку main. Название Pull Request дайте
по названию задания. Описание Pull Request дайте по схеме . Мержить данный Pull Request не нужно. Ссылку на данный пулл реквест необходимо
сабминтуть в форму кросс-чека. В пулл реквесте необходимо досконально описать, что сделано и что нет. В
частности - описаны все разрешения, при которых проверяющему необходимо смотреть - например:
мобильная версия 320x568, версия для ПК - 1920×1080.
Выполнено, +5
Слайдер позиционируется с равными отступами слева и справа max +10
Можно проверить посмотрев, например: container
main__container
;и
.carousel__wrapper > justify-content: space-evenly
;Выполнено, +10
Соблюдено расположение картинок, подписей к ним и контролов max +10
Картинки имеют разную ширину и высоту. Поэтому я использовал контейнер
div.carousel__image
с жёсткозаданными width и height, внутри в качестве
background-image
использовал картинки с параметромbackground- size: contain;
. Таким образом, контейнер картинки, подписи и контролы расположены как того требует ТЗ.Выполнено, +10
Имеется плавная анимация смены картинок max +20
Я выстроил структуру слайдера таким образом, что при смене
radio-button
просисходит смещениеtransform: translateX(units);
на заданную величину оболочки с картинкамиdiv.layout-multiple-columns.carousel__content
и оболочки с подписямиdiv.layout-multiple-columns.carousel__content-description
. Для плавности перемещения указанных оболочек им присвоеноtransition: all 0.7s ease 0s;
.Выполнено, +20
Имеется плавная анимация смены подписей к картинкам max +10
Описал выше.
Выполнено, +10
Подписи к картинкам должны быть строковыми значениями (текстом), т.е. текст не должен быть частью картинки max +15
Все подписи заданы в теге
p.paragraph-l.paragraph_width
и могут быть выделены и скопированы.Выполнено, +15
Каждый контрол имеет большую область нажатия, чем размер самого контрола max +5
Каждый контрол обёрнут в
label.layout-one-column.carousel__button
, которому задана ширина и высота большая, чем radio-buttoninput.carousel__button-inner
.габариты label
.carousel__button
width: 3vw; height: 3vh;
;габариты input
.carousel__button-inner
width: 3vw; height: 3vh;
;Для input отключил указатели, задав свойство
pointer-events: none
Для label: hover задал
cursor: pointer
.Выполнено, +5
Контролы имеют интерактивность (момент наведения, момент нажатия, активный контрол, изменение курсора) max +10
Для стилизации
radio-buttons
я использовал свойствоappearence: none
.Далее задал изменение цвета активного контрола, на :hover по контролу, изменение курсора
cursor: pointer;
.Интерактивноть и плавность наведения приблизил к указанному в ТЗ. (background синий полностью по
умолчанию, у активного контрола - синий border и тёмный background.
На :hover контрол становится белым, а активный (:checked) - меняет только цвет border.
Выполнено, +10
Имеется мобильная версия слайдера, и соблюдено расположение картинок, подписей к ним и контролов max +20
Слайдер адаптирован и проверен на соответствие ТЗ на разрешениях экрана mobile 500px ... desktop 1024px.
Так как слайдер выполнен по адаптивной технологии, предельные разрешения экрана: mobile 320px, desktop 2560px.
Выполнено, +20
Используются только относительные единицы измерения для основных блоков (картинки, контролы, подписи).
Относительные единицы: rem, em, %, vh, vw, fr and etc... - присутствует "резиновость" слайдера max +10
Везде применены относительные единицы vw, vh, %.
PX использованы только в
@media
. Вёрстка выполнена по адаптивной технологии.Выполнено, +10
Все блоки/составные части слайдера (контролы, картинки, подписи к слайдеру) находятся в базовом потоке дом-элементов, не позиционируются (position) с помощью top, left, right, bottom, не используется float. Можно: flex, grid, margin-left and etc... - все, что не упомянуто в запретах выше. Свойство position может быть только static. max +5
Применялся только
flex
сmargin, padding
Выполнено, +5
Не используются псевдоэлементы (не путать с псевдоклассами, псевдоклассы можно). max +5
Использовал только псевдоклассы. Основной:
:has
, без него было бы крайне проблематично выполнить слайдер с использованием radio-buttons.Выполнено, +5