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

Optimize website for wide viewports #574

Closed
1 of 2 tasks
tunetheweb opened this issue Nov 30, 2019 · 16 comments · Fixed by #1191
Closed
1 of 2 tasks

Optimize website for wide viewports #574

tunetheweb opened this issue Nov 30, 2019 · 16 comments · Fixed by #1191
Assignees
Labels
design Creating the Almanac UX development Building the Almanac tech stack good first issue Good for newcomers question Further information is requested

Comments

@tunetheweb
Copy link
Member

tunetheweb commented Nov 30, 2019

As noted in a recent PR there are a couple of inconsistencies with the home page, design, notably:

  • - It's full width, but the other pages are constrained (to 1280px or 1324 - note bringing these inline with above PR)
  • - The Start Exploring button in the top right, means the other navigation items (TOC, Contributors, and Methodology) are in a different place on the home page than every other page and "jump" as you navigate to/from this - Completed in Support translations #684

Are these items we want to fix? Or happy this should be different? The original design seemed to have a more constrained width (except for the header) though did always have the Start Exploring button in the same place as currently.

Thoughts?

@tunetheweb tunetheweb added development Building the Almanac tech stack design Creating the Almanac UX labels Nov 30, 2019
@tunetheweb tunetheweb added the question Further information is requested label Nov 30, 2019
@tunetheweb
Copy link
Member Author

I'm gonna suggest constraining the home page width to the same width as the other pages. Gimme a 👍if you agree or a 👎is you prefer it to remain full width.

@tunetheweb
Copy link
Member Author

I'm gonna suggest we move the "Start Exploring" button down from the header into the body, so we have a consistent header. Gimme a 👍if you agree or a 👎is you prefer it to remain where it is.

@tunetheweb
Copy link
Member Author

@HTTPArchive/developers anyone got an opinion on this?

@catalinred
Copy link
Member

I'm gonna suggest constraining the home page width to the same width as the other pages. Gimme a 👍if you agree or a 👎is you prefer it to remain full width.

The index shouldn't necessarily use a fixed page width as the other pages, as it can use a mix from both worlds .e.g. full width header, fixed-width featured chapter, then full-width contributors block design etc.

I do agree on the point that stuff should be implemented consistently, therefore the index page should use indeed the newly added .container used within #573 but that's another story and task.

I'm gonna suggest we move the "Start Exploring" button down from the header into the body, so we have a consistent header. Gimme a 👍if you agree or a 👎is you prefer it to remain where it is.

At this time, on the index page, the "Start Exploring" is indeed redundant as it links to /table-of-contents, as the "Table of Contents" menu item does too. So I guess this is a valid point as we have two links that go to same resource.

We should either link to the first Table of Contents chapter (to justify the "Start exploring" meaning) or just remove it completely from the header as @bazzadp suggests, as it's already in the "HTTP Archive's annual state of the web report" section below.

Thanks for raising these questions @bazzadp

@tunetheweb
Copy link
Member Author

At this time, on the index page, the "Start Exploring" is indeed redundant as it links to /table-of-contents, as the "Table of Contents" menu item does too. So I guess this is a valid point as we have two links that go to same resource.

We should either link to the first Table of Contents chapter (to justify the "Start exploring" meaning) or just remove it completely from the header as @bazzadp suggests, as it's already in the "HTTP Archive's annual state of the web report" section below.

To be clear, I have no problem with multiple links to the same page especially if some of those links are in the header and if we specially want to call them out in the body. And I actually quite like the “Start Exploring” call to action. I just think it’s current location is problematic as the menu items jump between pages so it’s not consistent so think it should be moved down out of the header.

@catalinred
Copy link
Member

To be clear as well, I was just trying to add to the conversation as you asked above for an opinion and as you know, I also raised the "jumping" header issue here.

FWIW, I still believe multiple links to the same page within a menu is a UX issue.

@tunetheweb
Copy link
Member Author

Oh within a menu I agree with you.

I just meant I like having the “Start Exploring” call to action itself, just not in the menu. It should be moved down the page IMHO. But actually, I forgot there is already another “Start Exploring” button there (in yellow)!

EF51A841-60D6-4315-B2EC-D5E77E4AD80C

So yeah we definitely don’t need it in the menu.

So think we’re agreeing with each other? 😀

@rviscomi
Copy link
Member

rviscomi commented Jan 6, 2020

I agree that the UI could be improved for ultra-wide viewports, but I don't know if fixed-width is the right solution. There are some parts of the design that seem to be optimized for full-width, for example the header background and Contributors section. Maybe the compromise is to have background graphics/colors full-width but their text context fixed?

