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

Draft: Home page redesign prototype #610

Closed
wants to merge 6 commits into from

Conversation

just-max
Copy link
Contributor

@just-max just-max commented Jul 31, 2022

I've put together a prototype for a start page redesign (#597).

Screenshots

Desktop

desktop

Tablet and Mobile

Ideas/Background/Plan

Things that are not yet fully implemented marked with [*].

Design is based on the course overview page.

The idea for the sections (names could be changed, for now it's just what I came up with). Based on the idea of moving the things I want to see when I visit the home page close to the top and accessible in a single click.

  • Suggested: Currently live streams, then completed VoDs, sorted by most recent completed first. If the user is logged in, only show streams/VoDs from pinned or registered courses (the ones that currently show up under "My Courses") that are not watched (currently considered "watched" if marked as watched or >50% watch percentage); otherwise from all courses. When a user opens TUM-Live, they are likely to want to join a lecture that is currently streaming or watch a recently completed VoD that they missed (at least that's my thinking). [*] Probably limit to about two rows of suggestions depending on screen size.
  • My Courses: Combine pinned courses and registered courses. These are the courses a user is interested in seeing. Pinned courses are shown first. For each course ([*] if there are any streams to show), show the current live stream for the course (if there is one), then the most recent completed VoDs. [*] Limit to one row/a handful of items per course.
  • Other Courses: The user is generally not interested in these courses, so just show the same information that is shown on the current home page.
  • [*] A section at the bottom would contain the hidden courses, and be collapsed by default. As suggested in the issue, the handling for this should be unified with that for pinned courses (i.e. also saved server-side).

Courses can be [*] (un-)hidden/pinned from a dropdown menu:

image

This has been made keyboard-navigation-friendly too.

I added friendly dates (e.g. "Streamed 1 hour ago" or "Scheduled tomorrow") with Luxon. Personally I think this adds some polish, and the exact date/time is shown by hovering (i.e. on the title). The current home page kind of does this for upcoming streams. [*] This should be made dynamic, which should be doable since it's contained in a component, but I haven't thought about how to do that best.

I've also moved a whole bunch of components into reusable templates. Not sure if it could be done better with the template engine, maybe someone has better ideas.

TODO/Next Steps

  • Just search for "TODO" on the diff :)
  • Didn't add any database queries yet, the things I needed are currently just hacked together.
  • Logged-out view could use work.
  • Tests

Very much work in progress.
@just-max just-max mentioned this pull request Jul 31, 2022
@MatthiasReumann
Copy link
Collaborator

Looks neat 💯

Some more ideas:

  1. I think it would be pretty cool to have thumbnails above the titles of VODs.
  2. We could move Other Courses to the left side (Similiar to YouTube and Twitch) so that all the important stuff is in the center.

- Display watch percentage
- Reduce the height of the header
- Always show dropdown button on touchscreens
@just-max
Copy link
Contributor Author

just-max commented Aug 2, 2022

Added thumbnails, at least in the UI :) Not sure what to actually show as a thumbnail though

image
image

@MatthiasReumann
Copy link
Collaborator

MatthiasReumann commented Aug 4, 2022

Added thumbnails, at least in the UI :) Not sure what to actually show as a thumbnail though

image image

How about some kittens?

Jokes aside, we already generate thumbnail sprites (See: #472) - maybe we could utilize that for that use-case also.

@joschahenningsen
Copy link
Sponsor Member

joschahenningsen commented Aug 4, 2022

@just-max you have contributed some amazing stuff lately, do you mind if I add you to our dev channel on matrix.in.tum.de?

@just-max
Copy link
Contributor Author

just-max commented Aug 4, 2022

Sure, I've sent you an email :>

@alexanderstephan
Copy link
Collaborator

I think we could use the existing thumbnails here, but we should probably wait for #582 as only a small portion of the videos has thumbnails (> July 22). I am rather busy this week, so we could also show a placeholder until then.

(also make LATEST panel use full width, previously blocked by dropdown menu button)
@just-max
Copy link
Contributor Author

just-max commented Aug 8, 2022

currently focused on exams, so I'll probably make (actual) progress on this again next week

for now, maybe we could come back to the kitten idea :)

image

@just-max
Copy link
Contributor Author

This has stalled a bit on my end, since I'm focused on writing my bachelor's thesis right now. If someone wants to pick this up, please feel free. Otherwise, if there's still interest, I'd get working on it again in about a month's time.

@alexanderstephan
Copy link
Collaborator

alexanderstephan commented Oct 21, 2022

Ah, that's fine. I am not that good with UI, and we've decided that we have a feature freeze for some time until all bugs in are fixed for the new semester.

At least from my side there is also a lot of interest in this feature, this would be the biggest UI update in a long time and the start page really seems dated as the UI of all the other parts was updated.

@MatthiasReumann
Copy link
Collaborator

Hey @just-max! Would you mind if I take over the task of the start page redesign?

@just-max
Copy link
Contributor Author

Yeah, go ahead! The latest version of everything is in my branch, if you want to reuse anything, but otherwise I haven't found the motivation to keep working on it, sorry about that.

@MatthiasReumann
Copy link
Collaborator

Yeah, go ahead! The latest version of everything is in my branch, if you want to reuse anything, but otherwise I haven't found the motivation to keep working on it, sorry about that.

Great. Anyways, thanks for the work!

@MatthiasReumann
Copy link
Collaborator

Closed in favor of #964

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

Successfully merging this pull request may close these issues.

None yet

5 participants