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

Add ability to collapse the Folder Owners box on folder and node pages. #72

Merged
merged 1 commit into from
Oct 7, 2018

Conversation

refortin
Copy link

@refortin refortin commented Apr 5, 2018

The Folder Owners box can be large on folders with many secondary owners obscuring the folder content. The changes are a proposal to make the box collapsible. Click the folder header to open and close the box.

@oleg-nenashev
Copy link
Member

Screenshots before and after the change would be useful

@oleg-nenashev oleg-nenashev self-requested a review April 5, 2018 18:18
@oleg-nenashev
Copy link
Member

Test failure is unrelated, my fault: jenkins-infra/pipeline-library#37

@oleg-nenashev
Copy link
Member

And yes, the code itself looks good to me, just need to do some manual testing

@refortin
Copy link
Author

refortin commented Apr 5, 2018

Three images showing the original Folder Owners view, the closed view and the open view.

Original View
folder-orignal

Closed View
folder-closed

Open View
folder-open

@oleg-nenashev oleg-nenashev self-assigned this Jun 16, 2018
@oleg-nenashev
Copy link
Member

I somehow forgot about this PR. Time to finally release it, I'd guess

@oleg-nenashev oleg-nenashev merged commit 48b99ad into jenkinsci:master Oct 7, 2018
@icereed
Copy link

icereed commented Nov 14, 2018

hi @oleg-nenashev , thanks for merging! Could you also release the new version since we are not able to use this plugin otherwise. We are looking very forward to using this new feature.

@KalleOlaviNiemitalo
Copy link

Too bad that Internet Explorer does not support <details>.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants