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

Update UI for summary page #113

Open
SirusCodes opened this issue Nov 10, 2022 · 7 comments
Open

Update UI for summary page #113

SirusCodes opened this issue Nov 10, 2022 · 7 comments
Labels
enhancement New feature or request

Comments

@SirusCodes
Copy link
Member

Currently the summary page looks like this:

image

To something that looks like this:

image

What has changed? and Motivation

  • Removed version number (Do we need it?)
  • Removed about catchup section as it was taking up a large space and providing not much value to the reader who already know what catchups are. (We can add a one liner or add it somewhere else)
  • Removed duplicate title also worked in Feat remove duplicate title #101
  • If we are generating OG images as screenshot of summary then I feel it's better to have more content upfront than these things.

What are your views @OurTechCommunity/core

@PranavDani
Copy link
Member

image

image

Currently, About CatchUp acts as the only link between summaries, the latest summary, and OTC CatchUp.
So what I suggest is, instead of having the hamburger on the top right, we can move it to the top left across every single page so that it does not collide with the dark/light switch on summaries (This can also allow the addition of the switch across all the pages.). (Also having it on the right side feels RIGHT for some reason.)

  • This will allow users to use the hamburger across all the pages.
  • Maintains consistency across all the pages.
  • Allows a proper replacement for the above-mentioned changes.

@SirusCodes
Copy link
Member Author

@PranavDani that does makes sense. But it should a whole another issue instead of being part of this issue.

@HarshKapadia2
Copy link
Member

  • Removed about catchup section as it was taking up a large space and providing not much value to the reader who already know what catchups are. (We can add a one liner or add it somewhere else)

I think we really need at least one line explaining what OTC CatchUp sessions are. Although the probability of someone landing on a CatchUp summary page without knowing about CatchUp is low, we cannot just assume these things. (There are a lot of times when we share links with newcomers.) We can skip a section tag and just add a one line description and some links.

  • If we are generating OG images as screenshot of summary then I feel it's better to have more content upfront than these things.

We haven't decided anything about #3, so we can comment on this motivation only after that is decided.

@tusharnankani
Copy link
Member

I think we really need at least one line explaining what OTC CatchUp sessions are. Although the probability of someone landing on a CatchUp summary page without knowing about CatchUp is low, we cannot just assume these things. (There are a lot of times when we share links with newcomers.) We can skip a section tag and just add a one line description and some links.

Yes, sounds good.

@SirusCodes, would you like to work on this?

@SirusCodes
Copy link
Member Author

@HarshKapadia2

I think we really need at least one line explaining what OTC CatchUp sessions are. Although the probability of someone landing on a CatchUp summary page without knowing about CatchUp is low, we cannot just assume these things. (There are a lot of times when we share links with newcomers.) We can skip a section tag and just add a one line description and some links.

Yeah makes sense, just a single line. But need to remove that big information box.

I would make a PR for this soon.

@tusharnankani
Copy link
Member

tusharnankani commented Feb 27, 2023

Updates?

cc: @SirusCodes

@tusharnankani tusharnankani added the enhancement New feature or request label Feb 27, 2023
@SirusCodes
Copy link
Member Author

I will try to do it tomorrow, thanks for reminding @tusharnankani

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants