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

Update the header for pull request description #1533

Merged
merged 11 commits into from Mar 15, 2018

Conversation

@donokuda
Copy link
Member

@donokuda donokuda commented Mar 13, 2018

This is the first of a set of changes to the pull request details view based on discussions in #1452:

  • Move "View conversation" link to above the description
  • Make the description area collapsable
  • Move PR Checkout features outside of the header
  • Update Base / Target branch UI

Before

After

screen shot 2018-03-13 at 9 38 08 am

@grokys
grokys approved these changes Mar 15, 2018
@grokys grokys merged commit b2683ae into feature/pr-reviews Mar 15, 2018
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/branch AppVeyor build failed
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
@grokys grokys deleted the ui/pr-detail-new-layout branch Mar 15, 2018
grokys added a commit that referenced this pull request Mar 19, 2018
Update the header for pull request description.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.