Skip to content

Comments

[JENKINS-56734] Fix prefix ending up outside of console#127

Open
faircreek wants to merge 1 commit intojenkinsci:masterfrom
faircreek:master
Open

[JENKINS-56734] Fix prefix ending up outside of console#127
faircreek wants to merge 1 commit intojenkinsci:masterfrom
faircreek:master

Conversation

@faircreek
Copy link

@faircreek faircreek commented Mar 13, 2019

See JENKINS-56734.

Pipeline prefixes how it looked in 2.25
pipeline_prefix_how_it_looks_in_2 25

How it looks in 2.32
pipeline_prefix_bug_2

This might be somewhat ok with short prefixes but with long ones it becomes hard to read.
pipeline_prefix_bug

It's a very small change to make it look like in 2.25.

@dwnusbaum
Copy link
Member

Thanks for the PR! See #111 for additional context around the change that added the current visualization after the log-handling rewrite removed the branches from the log itself. If we make this change I feel like it might make more sense to just move the branches back into the log itself so that they are also searchable via ctrl+f in most browsers. CC @jglick

@faircreek
Copy link
Author

Yep that is probably the easiest. In re to #111 that I didn't read before, I did actually figuring out what you tried to achieve. Better is probably two "columns" one right adjusted for the prefixes and one left adjusted for the log. The whole thing does need to be in the console area though. This is doable by calculating string lengths and the adjust the padding dynamically. The question is how it will look with very long prefixes, line wrapping and such.

@jglick
Copy link
Member

jglick commented Mar 18, 2019

Rather than shifting the placement it may make more sense to trim long branch names and make the full version available via tooltip or similar.

Searchability is an independent question.

@dwnusbaum dwnusbaum changed the title Fix prefix ending up outside of console [JENKINS-56734] Fix prefix ending up outside of console Jan 24, 2020
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.

3 participants