Skip to content

DOC: Add favicon to doc pages #21440

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

Merged
merged 1 commit into from
Jun 13, 2018

Conversation

joelostblom
Copy link
Contributor

@joelostblom joelostblom commented Jun 12, 2018

This is a suggestion for adding a favicon to the doc pages. Having a favicon makes the pandas browser tabs easier to locate, especially with many open tabs or many pinned tabs.

I used the image from the pandas-dev GitHub organization, cropped some of the bars to make it appear less cluttered in the small favicon format, and resized it to a 32x32 .ico-file (the cropped png is attached at the end of this issue).

A few screenshots (pinned and normal tabs, alternating with and without the favicon):

Firefox Quantum
image

Chromium
image

Firefox pre-Quantum
image

This is somewhat related to #21376, in that the favicon would change if a new logo is decided upon.

Cropped png
favicon

  • tests added / passed
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

@joelostblom joelostblom changed the title [DOC] Add favicon to doc pages DOC: Add favicon to doc pages Jun 12, 2018
@codecov
Copy link

codecov bot commented Jun 12, 2018

Codecov Report

Merging #21440 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #21440   +/-   ##
=======================================
  Coverage   91.89%   91.89%           
=======================================
  Files         153      153           
  Lines       49600    49600           
=======================================
  Hits        45580    45580           
  Misses       4020     4020
Flag Coverage Δ
#multiple 90.29% <ø> (ø) ⬆️
#single 41.86% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 92e9882...c0b1eef. Read the comment docs.

@gfyoung gfyoung added the Docs label Jun 13, 2018
Copy link
Member

@gfyoung gfyoung left a comment

Choose a reason for hiding this comment

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

I don't see why not. LGTM!

@jorisvandenbossche jorisvandenbossche merged commit 11a26be into pandas-dev:master Jun 13, 2018
@jorisvandenbossche
Copy link
Member

Thanks!

@jorisvandenbossche jorisvandenbossche added this to the 0.24.0 milestone Jun 13, 2018
david-liu-brattle-1 pushed a commit to david-liu-brattle-1/pandas that referenced this pull request Jun 18, 2018
Sup3rGeo pushed a commit to Sup3rGeo/pandas that referenced this pull request Oct 1, 2018
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.

3 participants