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

Expected hours tooltip #378

Merged
merged 7 commits into from
Mar 10, 2020
Merged

Expected hours tooltip #378

merged 7 commits into from
Mar 10, 2020

Conversation

georgiashay
Copy link
Collaborator

Closes #224

Shows a tooltip table that breaks down expected hours by quarter and subject. Useful to see how hard each quarter will be, and also to see where large amounts of hours come from in your schedule.

image

Copy link
Collaborator

@npfoss npfoss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a nice Vuetify way to do this table? I think it's a little too 'raw-html aesthetic' compared to the rest of the UI

The information is great though, super useful.

@georgiashay
Copy link
Collaborator Author

They have one in Vuetify 2, but the one for 1.5 doesn't seem like it can easily support merged cells + vertical headers.

I put it in the 1.5 data table and it looks okay-ish, but I can't make it small so I honestly like the raw html one better. If we can figure out how to make it small it would look better though.

@npfoss
Copy link
Collaborator

npfoss commented Feb 29, 2020

Hmm I see... Alright can you just put it back to the raw html version and I'll spend some time styling it manually? Sorry for the wild goose chase, but you're right, Vuetify is worse here

Copy link
Collaborator

@npfoss npfoss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made some tweaks to the table:
image

If you think that looks good, I think this is ready to merge

@georgiashay
Copy link
Collaborator Author

Looks good to me

@npfoss npfoss merged commit a79af36 into master Mar 10, 2020
@npfoss npfoss deleted the expected-hours-tooltip branch March 10, 2020 04:09
miriam-rittenberg pushed a commit that referenced this pull request Oct 21, 2020
* Tooltip shows table of hours broken down by class and quarter

* Only show one quarter if classes are only in one quarter

* Test putting in vuetify table

* lint

* Remove vuetify table

* styling for table

* lint, remove unused

Co-authored-by: Nate Foss <[email protected]>
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.

Handle expected hours differently for quarter classes
2 participants