search

jdi-testing / jdn-ai

JDN – is the Chrome plugin for UI test automation. It helps Software Test Automation Engineer to create Page Objects for the test automation framework and start test development fast
MIT License
9 stars 3 forks source link

Issue 739 tooltip align fix #1723

Closed Alinochka07 closed 6 months ago

Alinochka07 commented 7 months ago

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

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

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

Alinochka07 commented 7 months ago

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

Iogsotot commented 7 months ago

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

Iogsotot commented 7 months ago

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

Alinochka07 commented 7 months ago

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

Iogsotot commented 6 months ago

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

Что сделала, чтобы поправить историю комитов:

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