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

Expand when only one graph is selected #1988

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

s7tya
Copy link
Contributor

@s7tya s7tya commented Oct 8, 2024

closes #1938

Though it's bit tricky that I'm using some elements' width and height to calculate graphs size, it might be somewhat useful.

CleanShot 2024-10-09 at 01 01 21

@s7tya s7tya force-pushed the expand-if-only-one-graph-selected branch from 85b4e0f to 92bc848 Compare October 8, 2024 16:08
@s7tya s7tya force-pushed the expand-if-only-one-graph-selected branch from 92bc848 to e3f05d5 Compare October 8, 2024 16:13
@Kobzol
Copy link
Contributor

Kobzol commented Oct 9, 2024

Thank you! I tried to make the calculations a bit simpler, and tested it with 1/4 columns on desktop/mobile. Let me know what you think!

@s7tya
Copy link
Contributor Author

s7tya commented Oct 9, 2024

Thanks! I think it looks great on mobile screens, but on larger screens, isn't the height getting fixed at 300px?

CleanShot 2024-10-10 at 08 09 24@2x

@Kobzol
Copy link
Contributor

Kobzol commented Oct 10, 2024

It should not be (const height = countGraphs === 1 ? window.innerHeight - top - 100 : 300;). Did you reload the page after going out of the mobile view? It needs to be refreshed.

It looks like this on my side:
image

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.

Improve rendering of history graphs
2 participants