search

finec-mgimo / finec-mgimo.github.io

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

Улучшенная фотокарусель #131

Closed epogrebnyak closed 2 years ago

epogrebnyak commented 2 years ago

По результатам #119 - дорабатывам фотокарусель чтобы ее можно было исопльзовать на разных страницах сайта.

Одна их страниц - кандидатов: "Партнерства" (фото визитов к партнерам факультета)

super16 commented 2 years ago

Добавил PR, где улучшил карусель, что теперь при использовании в разметке, нужно задать название data-файла в виде аргумента. Тем самым компонент можно использовать в разных местах, задавая им названия отдельных файлов. Также добавил документацию по использованию. В качестве документации для разработчиков предлагаю использовать Markdown-разметку, то есть смотреть документацию можно прямо в IDE или в интерфейсе Github, добавил также в README.md ссылку на страницу с описанием карусели

epogrebnyak commented 2 years ago

В качестве документации для разработчиков предлагаю использовать Markdown-разметку, то есть смотреть документацию можно прямо в IDE или в интерфейсе Github,

По формату понятно - по месту - в README пока доку собираем?

epogrebnyak commented 2 years ago

В качестве документации для разработчиков предлагаю использовать Markdown-разметку, то есть смотреть документацию можно прямо в IDE или в интерфейсе Github,

По формату понятно - по месту - в README пока доку собираем?

кажется понял - в папке docs, хороший вариант.

super16 commented 2 years ago

Можно ещё для полной неперекрываемости имён назвать директорию .docs вместо docs, т.е. чтобы оставить место для finec.mgimo.ru/docs, если появится такая необходимость

epogrebnyak commented 2 years ago

Можно ещё для полной неперекрываемости имён назвать директорию .docs вместо docs, т.е. чтобы оставить место для finec.mgimo.ru/docs, если появится такая необходимость

Теоретически да, но мы все равно основной сайт через Github Pages в ветке gh-pages показываем,так что docs не будет задействован, но идея полезная. Предлагаю оставить docs.

epogrebnyak commented 2 years ago

Нам в PR предлагаю переименовать в примере /data в другой каталог для картиноки можно мерджить (не хочется подталкивать к хранению картинок в data )

epogrebnyak commented 2 years ago

Можно ещё для полной неперекрываемости имён назвать директорию .docs вместо docs, т.е. чтобы оставить место для finec.mgimo.ru/docs, если появится такая необходимость

finec.mgimo.ru/docs хорошая идея,я что-то не подумал, что на имя домена может выйти

epogrebnyak commented 2 years ago

Я смерджил и подправил коммитами выше, надеюсь это закрывает вопрос с каруселью. @super16 - просьба посмотреть shortcodes.md - если что-то не так, правьте в мастер. Если все ок - просьба вы нажмите закрыть ишью.