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

adds links to assignee's github profiles #246

Merged
merged 1 commit into from
Sep 18, 2017

Conversation

ZooeyMiller
Copy link
Collaborator

Turns the assignee images into links to the assignee's github profile, and makes it to that when the cursor hovers an assignee's image, the assignee's github username shows.

fix #245

@codecov
Copy link

codecov bot commented Sep 15, 2017

Codecov Report

Merging #246 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #246   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           7      7           
  Lines          82     82           
=====================================
  Hits           82     82

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 579bb4d...0b11712. Read the comment docs.

@@ -53,7 +53,9 @@
</td>
<td class="tc">
<%= for assignee <- generate_avatar(issue) do %>
<img src="https://www.github.com/<%= assignee %>.png" alt="<%= assignee %>'s github avatar'" class="w1plus br2">
<a href="https://www.github.com/<%= assignee %>" title="<%= assignee %>">
<img src="https://www.github.com/<%= assignee %>.png" alt="<%= assignee %>'s github avatar'" class="w1plus br2">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This avatar png link is much nicer than what I've been using 👍

We should be referencing images like this in the future.

(From here, https://github.com/shouston3.png would have been so much nicer than https://avatars3.githubusercontent.com/u/15983736?v=4&u=8b6cfb76f2bf2c65dc0b215e179abe1d4cf9c42a&s=400)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I was so surprised when I discovered it! I didn't even know redirect urls could be used as img src.

@samhstn samhstn removed their assignment Sep 15, 2017
@samhstn samhstn removed the in-review label Sep 15, 2017
Copy link
Collaborator

@finnhodgkin finnhodgkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@finnhodgkin finnhodgkin merged commit 8a0d92c into master Sep 18, 2017
@finnhodgkin finnhodgkin deleted the enhance-assignee-avatar branch September 18, 2017 12:00
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.

Make assignee avatars links to github profiles
4 participants