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

Add Twitter layer #2837

Closed
wants to merge 1 commit into from
Closed

Conversation

CestDiego
Copy link
Contributor

No description provided.

@CestDiego
Copy link
Contributor Author

@syl20bnr I rebased this.

@syl20bnr
Copy link
Owner

Can you resize the logo to 200px height ?

@bandali0
Copy link

Look forward to seeing this merged!

@CestDiego As @syl20bnr mentioned can you please resize the logo to 200px, and also use the new twitter logo instead of the old one? (See here)

@CestDiego
Copy link
Contributor Author

It has been rebased and changed to 200px

@bandali0
Copy link

@CestDiego Fantastic. @syl20bnr Can we have this merged please? 😃

@StreakyCobra
Copy link
Contributor

@CestDiego You probably have to adapt the README to the new template (like :noexport: tag for TOC).

@CestDiego
Copy link
Contributor Author

@StreakyCobra I modified the tag :)

@tsenart
Copy link

tsenart commented Nov 9, 2015

+1

@nosami
Copy link

nosami commented Jan 3, 2016

+1

@syl20bnr syl20bnr self-assigned this Feb 15, 2016
@cpaulik
Copy link
Contributor

cpaulik commented Feb 29, 2016

This will need a few keybindings since SPC is used by twittering-mode. See also #5304.

@syl20bnr
Copy link
Owner

Cherry-picked into develop branch !!
I improved it a bit in commit 92cc50b

Thank you 👍

@syl20bnr syl20bnr closed this Mar 24, 2016
@syl20bnr syl20bnr removed the Merged label Mar 24, 2016
@nixmaniack
Copy link
Contributor

Awesome! I can remove my layer now. 😄

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.

8 participants