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

Responsive Chart Error #70

Open
Andreyra1 opened this issue Jun 22, 2022 · 2 comments
Open

Responsive Chart Error #70

Andreyra1 opened this issue Jun 22, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@Andreyra1
Copy link

The plot only adjusts to the window size when the window is less than half of the screen width.

OS: Mac OS 12.4
Browser: Any

Window is aprx. 40% of screen width:
CleanShot 2022-06-22 at 18 33 43@2x

Window is full screen:
CleanShot 2022-06-22 at 18 33 29@2x

@ngfelixl
Copy link
Owner

ngfelixl commented Jul 5, 2022

Hey, thanks for your report. It is already responsive, however the logic for the grid is the following:

:host {
  display: grid;
  grid-template-columns: repeat(auto-fill, minmax(500px, 1fr));
  grid-gap: 8px;
  margin: 12px;
  grid-template-rows: auto;
}

This means it creates more columns the larger the window gets. What would you expect to happen? I could imagine to have a select in the toolbar or below the toolbar to adjust the number of columns manually.

@ngfelixl ngfelixl added the enhancement New feature or request label Jul 5, 2022
@Andreyra1
Copy link
Author

Thank you for the reply.

Maybe creating columns makes sense only if there are multiple charts. Since when there is only one, it just doesn't use all the space available.

Or an option to adjust the number of columns manually is fine too.

@Andreyra1 Andreyra1 reopened this Jul 20, 2022
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

2 participants