Skip to content

Updates #21

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
Merged
merged 7 commits into from
Apr 21, 2020
Merged

Updates #21

merged 7 commits into from
Apr 21, 2020

Conversation

radarhere
Copy link
Member

  • Corrected the canonical URL. See also 2020 #20 (comment).
  • Restored the previous 'description' meta tag, instead of just having an empty string
  • Removed the 'author' meta tag
  • Removed the middle 'View details' button, since it didn't link to anything. Credit to 2020 #20 (comment)
  • Added 'target="_blank"' to the one link that doesn't have it. Credit to 2020 #20 (comment)
  • Removed some use of the string 'example'
  • Corrected the navbar height. Aside from just being correct now, it also stops the central black box from touching the navbar on mobile.

hugovk
hugovk approved these changes Apr 21, 2020
View reviewed changes
Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Perhaps also remove the fullstop from this button?

image

@aclark4life aclark4life merged commit ab9f438 into python-pillow:master Apr 21, 2020
@aclark4life
Copy link
Member

Thanks!

@radarhere radarhere deleted the updates branch April 21, 2020 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hugovk hugovk hugovk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@radarhere @aclark4life @hugovk