-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
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
Remove 'Disable project' button from project view #9287
Remove 'Disable project' button from project view #9287
Conversation
…enkinsci#9272)" This reverts commit 4623738.
/label ready-for-merge This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback. Thanks! |
I hope the community will make you bring that button back, because this change is truly horrible and the supposed few pixel gains are negligible on current displays. |
I agree. With the same arguments you could state that "Add description" should vanish as well. I'd prefer to use a different layout with one column with infos to the left and one with buttons to the right. Or move "Disable Project" to the left to the other links as well. |
My experience is in the other direction. I use a Chromebook laptop and vertical space is quite precious when I'm using that laptop. Removing the wasted vertical space gives me a better chance that the actions I frequently perform will not require that I scroll the screen downward. This change is expected to save me multiple clicks and scrolling to see useful information. In those rare cases where I want to disable a job, I click the app bar and disable it. |
This moves the disable into the configure job page as its expected to be a rare action and its already available there. Enable has been left on the page as it will be more convenient to quickly enable something and it shouldn't often show. |
Well it's half of 2024, this button has been there for the last 15 years, back in time when 1280x800 laptops were a norm and yet it is now - in the era of 4K screens everywhere - that you found this particular button space-wasting? Make it in line with "add description" - which of the two is definitely more useless and more easily movable to settings - or make it an option for the user to decide. Removing it completely is the laziest and worst decision possible. |
I would rather have it there since it doesn't take that much space anyway |
Reverts the revert in #9272
Fix for the
workflow-job-plugin
test failure here jenkinsci/workflow-job-plugin#437Discussed briefly in the last UX Sig, the 'Disable project' (and 'Add/edit description') button take up a lot of vertical space (roughly 100px) and this forces useful information, such as stage view plugin etc down.
My proposal would be just to remove the 'Disable project' button altogether and instead have users rely on the configure screen to disable projects.
Before
After
Testing done
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
@Restricted
or have@since TODO
Javadocs, as appropriate.@Deprecated(since = "TODO")
or@Deprecated(forRemoval = true, since = "TODO")
, if applicable.eval
to ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@jenkinsci/sig-ux
Before the changes are marked as
ready-for-merge
:Maintainer checklist
upgrade-guide-needed
label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidate
to be considered (see query).