Showing results for 
Search instead for 
Did you mean: 
Sign up Log in

Earn badges and make progress

You're on your way to the next level! Join the Kudos program to earn points and save your progress.

Deleted user Avatar
Deleted user

Level 1: Seed

25 / 150 points

Next: Root


1 badge earned


Participate in fun challenges

Challenges come and go, but your rewards stay with you. Do more to earn more!


Gift kudos to your peers

What goes around comes around! Share the love by gifting kudos to your peers.


Rise up in the ranks

Keep earning points to reach the top of the leaderboard. It resets every quarter so you always have a chance!


Come for the products,
stay for the community

The Atlassian Community can help you and your team get more value out of Atlassian products and practices.

Atlassian Community about banner
Community Members
Community Events
Community Groups

"Changes Requested" pull request UI needs work

I'm finding the PR UI changes to support this feature to be non-intuitive. Changing the grey background to blue when you click Approve doesn't really convey you've approved the PR, but then the Request changes button goes away, so I guess so?

At least change the button text to "Approved".

Also, I generally Approve PRs and then Merge, and it's annoying that I now have to go dig it out of the ... button menu.

Perhaps if merge conditions are met, display the Merge button?

I'm super happy someone made a new button for us, but don't hide a button that's going to get WAY more use just to make sure we know there's a new button available.



1 comment

Brandon Reppert
Atlassian Team
Atlassian Team members are employees working across the company in a wide variety of roles.
Dec 16, 2020 • edited

Hi @Sean Lively, thanks for this feedback.

I agree with the button text changing when the buttons are selected. We'll have a change out shortly for "Approve" to become "Approved" when set and likewise for "Request changes" to become "Changes requested".

We're also considering your feedback for merge button placement for PR reviewers in conjunction with some other feedback we've seen. Our general stance is that merging is a primary action for authors while requesting changes is a primary action for reviewers and there is limited permanent real estate in the title bar, but we're aware that there are many cases where reviewers regularly merge pull requests and we're discussing what the right solution for that will be.

Brandon, thanks for the quick changes on the button.

Sadly, now I get to gripe about the file list change. The list/treeview toggle is cool, but give us an option to put it back into the Details tab. 


Log in or Sign up to comment
AUG Leaders

Atlassian Community Events