Not sure about the Start Exploring button consistency issue. 40% of visitors land on the home page and I think it's good to have as many prominent calls to action as the UI can afford. The header links jumping is a valid point but there are other home-specific aspects of the UI that change as well (eg the header background) so I'm not too concerned about it.

@tunetheweb
Copy link
Member Author

BTW this gets worse when you add the Language switcher:

image

Without this extra CTA it's much neater IMHO with logo on left, nav in middle, and language on right:

image

So I still think we should remove it from the header.

It would also be interesting to see which of the four CTAs to table of contents is used most. @rviscomi can you enable Enhanced Link Attribution in GA->Properties and in Google Analytics tag in GTM to allow this to be seen?

@tunetheweb
Copy link
Member Author

And btw another design issue on the home page is the buttons:

  • Top right "Start Exploring" CTA is filled yellow (empty on hover).
  • Next "Start Exploring" CTA is empty (yellow on hover).
  • Featured Chapter CTA is filled dark blue (empty on hover).
  • "See all Contributors" CTA is empty (yellow on hover).
  • "Learn about our Methodology" CTA is empty (dark on hover)

Shouldn't these be more consistent UX? Or is the alternating filled/not-filled part of the design?

@tunetheweb tunetheweb changed the title Home page design question Home page design questions Jan 6, 2020
@rviscomi
Copy link
Member

rviscomi commented Jan 8, 2020

So I still think we should remove it from the header.

I have no strong feelings about this so your idea SGTM.

@rviscomi can you enable Enhanced Link Attribution in GA->Properties and in Google Analytics tag in GTM to allow this to be seen?

I don't think this will give us much better visibility without also adding unique IDs to the buttons.

Shouldn't these be more consistent UX? Or is the alternating filled/not-filled part of the design?

At the time our designer gave me a good reason for these but it's not obvious to me now. I agree that we can be more consistent.

@tunetheweb
Copy link
Member Author

I don't think this will give us much better visibility without also adding unique IDs to the buttons.

Oh good point! Yes either the link or a parent needs an I’d. Happy to turn these on if I submit a PR with this? Think it would also be good for future anyway in case a question like this arises.

@tunetheweb
Copy link
Member Author

Going to close this. We solved one of my concerns and the full width home page, while a little inconsistent, isn’t that unusual and is per the design.

@tunetheweb
Copy link
Member Author

tunetheweb commented Jul 4, 2020

Reopening this after discussion in #947

I still think we should revisit the home page on ultra-wide monitors. Should we limit this the intro paragraph and icon to a max certain width to avoid the big blank space?:

Home page on ultra-wide monitor with big grey section in the middle

Happy for background image to stretch to full width - though need to test that text remains readable at all widths and we don't lose the grey back ground for any of the white text. E.g. in this extreme example you can see the featured chapter text already has a problem that we should fix as part of this if easy:

Zoomed out home page

I'd also prefer the navigation menu to be same size as chapters for consistency and to avoid having to scroll right across the screen to use it! This may be fixed as part of #951

Also, while the contributors can have a higher max than the intro-paragraph/image it probably should still be constrained at some point as above extreme example shows.

@tunetheweb tunetheweb reopened this Jul 4, 2020
@tunetheweb tunetheweb added the good first issue Good for newcomers label Jul 4, 2020
@rviscomi rviscomi changed the title Home page design questions Optimize website for wide viewports Jul 17, 2020
@tunetheweb
Copy link
Member Author

@HTTPArchive/developers anyone with good CSS skills fancy tackling this one?

@catalinred
Copy link
Member

If that's ok, I'll take a look and try to come up with something.

@catalinred catalinred self-assigned this Aug 5, 2020
catalinred added a commit to catalinred/almanac.httparchive.org that referenced this issue Aug 10, 2020
catalinred added a commit to catalinred/almanac.httparchive.org that referenced this issue Aug 10, 2020
tunetheweb pushed a commit that referenced this issue Aug 10, 2020
* Narrow home page
#574

* The contributors uses full width grid template area.
This is redundant now.

* Fix the grid columns rendering.

* Optimize the contributors block rendering on different displays.
Ordered MQs in logical order.
Removed redundant CSS code.

* CSS Linting

* Fix off-centered .intro on mobile displays.

* Add block container, revert adding .container to other HTMLs

* Revert changes.

* Fix cut content issue.

* Remove extras due to #870

* Fix potential horizontal scroll in some scenario.
Minimize the HTML changes

* Reindenting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design Creating the Almanac UX development Building the Almanac tech stack good first issue Good for newcomers question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants