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.

Sign up for GitHub

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

Jump to bottom

remove pin, tweak place label #941

Merged
merged 2 commits into from
Jun 16, 2017
Merged

remove pin, tweak place label #941

merged 2 commits into from
Jun 16, 2017

Conversation

brendansudol
Copy link
Contributor

@jeremiak
Copy link
Contributor

jeremiak commented Jun 15, 2017 •
edited
Loading

@brendansudol I like this, what do you think about right aligning the text and including the state?

screen shot 2017-06-15 at 3 29 25 pm

@brendansudol
Copy link
Contributor Author

i have no pref on alignment, thoughts @AvivaOskow?

@jeremiak
Copy link
Contributor

I made the change but we can always open a new issue to change it going forward.

@jeremiak jeremiak merged commit 294c846 into master Jun 16, 2017
@jeremiak jeremiak deleted the bjs-remove-pin branch June 16, 2017 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brendansudol @jeremiak