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

Chrome: Adding the post external link component #971

Closed
wants to merge 1 commit into from

Conversation

youknowriad
Copy link
Contributor

closes #969

I'm using a "title" instead of a tooltip (There's a current PR for tooltips).

@youknowriad youknowriad self-assigned this Jun 1, 2017
@youknowriad youknowriad requested review from jasmussen and ellatrix June 1, 2017 10:15
@ellatrix
Copy link
Member

ellatrix commented Jun 1, 2017

How do I test? I'm not seeing any added UI.

@youknowriad
Copy link
Contributor Author

youknowriad commented Jun 1, 2017

@iseulde did you save the post. (Sorry it's not clear, but yeah the link is showing only for saved posts)
screen shot 2017-06-01 at 15 24 05

@mtias mtias added the General Interface Parts of the UI which don't fall neatly under other labels. label Jun 1, 2017
@youknowriad
Copy link
Contributor Author

youknowriad commented Jun 5, 2017

@jasmussen What's the status for this, should we close for now?

@jasmussen
Copy link
Contributor

Let's keep it around, but unmerged. I still have a hunch we might need it, but it'd be good to blog with this thing for a bit to know for sure. Thanks for working on it.

@youknowriad
Copy link
Contributor Author

Closing as the issue was closed too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Interface Parts of the UI which don't fall neatly under other labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UI: Add external link button
4 participants