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.

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

Update branding doc #11020

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update branding doc #11020

wants to merge 1 commit into from

Conversation

drewnoakes
Copy link
Member

A few minor changes:

  • Removed statement that the logo was designed by Immo (sorry Immo!).
  • Added pixel dimensions of images.
  • Moved images out of headings, as they looked like they belonged to the previous section.

@drewnoakes
Update branding doc
d8fd89e 
A few minor changes:

- Removed statement that the logo was designed by Immo (sorry Immo!).
- Added pixel dimensions of images.
- Moved images out of headings, as they looked like they belonged to the previous section.
@drewnoakes drewnoakes added the Documentation Issues about docs, including errors and areas we should extend (this repo and learn.microsoft.com) label Nov 22, 2024
@drewnoakes
Copy link
Member Author

On the NuGet icon, the docs suggest an image size of 128x128px. From https://learn.microsoft.com/en-us/nuget/reference/nuspec#icon:

Image file size is limited to 1 MB. Supported file formats include JPEG and PNG. We recommend an image resolution of 128x128.

Also, is there an SVG version of the logo that can be added here?

@YuliiaKovalova
Copy link
Member

On the NuGet icon, the docs suggest an image size of 128x128px. From https://learn.microsoft.com/en-us/nuget/reference/nuspec#icon:

Image file size is limited to 1 MB. Supported file formats include JPEG and PNG. We recommend an image resolution of 128x128.

Also, is there an SVG version of the logo that can be added here?

Hi Drew,

We don't have 128x128, only 64x64 or 256x256
and no svg format in the initial set. Is it necessary to request these pics?

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

@YuliiaKovalova YuliiaKovalova Awaiting requested review from YuliiaKovalova

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
Documentation Issues about docs, including errors and areas we should extend (this repo and learn.microsoft.com)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@drewnoakes @YuliiaKovalova