Skip to content
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

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

Open
wants to merge 1 commit into
base: landings-dream-job
Choose a base branch
from

Conversation

thegruffaloop
Copy link

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

Мотивационное письмо ✉️ — Трудоустройство — Дока 2024-08-11 19-03-48

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

Copy link
Member

@skorobaeus skorobaeus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

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

@thegruffaloop
Copy link
Author

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

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

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

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

@igsekor
Copy link
Member

igsekor commented Sep 13, 2024

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

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

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

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

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

@igsekor igsekor added the ошибка Что-то критичное и мешает label Sep 13, 2024
@thegruffaloop
Copy link
Author

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

Хорошо )

@igsekor
Copy link
Member

igsekor commented Sep 14, 2024

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

Хорошо )

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

@thegruffaloop thegruffaloop changed the base branch from main to landings-dream-job September 14, 2024 19:20
@thegruffaloop
Copy link
Author

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

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

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

@igsekor
Copy link
Member

igsekor commented Sep 14, 2024

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

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

Copy link

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

@thegruffaloop
Copy link
Author

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ошибка Что-то критичное и мешает
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants