pagure

#4665 swap "Add" and "Cancel" button ordering for access management
Merged by pingou. Opened by ktdreyer.
ktdreyer/pagure add-user-buttons  into  master

Download 4665.patch

Prior to this change, the "Cancel" button was much too close to the place where users click to select an access level. This made it too easy
for users to accidentally click the "Cancel" button after quickly clicking on all the other interface choices.

Move the "Add" button to the left so that it is directly below the dropdown input field.

:thumbsup:

pretty please pagure-ci rebuild

Code looks fine

pretty please pagure-ci rebuild

@ktdreyer could you edit the PR and click on the checkbox allowing us to rebase your PR please? :)

Thanks!

pretty please pagure-ci rebuild

Ok, the tests are passing locally, so I'll go ahead and merge this manually, but since this changes the UI, I'll make a 5.8.1 release before.

Commit c44f49ef fixes this pull-request

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.