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

Adding tweet to share button on project list and detail pages, refs #357. #358

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

juliaelman
Copy link

This helps solve the initial parts of #357 by adding a Twitter share button to project list and detail pages.

@courte
Copy link
Member

courte commented Mar 6, 2015

Share button in project partial
screenshot 2015-03-06 09 07 13

Share button on project show page (pretend there's a logo instead of an ugly box)
screenshot 2015-03-06 09 08 03

@courte
Copy link
Member

courte commented Mar 11, 2015

@juliaelman, thanks for working on this last week! For now, we'd prefer to keep the tweet button on the show page only - since adding it to the project partial brings up some design decisions we'd rather not make at the moment.

I know you've got a lot going on this week! If you don't have time to make the change, don't hesitate to let me know. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants