Корректирует отображение эмодзи в breadcrumbs

[thegruffaloop]

Если в конце названия статьи есть эмодзи, то ее правая часть обрезается в breadcrumbs в режиме прокрутки страницы. Это видно в разделах спец проекта "Трудоустройство".

PS: Ветка для проекта "Трудоустройство" эта? Туда нужно будет тоже влить правки.

[thegruffaloop]

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

@igsekor помоги, пожалуйста, либо вмержить этот фикс только в Трудоустройство, либо накатить на Трудоустройство новый хэдер 🙏

Привет! Можно сделать в два этапа:

1. Я могу накатить этот коммит в ветку трудоустройства у себя в fork репы и потом создать новый PR
2. Потом свободный день могу посмотреть по истории коммитов что было сделано по новому хэдэру и перенести этот функционал в ветку трудоустройства чтобы был одинаковый UI.

[igsekor]

Я сначала поставила аппрув, а потом поняла, что в новом хэдере, который мы буквально только что выкатили, нет названия статьи в хлебных крошках. Поэтому фиксить обрезание иконки уже не нужно. @igsekor помоги, пожалуйста, либо вмержить этот фикс только в Трудоустройство, либо накатить на Трудоустройство новый хэдер 🙏

Привет! Можно сделать в два этапа:

1. Я могу накатить этот коммит в ветку трудоустройства у себя в fork репы и потом создать новый PR
2. Потом свободный день могу посмотреть по истории коммитов что было сделано по новому хэдэру и перенести этот функционал в ветку трудоустройства чтобы был одинаковый UI.

Во-первых, спасибо за правку!!! Предлагаю сделать так:

1. Мы впилим этот пиар в ветку с трудоустройством landings-dream-job.
2. Я впилю новый хедер в раздел с трудоустройством отдельно уже своими силами.

[thegruffaloop]

1. Мы впилим этот пиар в ветку с трудоустройством landings-dream-job.
2. Я впилю новый хедер в раздел с трудоустройством отдельно уже своими силами.

Хорошо )

[igsekor]

1. Мы впилим этот пиар в ветку с трудоустройством landings-dream-job.
2. Я впилю новый хедер в раздел с трудоустройством отдельно уже своими силами.

Хорошо )

Нужно только ветку поменять с твоей стороны, чтобы всё сделать правильно :)

[thegruffaloop]

Нужно только ветку поменять с твоей стороны, чтобы всё сделать правильно :)

Я поменял base в PR но там миллион коммитов сразу вылез в merge request, ветка трудоустройства сильно отличается.

@igsekor, Если я локально создам новую ветку breadcrumbs-emoji-cutoff уже из landings-dream-job, черри-пикну туда коммит с правками пофиксив конфликты, форс-пушну ее в свой fork, то PR автоматом обновится?

[igsekor]

@igsekor, Если я локально создам новую ветку breadcrumbs-emoji-cutoff уже из landings-dream-job, черри-пикну туда коммит с правками пофиксив конфликты, форс-пушну ее в свой fork, то PR автоматом обновится?

Кажется, проще будет, если запулить ветку breadcrumbs-emoji-cutoff со всеми правками этого пиара в твой в форк, и поменять ветку твоего форка в пиаре. Вроде бы манипуляций будет меньше. Пиар же можно создавать не только из main.

[github-actions[bot]]

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

[thegruffaloop]

Кажется, проще будет, если запулить ветку breadcrumbs-emoji-cutoff со всеми правками этого пиара в твой в форк, и поменять ветку твоего форка в пиаре. Вроде бы манипуляций будет меньше. Пиар же можно создавать не только из main.

Поскольку гитхаб мне не дает менять ветку из которой я уже создал PR и чтоб не плодить новые PR, я обновил свою локальную ветку чтоб она соответствовала ветке трудоустройства, перенес туда свои правки (с учетом коммита @TatianaFokina который добавляет пустую строку) и форс-пушнул ее тем самым обновив PR.

Теперь конфликтов нет и destination ветка корректная.