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

Don't hide other semesters #76

Open
qaisjp opened this issue Dec 12, 2018 · 10 comments
Open

Don't hide other semesters #76

qaisjp opened this issue Dec 12, 2018 · 10 comments

Comments

@qaisjp
Copy link
Member

qaisjp commented Dec 12, 2018

It turns out that #25/#35 is bad UX. I added this feature and even I sometimes forget to click "semester 2" when looking through courses.

Option 1

Show each course in their entirety. But have one section at the top (the current semester), and a second section at the bottom (for the other semester).

Option 2

Have all the courses in their correct order (alphabetically), but collapse non-current-semester courses. There could be shift+expand to expand all courses.


I'm leaning on the latter (ordering never changes, just the collapseness). Feedback appreciated.

cc @sleepyfoxen, @neanias

@neanias
Copy link
Member

neanias commented Dec 13, 2018

I would say keep the current 'View Semester {x}' buttons at the top, but sort courses into two collapsible sections for each semester. The 'view' buttons should take the user to the section anchor for that semester. Maybe a 'Back to top' button as well.

@qaisjp
Copy link
Member Author

qaisjp commented Dec 13, 2018

Back to top sounds like a good idea. What do you think about scrolling through courses alphabetically?

I guess we could do your idea for now and change it later if that feels nicer

@neanias
Copy link
Member

neanias commented Dec 13, 2018

I'd say have them sorted alphabetically within the semester section grouping. It's ok for the first 2 years when there aren't that many courses, but in later years there are lots, many of which might be irrelevant. At least there'll be fewer irrelevant courses if the user is only looking at semester 1.

@neanias
Copy link
Member

neanias commented Dec 13, 2018

I also think that having all courses visible, but having semesters collapsible, is much better UX than just completely hiding them.

@neanias
Copy link
Member

neanias commented Dec 13, 2018

A better idea for your option 2, I think, would be to change the highlighting on courses for that semester; the title being bold, for instance. Both semester and course sections should be collapsible.

@qaisjp
Copy link
Member Author

qaisjp commented Dec 13, 2018

I'd say have them sorted alphabetically within the semester section grouping. It's ok for the first 2 years when there aren't that many courses, but in later years there are lots, many of which might be irrelevant. At least there'll be fewer irrelevant courses if the user is only looking at semester 1.

Actually, interestingly enough I think there's the opposite problem.

In the first two years, there's a very small number of courses you can take, and so you don't really want the extra clutter.

In other years, there are already so many courses, so there's no point in trying to make it easier to scroll as I guess it doesn't make much of a difference(?)

I also think that having all courses visible, but having semesters collapsible, is much better UX than just completely hiding them.

Agreed.

A better idea for your option 2, I think, would be to change the highlighting on courses for that semester; the title being bold, for instance. (*) Both semester and course sections should be collapsible.

(*) I wasn't thinking of having semester sections for this option, so whilst boldness is a useful indicator, I'm not sure it'll be entirely obvious to people.


I'm also keen on getting rid of row of buttons completely (drive, sem1, sem2, view all).

In general, is this what we're leaning towards right now?

[top nav]

## sem 1                [-]
[ course ]
[ course ]
[ course ]

## sem 2 (collapsed)    [+]

@neanias
Copy link
Member

neanias commented Dec 13, 2018

Is just the current semester going to be expanded by default?

@qaisjp
Copy link
Member Author

qaisjp commented Dec 13, 2018

Yeah

@qaisjp
Copy link
Member Author

qaisjp commented Dec 13, 2018

And the courses themselves

@neanias
Copy link
Member

neanias commented Dec 13, 2018

Sounds good to me

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

No branches or pull requests

2 participants