Skip to content

Update logos page in docs#9167

Merged
jrbourbeau merged 7 commits intodask:mainfrom
scharlottej13:update-logos
Jun 7, 2022
Merged

Update logos page in docs#9167
jrbourbeau merged 7 commits intodask:mainfrom
scharlottej13:update-logos

Conversation

@scharlottej13
Copy link
Copy Markdown
Contributor

  • Passes pre-commit run --all-files

Update logos page in Dask docs (see #9160 for more context).

Does anyone know if these logos were used outside this docs page? I didn't see any mention elsewhere in dask/dask.

I don't think we have an updated equivalent for with padding (they are all "no pad"). Will this be an issue?

@github-actions github-actions Bot added the documentation Improve or add to documentation label Jun 6, 2022
Copy link
Copy Markdown
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

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

Thanks @scharlottej13!

Does anyone know if these logos were used outside this docs page? I didn't see any mention elsewhere in dask/dask.

There's also the marketing.dask.org page. See https://marketing.dask.org/en/latest/logos.html for the section about logos.

I don't think we have an updated equivalent for with padding (they are all "no pad"). Will this be an issue?

My sense is this won't be a big issue. But cc @jacobtomlinson @jsignell for thoughts

@scharlottej13
Copy link
Copy Markdown
Contributor Author

Thank you @jrbourbeau!

There's also the marketing.dask.org page. See https://marketing.dask.org/en/latest/logos.html for the section about logos.

I think marketing.dask.org is getting a new home at dask.org/style-guide-- I'll add a link to logos.rst to direct people there as well.

Copy link
Copy Markdown
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

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

Thanks for the info @scharlottej13. I opened dask/marketing#13 to see if we should go ahead and archive the dask/marketing repo then.

I left one minor comment, otherwise this looks good to go

Comment thread docs/source/logos.rst Outdated
Co-authored-by: James Bourbeau <jrbourbeau@users.noreply.github.com>
Comment thread docs/source/logos.rst Outdated
Comment thread docs/source/logos.rst
.. image:: images/dask_stacked_white.svg
:alt: Dask logo.
.. image:: images/dask_horizontal_on_blue.svg
:alt: Dask logo to use on a blue background.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is kind of unrelated, but the janelia logo below seems out of place. I think we should just remove it from this page.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sounds good, I'll remove it. Do we need to include it elsewhere? I'm not too familiar with the context of this logo.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think keeping it in the assets and just not having it on this page makes sense.

Copy link
Copy Markdown
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

I think ultimately we need to get the designers to add the logos to the style guide page on the main website and then redirect this docs page over there.

But this is nice to have in the meantime.

Copy link
Copy Markdown
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

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

Thanks @scharlottej13. Merging as the docs build has finished and the changes here don't impact tests

@jrbourbeau jrbourbeau merged commit 3ab687b into dask:main Jun 7, 2022
@scharlottej13 scharlottej13 deleted the update-logos branch June 7, 2022 16:35
@AdiReske
Copy link
Copy Markdown

AdiReske commented Jun 7, 2022

The branding is here https://www.dask.org/brand-guide. I'm not sure how this page helps https://www.dask.org/style-guide

@scharlottej13
Copy link
Copy Markdown
Contributor Author

scharlottej13 commented Jun 7, 2022

The branding is here https://www.dask.org/brand-guide. I'm not sure how this page helps https://www.dask.org/style-guide

Ah, my mistake! Since we have https://www.dask.org/brand-guide, should we do a redirect instead? (sorry @jacobtomlinson if that's what you meant in your earlier comment).

Mistakes to fix (if we choose to keep this page):

@jacobtomlinson
Copy link
Copy Markdown
Member

Yup looks like /style-guide was the wrong link and we should be directing to /brand-guide. Sorry about that.

I'll update the cloudflare rules tomorrow.

cc @jsignell

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

Labels

documentation Improve or add to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants