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

Issue 739 tooltip align fix #1723

Merged
merged 2 commits into from
May 7, 2024
Merged

Conversation

Alinochka07
Copy link
Contributor

Ссылка на задачу

Расстояние между стрелкой тултипа и иконкой стало теперь одного размера для всех случаев.
Исчез стык между стрелкой и телом тултипа в суб-локаторе, где была кнопка копирования.

Новая ветка на задачу.

@Alinochka07
Copy link
Contributor Author

Создана новая ветка. Предыдущая: issue_739-tooltip-align-fix-seam будет удалена и ПР https://github.com/jdi-testing/jdn-ai/pull/1717 тоже будет закрыт.

@Alinochka07 Alinochka07 force-pushed the issue_739-tooltip-align-fix branch 2 times, most recently from 42dde8f to 2106505 Compare April 23, 2024 17:15
@Iogsotot
Copy link
Contributor

image
этот комит мне очень сильно не нравится.
Как будет готов ПР - напиши в нашем чатике в дискорде, пожалуйста

Copy link
Contributor

@Iogsotot Iogsotot left a comment

Choose a reason for hiding this comment

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

надо поправить историю комитов:

  • внутри ПР должен быть актуальный мастер
  • комиты должны содержать только твои изменения
  • не должно быть слишком длинной и запутанной истории комитов

manifest.json Outdated Show resolved Hide resolved
@Iogsotot
Copy link
Contributor

Iogsotot commented Apr 24, 2024

Вижу, что с историей комитов почти получилось. Осталось почистить тело комит-сообщения:
image

@Alinochka07 Alinochka07 force-pushed the issue_739-tooltip-align-fix branch 3 times, most recently from 69e0b6c to 6fbb80b Compare May 2, 2024 14:12
@Alinochka07
Copy link
Contributor Author

Все вроде бы сделано кроме чистки истории коммитов в родительском коммите: fix: align tooltips, seams. Как только пытаюсь тоже сквошить их, так другие коммиты появляются те что на уровне родительского коммита. В интернете вообще нет информации про это. Все только про squash. Напиши если нужно все же решать этот вопрос. Спасибо

@Alinochka07 Alinochka07 requested a review from Iogsotot May 3, 2024 09:54
@Iogsotot Iogsotot force-pushed the issue_739-tooltip-align-fix branch from 6fbb80b to c0d69ad Compare May 7, 2024 07:32
@Iogsotot
Copy link
Contributor

Iogsotot commented May 7, 2024

Всё готово к мерджу.

Что сделала, чтобы поправить историю комитов:
0. (обновила мастер по последнему origin)

  1. сбросила историю комитов git reset --soft
  2. нашла твой код, добавила проиндексировала, добавила в комит fix: align tooltips, seams
  3. ребейзнула ветку по локальному (и обновленному) мастеру
  4. подняла версию плагина npm run patch
  5. запушила все изменения через --force на origin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants