Skip to content

Conversation

@Sveteeen
Copy link

@Sveteeen Sveteeen commented Apr 28, 2025

Изменения

Детали реализации

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@BatuevIO BatuevIO self-requested a review April 30, 2025 11:55
@BatuevIO
Copy link
Contributor

Давай сделаем здесь тоже добавим тултип -- там может быть фиксированный текст, типа "4% от всей полученной суммы".

image

Чтобы так не показывалось, мб имеет смысл сделать disabled если текст пустой

image

@Sveteeen Sveteeen requested a review from BatuevIO May 7, 2025 11:27
@Sveteeen
Copy link
Author

Sveteeen commented May 7, 2025

Давай сделаем здесь тоже добавим тултип -- там может быть фиксированный текст, типа "4% от всей полученной суммы".

image

Чтобы так не показывалось, мб имеет смысл сделать disabled если текст пустой

image

Тут все доделала и поправила

Copy link
Contributor

@BatuevIO BatuevIO left a comment

Choose a reason for hiding this comment

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

Все супер! Одно последнее изменение, чтобы заработало на мобилке, и можем катить в тест. Дальше уточним про тексты

<div id="gss" class="sum-plus bg-primary">{{ formattedStipend(recount['gss']) }}</div>
<div class="d-flex">
<div id="gss" class="sum-plus bg-primary">{{ formattedStipend(recount['gss']) }}</div>
<v-tooltip
Copy link
Contributor

Choose a reason for hiding this comment

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

Проверил, с мобилки не открывается тултип. Нашел тут, что можно пофиксить, добавив аттрибуты open-on-click, open-on-hover, open-on-focus.

Еще лучше будет, чтобы не дублировать это каждый раз, добавить это в тему vuetify. Для этого в файле /src/vuetify.ts в конфиге (внутри createVuerify) в объекте defaults добавить вот такой объект:

VTooltip: {
	'open-on-click': true,
	'open-on-hover': true,
	'open-on-focus': true,
}, 

Copy link
Author

Choose a reason for hiding this comment

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

Добавила, в мобилке кнопка у меня отображается, но текст все равно нет...(
Тестила в Devtools

Choose a reason for hiding this comment

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

Укажи явно на странничке open-on-click, open-on-hover, open-on-focus, без добавления в vuetify.
Так будет точно все работать)

@Sveteeen Sveteeen requested a review from Semedanik May 11, 2025 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants