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

Display "Updated" instead of "Opened" in PR list tooltip. #1070

Merged
merged 2 commits into from Jul 14, 2017

Conversation

Projects
None yet
2 participants
@grokys
Contributor

grokys commented Jul 13, 2017

The time being displayed us the UpdatedAt time but the tooltip was saying "Opened at". Change the text to match the data being shown.

Before:

image

After:

image

Fixes #1035

Display "Updated" instead of "Opened" in PR list tooltip.
The time being displayed us the `UpdatedAt` time but the tooltip was saying "Opened at". Change the text to match the data being shown.

Fixes #1035

@grokys grokys requested review from jcansdale and paladique Jul 13, 2017

@grokys grokys merged commit d210d20 into master Jul 14, 2017

5 checks passed

GitHub CLA @grokys has accepted the GitHub Contributor License Agreement.
Details
VisualStudio Build #7288543 succeeded in 96s
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
jenkins/build_log Jenkins Build Log
Details

@grokys grokys deleted the fixes/1035-pr-list-tooltip branch Jul 14, 2017

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