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

Use larger icon on home page #769

Open
Nuru opened this issue Sep 7, 2019 · 6 comments
Open

Use larger icon on home page #769

Nuru opened this issue Sep 7, 2019 · 6 comments
Labels
feature New functionality/enhancement Stale

Comments

@Nuru
Copy link
Contributor

Nuru commented Sep 7, 2019

There is a nice big icon in the repo. Let us use it (maybe scaled down a bit) on the home/index page instead of the puny 64x56 pixel icon currently being used.

<a title="atlantis" href="{{ .CleanedBasePath }}/"><img src="{{ .CleanedBasePath }}/static/images/atlantis-icon.png"/></a>

The project homepage uses a 240x240 image, maybe that is a better choice, but please, for the hero (and only) image on the server index page, let's use something nicer than what we are currently using.

@lkysow
Copy link
Member

lkysow commented Sep 9, 2019

Hi Nuru, can you explain why you want a bigger icon there?
The idea was for it to be a logo image like on most sites.

image

Like GitHub:

image

@Nuru
Copy link
Contributor Author

Nuru commented Sep 11, 2019

@lkysow Nothing about the page layout suggests the logo is meant to be an unobtrusive logo in the top left corner like the GitHub image you posted. The image is in the "hero image" location top center with lots of white space around it, and when there are no locks it looks particularly lame.


image


Compare to runatlantis.io:


image


If you want to change the page layout so that the icon and title are at the top left edge of the page, like on GitHub or like the menu bar on runatlantis.io that would be another option, but there is so little content on the page that I think a hero image is a better choice.

Of course it is subjective, but on a big screen (not a phone), I think this looks better:


image


and by reducing the margin above the image from 10rem to 4vw, it stays responsive but has a lot less white space while not taking up much, if any, extra vertical height.

@lkysow
Copy link
Member

lkysow commented Sep 11, 2019 via email

@Nuru
Copy link
Contributor Author

Nuru commented Sep 11, 2019

Yeah without locks it doesn't look good but most users will have a lot of locks. I don't think we should take up more space with the image since unlike the runatlantis home page this page is meant to be functional.

I'm not suggesting we use the full height of the 512 pixel icon, but scaling down the big icon looks better than scaling up the little one, and the little one is too small for the layout. On my screen there are 100 pixels of white space above an icon 56 pixels high. You could make this 44 pixels of whitespace with an image 112 pixels high and have an image twice the size without taking up any more information-bearing space. I think it is odd and unattractive that the icon is smaller than the margin above it.

@lkysow
Copy link
Member

lkysow commented Sep 13, 2019

Okay, submit a PR and lets see how it looks.

This is what the UI looks like for me when I have locks:
image

@lkysow lkysow added the feature New functionality/enhancement label Sep 13, 2019
@Nuru
Copy link
Contributor Author

Nuru commented Sep 20, 2019

Okay, submit a PR and lets see how it looks.

@lkysow #783 is the PR you requested

@dosubot dosubot bot added the Stale label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality/enhancement Stale
Projects
None yet
Development

No branches or pull requests

2 participants