Skip to content

Add hints #20

Open
Sveteeen wants to merge 15 commits intomainfrom
add_hitsss
Open

Add hints #20
Sveteeen wants to merge 15 commits intomainfrom
add_hitsss

Conversation

@Sveteeen
Copy link

@Sveteeen Sveteeen commented Apr 28, 2025 •
edited
Loading

Изменения

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

Check-List

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

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

BatuevIO commented Apr 30, 2025

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

image

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

image

BatuevIO
BatuevIO requested changes Apr 30, 2025
View reviewed changes
@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

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

BatuevIO
BatuevIO requested changes May 8, 2025
View reviewed changes
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.

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

src/pages/ReverseCalc.vue Outdated
<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

@BatuevIO BatuevIO May 8, 2025

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

@Sveteeen Sveteeen May 11, 2025

Choose a reason for hiding this comment

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

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

Copy link

@Semedanik Semedanik May 11, 2025

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
@BatuevIO BatuevIO linked an issue Feb 26, 2026 that may be closed by this pull request
[ДОРАБОТКА] Добавить хинты в обратный калькулятор #15
Open
@Dmatrushka19
Сделал определение ширины экрана реактивным через useDisplay;
7a90a2b 
Добавил обработчик "." и ",", чтобы корректно обрабатывать введенную инфу;
Добавил hasInput с trim() для корректной обработки пустого/пробельного ввода;
использовал тултипы для выведения информации о выплатах (реализовал корректно, в том числе разобрался с иконкой);
@Dmatrushka19
стили
ffd2bb1
@Dmatrushka19
Copy link

Dmatrushka19 commented Mar 4, 2026

Последние обновления:

  • Добавил реактивное определение ширины: заменил screenWidth = window.innerWidth на useDisplay() и computed isNarrow, чтобы размер шрифта корректно обновлялся

  • Нормализовал ввод суммы: добавил parsedSum (trim + замена , на . + проверка Number.isFinite), чтобы одинаково обрабатывались значения с запятой и точкой.

  • Переписал вычисления поиска комбинации: found и recount теперь работают от parsedSum

  • Добавил hasInput с trim() для корректной обработки пустого/пробельного ввода.

  • Реализовал тултипы для выведения информации о выплатах (исправил на корректный вариант, в том числе разобрался с иконкой)

Дальше ждем дизайн и продуктовую постановку

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BatuevIO BatuevIO BatuevIO requested changes

@Semedanik Semedanik Awaiting requested review from Semedanik

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[ДОРАБОТКА] Добавить хинты в обратный калькулятор

4 participants

@Sveteeen @BatuevIO @Dmatrushka19 @Semedanik