Skip to content

Conversation

@mgomes
Copy link

@mgomes mgomes commented Jul 6, 2014

Twitter has recently added support for 400x400 thumbnails coinciding with their re-design. This PR adds support for that.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

@guilhermesimoes
Copy link
Contributor

I don't see that 400x400 size specified anywhere.

Also, the code original_url.sub('400x400', '400x400') does absolutely nothing. It's replacing a string with the exact same string.

@mgomes
Copy link
Author

mgomes commented Jul 6, 2014

I fixed the PR.

It doesn't look like 400x400 is documented in API v1.1. Maybe it'll come soon? 400x400 is the suffix being used for the larger profile images in the new design. Can this be added but maybe not documented until it's officially part of the API?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants