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

NLS logo size on tenant pages #554

Open
grahamjevon opened this issue Aug 9, 2024 · 3 comments
Open

NLS logo size on tenant pages #554

grahamjevon opened this issue Aug 9, 2024 · 3 comments

Comments

@grahamjevon
Copy link
Collaborator

grahamjevon commented Aug 9, 2024

Problem

Question from NLS: is there anything we can do to make the logo appear bigger on NLS tenant pages.

Image

Investigation and possible solution

We asked NLS to provide the highest res copy available, which they did.

  1. The new high-res image they have provided does not fill anymore of the logo space on Hyku. It appears the same size.

  2. I think the reason the logo appears small is because the image contains a lot of blank space around the text/image. This blank space is using up the logo space in Hyku. Presumably, the solution to making the logo appear bigger is to remove crop the logo so that the space is filled by the image/text part of the logo file and not the blank border.

Logo selection warning

The original logo they provided is preferable to the new logo they provided. This is because the new logo has a white background and the original logo has a clear background. The clear background looks better on Hyku.

@cziaarm
Copy link
Collaborator

cziaarm commented Sep 2, 2024

It is possible, but all logos would have to be bigger.

It is being restricted by some css keeping the max-width to 150px... this doesn't play so well with the landscape logos.

Can change that value, but it will be applied to all the member logos.

@cziaarm
Copy link
Collaborator

cziaarm commented Sep 10, 2024

Update max-width to 200px for logos seems to have desired affect across most tenants

@grahamjevon
Copy link
Collaborator Author

Logo is larger and looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants