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

0.20 launch, UI display weird behaviours list #6290

Closed
3 of 5 tasks
ReLoutre opened this issue Dec 4, 2024 · 13 comments
Closed
3 of 5 tasks

0.20 launch, UI display weird behaviours list #6290

ReLoutre opened this issue Dec 4, 2024 · 13 comments
Assignees
Labels
area/frontend Needs frontend code changes bug Something isn't working good first issue Great issue for new contributors

Comments

@ReLoutre
Copy link

ReLoutre commented Dec 4, 2024

Describe the issue

Here's a quick listing of the weird UI behaviour i came across in the newly launched 0.20 of Kestra, some points may be more of a "request and feedback" type, apologies.


image
image

Tasks



image

Tasks



image
image

Tasks



image

Tasks

Environment

  • Kestra Version: 0.20.0 OSS docker compose debian
@ReLoutre ReLoutre added area/backend Needs backend code changes area/frontend Needs frontend code changes bug Something isn't working labels Dec 4, 2024
@github-project-automation github-project-automation bot moved this to Backlog in Issues Dec 4, 2024
@MilosPaunovic MilosPaunovic added good first issue Great issue for new contributors and removed area/backend Needs backend code changes labels Dec 4, 2024
@Muppasanipraneeth
Copy link

@MilosPaunovic can you assign it to me

@MilosPaunovic
Copy link
Member

Yup, go ahead! 🚀

@Muppasanipraneeth
Copy link

Muppasanipraneeth commented Dec 4, 2024

sorry @MilosPaunovic sir I though it is reactjs but it is vuejs I am not familiar with it

@ReLoutre
Copy link
Author

ReLoutre commented Dec 5, 2024

Animation
Found out that expanding then collapsing some of the right side menus fix the display of the executions legends.

@sanjaykumar584
Copy link

@MilosPaunovic I would like to work on this issue. Can you assign it to me?

@MilosPaunovic
Copy link
Member

Absolutely, go for it @sanjaykumar584! 🚀

@MilosPaunovic
Copy link
Member

Hey @ReLoutre,

First two issues from the list are amended and merged to develop.

I need more details about the third one:

As you can see with the above two pictures, hovering the filter box trigger the scrollbar which hide the majority of the filter himself, interacting with the filter is a lot more complicated.

I cannot reproduce it on Google Chrome, on Linux. Can you specify more details about browser/OS used?

Also, for the future, if you could open separate issues per problem, it would make things a lot simpler from tracking purposes.

@MilosPaunovic
Copy link
Member

Only two things remain here:

  1. Scroll issue (@ReLoutre if you could open separate issue with more details, that would be great)

  2. As you can see with the above picture, green blocks, the field "next execution date" display a date for realTime triggers, which is weird, and what's more, it display instead the "creation date" i guess.

@Ben8t could give us more info about the second point, and will open a separate issue, if necessary.

@sanjaykumar584 We had to amend those issues quickly, so I took over. If you're still up for contributing, may I suggest that you check our curated list of good first issues to see if there is something you find interesting enough and think you can tackle independently. It can be further filtered by desired labels to narrow down the results (suggesting to filter by either area/backend or area/frontend ones, based on your preferences).

I am closing this issue now.

@github-project-automation github-project-automation bot moved this from Backlog to Done in Issues Dec 6, 2024
@Ben8t
Copy link
Member

Ben8t commented Dec 6, 2024

As you can see with the above picture, green blocks, the field "next execution date" display a date for realTime triggers, which is weird, and what's more, it display instead the "creation date" i guess.

RealTimeTriggers have waitTime and conditions properties so it make sense in some cases to have "next date". I believe it's best to see this "next execution date" as the next "evaluation" of the trigger in case of RealTimeTrigger.

Still I agree, "create date" and "next execution date" should be different. Can you open a dedicated issue for this @ReLoutre please ?

@ReLoutre
Copy link
Author

ReLoutre commented Dec 6, 2024

Hi @MilosPaunovic ,
Sry for the listing thing, thought it was a good idea (fail happens).

Regarding the third point :

  • I'm mainly using Firefox
  • I'm accessing the web page through a windows 10 os
  • Problem is that interacting with crosses of the filter are a bit tricky (while it's still possible) since the hovering box of the scroll bar seems to reach the center of the filter box
    image
    image
  • New info, seems like zooming out (even by 10%, so having a zoom of 90% instead of 100%) make the scroll bar completely disappear. And while zooming in up to 200% the scroll bar disappear as well. On some pages in does not appear and some other it does. And most of the time, it appears while not being useful at all (since the filter is already completely visible, while in some case it could be useful but it doesn't appear). The behaviour of this scroll bar is highly inconsistent. Having it under the box could be good.

Useful but do not appear :
image
Useless but appear :
image

@ReLoutre
Copy link
Author

ReLoutre commented Dec 6, 2024

As you can see with the above picture, green blocks, the field "next execution date" display a date for realTime triggers, which is weird, and what's more, it display instead the "creation date" i guess.

RealTimeTriggers have waitTime and conditions properties so it make sense in some cases to have "next date". I believe it's best to see this "next execution date" as the next "evaluation" of the trigger in case of RealTimeTrigger.

Still I agree, "create date" and "next execution date" should be different. Can you open a dedicated issue for this @ReLoutre please ?

I'm on it @Ben8t !

@ReLoutre
Copy link
Author

ReLoutre commented Dec 6, 2024

Scroll issue (@ReLoutre if you could open separate issue with more details, that would be great)

Forgot to make it separate, i'll do another issue too.

@ReLoutre
Copy link
Author

ReLoutre commented Dec 6, 2024

Here's the two new issues :
#6338
#6340

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Needs frontend code changes bug Something isn't working good first issue Great issue for new contributors
Projects
Status: Done
Development

No branches or pull requests

5 participants