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

[Medium] Система не считает посчитанные локаторы в счётчике над прогресс баром #1751

Open
AlexeyGirin opened this issue Jun 14, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@AlexeyGirin
Copy link
Contributor

AlexeyGirin commented Jun 14, 2024

Environment details (please complete the following information):

  • JDN v 3.16.0 Back-end v 0.3.0
  • Chrome Version 126.0.6478.62 (Official Build) (64-bit)
  • Win10

Describe the bug
Система не считает посчитанные локаторы в счётчике над прогресс баром

To Reproduce
Steps to reproduce the behavior:

  1. Откройте страничку которая не закэширована (я использую линкедин - например https://www.linkedin.com/in/valeriia-kvitkovskaia-83988b297/)
  2. Нажмите +Page Object
  3. Нажмите Generate All
  4. После того как система распознает элементы на странице - пронаблюдайте за второй фазой - расчётом локаторов

Actual behavior
Некоторые локаторы посчитаны, а счётчик - попрежнему на нуле
image

Expected behavior
Ситуация происходит т.к. CSS локаторы считаются асинхронно и дольше, поэтому счётчик сдвигается когда оба типа локаторов для элемента просчитаются.
Это неудобно.
Давайте сделаем так что бы счётчик учитывал только тот тип локатора рядом с которым крутиться спиннер

А вот для второго типа локаторов - нужно придумать как мы будем показывать что он ещё не посчитался, если пользователь откроет локатор на редактирование и попытается поменять тип - @SemyonKuznetsov


figma

Засчитываем в прогресс бар цифру когда full Xpath просчитывается

  • CSS дальше считается на фоне

Если мы хотим поменять на css в процессе просчёта на фоне, то

  • Открываем Edit локатора
  • Мы можем в предпросмотре увидеть пока ещё full CSS со спиннером
  • Можем его выбрать и получить full CSS в процессе просчёта
  • (если в этот момент его отредактировать, то останавливаем прогрузку этого локатора и сохраняем то что есть; второй вариант - дизейблим full css для редактирования)
  • Спустя время локатор обновится и получиться короткий CSS

image
image
image
image
image

@AlexeyGirin AlexeyGirin added enhancement New feature or request Design needed If design for the task isn't prepared yet labels Jun 14, 2024
@AlexeyGirin AlexeyGirin added this to the Release 3.17 milestone Jun 14, 2024
@savinovava savinovava removed this from the Release 3.17 milestone Jun 20, 2024
@savinovava savinovava moved this to BackLog in JDN.AI Board Jun 20, 2024
@SemyonKuznetsov SemyonKuznetsov moved this from BackLog to In progress in JDN.AI Board Jun 26, 2024
@SemyonKuznetsov
Copy link
Contributor

  1. Ждём пока прогрузится CSS и убираем лоадер
    • дольше чем обычно
    • зачем юзеру ждать то, что он не выбирал
  2. Засчитываем в прогресс бар цифру когда full Xpath просчитывается
    • CSS считается на фоне
    • CSS считается только при открытии Edit локатора - если стабильно и быстро
    • CSS считается только при переключении на CSS в Edit локатора

@SemyonKuznetsov
Copy link
Contributor

  • открываем на Edit
  • CSS считается всей "пачкой"
  • Мы можешь в предпросмотре увидеть full CSS с лоадером-спиннером
  • Можем его выбрать и получить full CSS в процессе просчёта
  • Спустя время локатор обновится и получиться короткий CSS

@SemyonKuznetsov SemyonKuznetsov removed the Design needed If design for the task isn't prepared yet label Jul 9, 2024
@SemyonKuznetsov SemyonKuznetsov removed their assignment Jul 9, 2024
@SemyonKuznetsov SemyonKuznetsov moved this from In progress to BackLog in JDN.AI Board Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: BackLog
Development

No branches or pull requests

3 participants