search

finec-mgimo / finec-mgimo.github.io

Финэк МГИМО - программы бакалавриата и магистратуры по экономике, менеджменту, туризму и бизнес-информатике в кампусе МГИМО-Одинцово.
https://finec.mgimo.ru
MIT License
1 stars 8 forks source link

доработка карусели на первой странице #193

Closed epogrebnyak closed 2 years ago

epogrebnyak commented 2 years ago

Примеры фото для карусели

epogrebnyak commented 2 years ago

Нужен размер баннеров, которые подходят для карусели на первой странице

epogrebnyak commented 2 years ago

Closes #87 and closes #113

super16 commented 2 years ago

@epogrebnyak размер баннеров желательно не больше 500px по ширине, по высоте не имеет значения, но важно, чтобы вся серия изображений были одного размера или, хотя бы, одной пропорции

super16 commented 2 years ago

@epogrebnyak, добавил прототип карусели на стартовую в #198

epogrebnyak commented 2 years ago

@epogrebnyak, добавил прототип карусели на стартовую в #197

@super16 - на каком-то отдельном бранче висит?

super16 commented 2 years ago

В ветке banner-carousel-193

epogrebnyak commented 2 years ago

Карусель замержду в мейн, вижу два возможных улучшения:

super16 commented 2 years ago

@epogrebnyak 1 и 3 пункты поправил в #212, теперь высота всех изображений одинаковая, кнопка больше не «прыгает», тонкой полосы тоже больше не должно быть, это недочёт деления calc в CSS.

По пункту два: путь к data файлу лучше не делать каким-то определённым с указанием расширения, так как архитектурой HUGO заложено, что data-файлы должны не зависеть от расширения, то есть равнозначно может использоваться YML, TOML, XML, JSON, одинаковые типы в data-файлах разных расширений обрабатываются одинаково. А также в шаблонах shortcodes неудобно обращаться к заданным путям и требуется дополнительная логика для их распарсивания и обработки, когда HUGO делает это всё автоматически.

epogrebnyak commented 2 years ago

@epogrebnyak размер баннеров желательно не больше 500px по ширине, по высоте не имеет значения, но важно, чтобы вся серия изображений были одного размера или, хотя бы, одной пропорции

Спасибо, а понятно какой аспект нужен для возможного баннера или подбирать надо?

epogrebnyak commented 2 years ago

По пункту два: путь к data файлу лучше не делать каким-то определённым с указанием расширения, так как архитектурой HUGO заложено, что data-файлы должны не зависеть от расширения, то есть равнозначно может использоваться YML, TOML, XML, JSON, одинаковые типы в data-файлах разных расширений обрабатываются одинаково. А также в шаблонах shortcodes неудобно обращаться к заданным путям и требуется дополнительная логика для их распарсивания и обработки, когда HUGO делает это всё автоматически.

Про пути понял, спасибо. Предлагаю в документации ответить про дефолтный путь этой карусели. Кстати класс карусели отличается на первой и не первой странице? Или это один и тот же?

epogrebnyak commented 2 years ago

@epogrebnyak 1 и 3 пункты поправил в #212, теперь высота всех изображений одинаковая, кнопка больше не «прыгает», тонкой полосы тоже больше не должно быть, это недочёт деления calc в CSS.

В мобильном выглядят отлично, стабильно, ничего не скачет

super16 commented 2 years ago

какой аспект нужен для возможного баннера или подбирать надо?

Вероятно, желательно, тоже не больше ширины изображения. Квадратные и 4:3 изображения в узкой колонке смотрятся ок, вытянутые по высоте уже не очень

Кстати класс карусели отличается на первой и не первой странице? Или это один и тот же?

Это два разных макроса, в документации я их разделил:

epogrebnyak commented 2 years ago

Хочу заказать баннер, 400px на 300 px будет хорошо?

super16 commented 2 years ago

Лучше тогда ориентироваться на 500px на 375px, в идеале также иметь и большую версию 1000px на 750px

epogrebnyak commented 2 years ago

500px на 375px, в идеале также иметь и большую версию 1000px на 750px

Тогда будем ориентирвоаться на этот размер

epogrebnyak commented 2 years ago

Делаем первый баннер для карусели в #216