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

closes #5548 add logo url to header #5558

Closed

Conversation

Nrestrepo05
Copy link

Summary
Add logo to the header as asked here: #5548

A picture of a cute animal (not mandatory but encouraged)
82712

@Nrestrepo05 Nrestrepo05 requested a review from a team June 30, 2021 18:30
@erezrokah erezrokah added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Jul 1, 2021
@erezrokah
Copy link
Contributor

erezrokah commented Jul 5, 2021

Thanks @Nrestrepo05, this logo seems a bit too big in my opinion:
image

I'll reduce the height a bit

Copy link
Contributor

@erezrokah erezrokah left a comment

Choose a reason for hiding this comment

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

Hi @Nrestrepo05, after going the PR again, it seems we should display the logo as a part of the SettingsDropdown left to the display URL:
https://github.com/netlify/netlify-cms/blob/18724ff991c4b8b16de1df40511dad368578a4b2/packages/netlify-cms-core/src/components/UI/SettingsDropdown.js#L74

That way it will also be present inside the editor. WDYT?

I'm aware SettingsDropdown doesn't reflect the exact functionality described by the name of that component

@erezrokah
Copy link
Contributor

Closing this as stale. Please comment if you'd like to pick this up

@erezrokah erezrokah closed this Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants