WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented May 29, 2025

Warning

Instead, update logo; search, Wikipedia article.

Reduced size of VASP logo.

The logo seems unnecessarily large.

Why?

Because:

  • the image resolution is too poor to merit that size
    At a large size, the poor resolution is highlighted.
  • the image floats within two paragraphs and across a header
    Floated images should not span paragraphs across sections (sections are demarcated by headings).
before after
before after

Related:

@wesleyboar wesleyboar added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels May 29, 2025
@wesleyboar wesleyboar marked this pull request as draft May 29, 2025 01:08
@wesleyboar wesleyboar requested review from susanunit and removed request for susanunit May 29, 2025 01:08
@wesleyboar wesleyboar added the hold Do not merge until contingencies are handled label May 29, 2025
@wesleyboar wesleyboar changed the title fix: vasp logo seems too big fix: vasp logo resolution too low May 29, 2025
@wesleyboar wesleyboar changed the title fix: vasp logo resolution too low fix: vasp logo seems too big May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request hold Do not merge until contingencies are handled

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants