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.

Sign up for GitHub

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

Jump to bottom

sometimes some charts in Statistics breaks the layout #849

Closed
lizable opened this issue Dec 8, 2020 · 0 comments · Fixed by #850
Closed

sometimes some charts in Statistics breaks the layout #849

lizable opened this issue Dec 8, 2020 · 0 comments · Fixed by #850
Assignees
@lizable
Labels
area:ux UI / UX issue. effort:easy Need to understand only a specific region of codes (good first issue, easy). type:fix Fix features that are not working urgency:blocker IT SHOULD BE RESOLVED BEFORE DISTRIBUTING
Milestone
20.12

Comments

@lizable
Copy link
Contributor

lizable commented Dec 8, 2020

Describe the bug
In range of 700px and 1015px of screen width, sometimes chart display overlaps the panel.
Also, almost every chart panel except cpu and memory part has cutting off issue due to unbalanced height between canvas and wrapping element.

To Reproduce
Steps to reproduce the behavior:
문제를 재현하기 위한 순서를 자세히 적어주세요:

  1. Go to 'Statistics' page.
  2. Stretch or Shrink screen of the browser in both horizontally and vertically.
  3. See the error.

Expected behavior
Chart of each component (e.g. cpu, memory, I/O write) should be placed in the panel.

Screenshots
chart_overlaps
@lizable lizable added type:fix Fix features that are not working effort:easy Need to understand only a specific region of codes (good first issue, easy). area:ux UI / UX issue. urgency:blocker IT SHOULD BE RESOLVED BEFORE DISTRIBUTING urgency labels Dec 8, 2020
@lizable lizable added this to the 20.12 milestone Dec 8, 2020
@lizable lizable self-assigned this Dec 8, 2020
lizable added a commit that referenced this issue Dec 8, 2020
@lizable
fix: responsive layout for each chart in statistics page(#849)
2d68bad
@lizable lizable mentioned this issue Dec 8, 2020
Merged
inureyes pushed a commit that referenced this issue Dec 9, 2020
@lizable
fix: responsive layout for each chart in statistics page(#849) (#850)
edd55c4
adrysn pushed a commit that referenced this issue Dec 9, 2020
@lizable @adrysn
fix: responsive layout for each chart in statistics page(#849) (#850)
c1a20ce
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lizable lizable

Labels
area:ux UI / UX issue. effort:easy Need to understand only a specific region of codes (good first issue, easy). type:fix Fix features that are not working urgency:blocker IT SHOULD BE RESOLVED BEFORE DISTRIBUTING
Projects
None yet
Milestone
20.12
Development

Successfully merging a pull request may close this issue.

fix: responsive layout for each chart in statistics page lablup/backend.ai-webui
1 participant
@lizable