review langauge switch works as intended

[epogrebnyak]

If we have a page in Russian language, that does not have a correspnding page in English , our current switch breaks the layout: the github icon is misplaced as below.

https://finec.mgimo.ru/alumni/

Also ther is no 'EN' on this page. If there is corresponding english page - the link should probably link to root /.

[epogrebnyak]

@AdnanBhatti - can you look? not this is not a critical issue, an enhancement. if you have an alternative - please make it on a new branch

@necelentano made the original switch implementation, he can review the enhancements

[epogrebnyak]

I think here is where we have the switch: https://github.com/finec-mgimo/finec-mgimo.github.io/blob/801f3a3d7ea605f6edb0a3a707fc9a0a71a2800f/layouts/partials/header/header.html

[necelentano]

Глянул со свежей головой, и вроде нашел ошибку. Больше не должно быть проблем со сломанным хедером. p.s. Ошибка была банальна, не там закрыл тег ul :(

[necelentano]

Для сохранения минимальной структуры в EN версии, вероятно не стоит удалять весь контент (я про программы, которые есть в меню навигации слева). Разделы Блог, О нас и Проекты, по идее, можно спокойно удалить из EN версии.

[epogrebnyak]

Нам было бы большим подспорьем не дублировать материалы, сильно бы облегчило жизнь.

Обратите внимание - модет быть ситуация, когда что-то в блоге на английском, без зеркальной статьи на русском.

У нас на англицском получается только первая страница, контакты и блог, видимо.

Не очень понятно, что делать боковиком, где по программам - их переводить не хочется

чт, 15 июл. 2021 г., 10:34 Андрей | Andrey @.***>:

Для сохранения минимальной структуры в EN версии, вероятно не стоит удалять весь контент (я про программы, которые есть в меню навигации слева). Разделы Блог, О нас и Проекты, по идее, можно спокойно удалить из EN версии.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/finec-mgimo/finec-mgimo.github.io/issues/95#issuecomment-880467389, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACGWBLTC2I2ZAIJJQY7VKT3TX2FQ7ANCNFSM5ALID46A .

[necelentano]

Как я понял, боковик слева на главной будет пустым, нужно хотя бы подзаголовки оставить UNDERGRADUATE, GRADUATE. Вполне возможно, что ничего не сломается даже если из левого сайдбара удалить все.

[epogrebnyak]

Кстати да - заголовки пусть останутся, они не мешают, а доков внутри не будет

чт, 15 июл. 2021 г., 15:08 Андрей | Andrey @.***>:

Как я понял, боковик слева на главной будет пустым, нужно хотя бы подзаголовки оставить UNDERGRADUATE, GRADUATE. Вполне возможно, что ничего не сломается даже если из левого сайдбара удалить все.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/finec-mgimo/finec-mgimo.github.io/issues/95#issuecomment-880641384, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACGWBLWGXSOWPLXWG66PP6DTX3FSVANCNFSM5ALID46A .

[epogrebnyak]

О, @necelentano, у нас кто-то форкит нашу разарботку, это чудесно! )

Вопрос: может экспериментально удалить в английском описание какой-то программы?

[necelentano]

Помню, когда мы "бились" над нужным отображением программ, у меня были ошибки из-за которых все программы и даже подзаголовки программ (UNDERGRADUATE, GRADUATE) не отображались вообще. При этом верстка страницы не ломалась – левый сайдбар был просто пустым. Думаю в EN версии можно просто удалить все программы и, по идее, все будет нормально работать.

[epogrebnyak]

@super16 - могли бы про верить, насколько оправданно у нас свитч на английский сделан - если ok - закрыть ишью, если нет - написать что можем поменять, заранее спасибо

[super16]

LGTM