Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Feedback: UI 💬 #1314

Closed
4 of 6 tasks
6543 opened this issue Oct 21, 2022 · 50 comments · Fixed by #2029 or #2140
Closed
4 of 6 tasks

Feedback: UI 💬 #1314

6543 opened this issue Oct 21, 2022 · 50 comments · Fixed by #2029 or #2140
Labels
good first issue Likely to be an easy fix summary it's a summary for lot of issues ui frontend related ux user experience

Comments

@6543
Copy link
Member

6543 commented Oct 21, 2022

Summary of UI issues and smaller changes which can be posted as comments.
For complex improvements please open a separate issue.
Followup of #468, witch helped quite a lot already :)

UI improvements

@6543 6543 added ui frontend related summary it's a summary for lot of issues ux user experience labels Oct 21, 2022
@6543 6543 mentioned this issue Oct 21, 2022
13 tasks
@6543 6543 pinned this issue Oct 21, 2022
@6543

This comment was marked as resolved.

@6543

This comment was marked as outdated.

@6543

This comment was marked as outdated.

@6543

This comment was marked as resolved.

@6543
Copy link
Member Author

6543 commented Oct 21, 2022

@qwerty287:

It would be nice to see public projects of other users on their "profile" list.


#468 (comment)

@6543
Copy link
Member Author

6543 commented Oct 22, 2022

image

move matrix lable in same column as name cc @lukashass

@6543 6543 added the good first issue Likely to be an easy fix label Oct 22, 2022
@6543

This comment was marked as resolved.

@qwerty287

This comment was marked as resolved.

@anbraten

This comment was marked as outdated.

@lafriks

This comment was marked as outdated.

@anbraten

This comment was marked as resolved.

@gapodo

This comment was marked as resolved.

@anbraten
Copy link
Member

anbraten commented Nov 19, 2022

image

Missing margin for pipelines with matrix and single workflow

@6543
Copy link
Member Author

6543 commented Nov 22, 2022

If a pipeline needs aproval, it does not show a link back to the forge, so you manually have to figure out what the pull is :(

so it encureage to just aprove and look afterwards what was the change ... !!!!!!

@ivictbor
Copy link

Most critical blocker for us now ant Devforth that we constantly receive "Page Unresponsive" when we try to check logs of build with lot's of line.
2 possible issues:

  1. semas like Woodpecker tries to load all lines of logs into DOM, without pagination. Possible fix is to add "On Scroll" pagination, which initially loads e.g. 300 lines, then when user scrolls them it loads next 300 if user again continues scrolling it might unload first 300 lines and so on. If it is pretty hard to implement, probably just having old-school button pagination could help here
  2. Seams like when tool produces show-bar indicator (like wget shows % of downloaded file), woodpecker does not respect \r only return, and always adds new line to log (even without \n) so pseudo-UI progress bars like from 0% to 100% turn in to MiBs of logs. We always disable such progress bars on wget and everywhere where we could, but it is not always easily, e.g. now we use it to build flutter app and flutter builder always shows spinner with \r carriage return only, so log is always totally dead there

image

@anbraten

This comment was marked as resolved.

@ivictbor

This comment was marked as resolved.

@6543 6543 unpinned this issue Dec 6, 2022
@6543

This comment was marked as resolved.

@MFTabriz

This comment was marked as duplicate.

@6543

This comment was marked as resolved.

@MFTabriz
Copy link

MFTabriz commented Jun 5, 2023

In matrix builds, name of the variable (here, "SOMETHING") should be vertically centered.
image

@6543

This comment was marked as resolved.

@6543

This comment was marked as outdated.

@6543 6543 reopened this Jul 24, 2023
@dR3b
Copy link

dR3b commented Jul 25, 2023

@6543 In the following screenshots, I will show you my suggestion for improvements:

  1. The font is way too big. On the left, as mainly that of the console.
  2. Why not use the space on the left so that the name is not cut off?
  3. An icon for the status of the entire job would be useful here.
  4. Always show the log file download dialog.

Take a look at the screenshot of Drone. The overall picture is more coherent.

20230725-082257
20230725-082350

Thanks!

@xoxys

This comment was marked as resolved.

@ambroisie

This comment was marked as duplicate.

@qwerty287
Copy link
Contributor

The pipeline counter has a quadrat background but a circle foreground

grafik

@qwerty287

This comment was marked as duplicate.

6543 pushed a commit that referenced this issue Aug 7, 2023
Was requested
[here](#1314 (comment))
for queue, but this also applies to agents: Do not show badges that only
show `???` because they don't provide any useful information.
@6543
Copy link
Member Author

6543 commented Aug 7, 2023

image

make it klickable and go to pipeline list filtered by branch e.g.: https://ci.woodpecker-ci.org/repos/3780/branches/release%2Fv1.0 in this case

@xoxys
Copy link
Member

xoxys commented Aug 7, 2023

The pipeline counter has a quadrat background but a circle foreground

grafik

@qwerty287 Am I blind 😅 ? The background and hover are squared, just the spinner is round, or what do you mean?

@qwerty287
Copy link
Contributor

@xoxys yes, I mean the spinner. Would be nice if it would spin around the background element as it was before with a round background

@xoxys
Copy link
Member

xoxys commented Aug 8, 2023

Hmm 🤔 personally I don't like the idea and don't think this would look nice.

@qwerty287
Copy link
Contributor

Do you think the current one looks good? At least, there should be some margin between the spinner and the borders of the background element. I think it would be better to completely use rounded squares.

@xoxys
Copy link
Member

xoxys commented Aug 8, 2023

We also use circle spinners in buttons etc. so yes I think if we want to have something spinning it should be a circle, but that might just be a personal preference.

@qwerty287
Copy link
Contributor

Hmm yes, from this side you're right. But there the spinners have the margin 🙃 (so yes that would work too)

@woodpecker-ci woodpecker-ci locked and limited conversation to collaborators Aug 8, 2023
@anbraten anbraten converted this issue into discussion #2162 Aug 8, 2023
@anbraten anbraten unpinned this issue Aug 8, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
good first issue Likely to be an easy fix summary it's a summary for lot of issues ui frontend related ux user experience
Projects
None yet