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

make IssuesTotalColumn not visible by default #1924

Closed
wants to merge 1 commit into from

Conversation

car-roll
Copy link

@car-roll car-roll commented Jan 9, 2025

Testing done

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@uhafner
Copy link
Member

uhafner commented Jan 9, 2025

This is enabled intentionally. If you don't want to use this setting you can define your own view that does not contain the column.

@uhafner uhafner closed this Jan 9, 2025
@jglick
Copy link
Member

jglick commented Jan 9, 2025

That would be awkward in the case of AllView, and (currently) impossible in the case of MultiBranchProjectViewHolder.ViewImpl. This column takes up considerable screen real estate in a prominent place for a feature you may or may not find useful, and just shows - for folders with no usages of this plugin. Possibly Jenkins core should provide a better UX for list view columns (per-user toggles?) so that plugins do not clutter the display unless they are being actively used.

@uhafner
Copy link
Member

uhafner commented Jan 10, 2025

Yes, this UI problem in Jenkins core is quite old, see https://issues.jenkins.io/browse/JENKINS-22340. It would be really helpful if we would get rid of the non-configurable all view. I don't use it on my own, maybe we should point those users to https://wiki.jenkins.io/JENKINS/Editing-or-Replacing-the-All-View.html.

Since the warnings column is there since 2013 removing it from the default view will also awkward for all other users that like it. So I won't change that behavior.

@jglick
Copy link
Member

jglick commented Jan 13, 2025

Unfortunately replacing the AllView at root (Jenkins) is not sufficient, since you would have to do the same in every Folder (and as noted, this is not even possible in ComputedFolder). 🤔

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