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

[BUG] Alert details flyout's responsiveness #296

Closed
xeniatup opened this issue Aug 4, 2022 · 1 comment · Fixed by #373
Closed

[BUG] Alert details flyout's responsiveness #296

xeniatup opened this issue Aug 4, 2022 · 1 comment · Fixed by #373
Labels
bug Something isn't working

Comments

@xeniatup
Copy link

xeniatup commented Aug 4, 2022

What is the bug?
The table and the push flyout do not resize gracefully in the Alerts by triggers list in the Alerting plugin for different screen sizes:

  1. The elements in the table header don't stack but move outside of the table
  2. On the smaller screens the table becomes unusable
  3. On the screens smaller than about 990px the flyout switches to overlay type overlapping and blocking the table.
  4. The monitor name is truncated even if there is enough space to accommodate the full name

How can one reproduce the bug?
Steps to reproduce the behavior (assuming the Alerts by triggers table is not empty):

  1. Go to the 'Alerting' plugin - this will land you on Alerts by triggers page
  2. Click on the "burger" icon in the top left corner of the screen
  3. Click on the last item in the main navigation panel Dock navigation
  4. Click on the number of alerts in the second column of the table - the flyout appears in the right part of the screen and resizes the table
  5. Resize the browser window to less than ~1700px
  6. Then change the size of your browser window to less than ~990px

What is the expected behavior?
All elements of the table should resize gracefully. The push flyout width/proportion should adjust to the screen size.
On the screens smaller than ~990px the overlay type flyout should not obscure the table (By setting ownFocus={false}, the underlying page content will be visible and clickable.). The monitor name column should be wider when screen size permits to show the names of the monitors without truncation.

What is your host/environment?

  • OS: macOS
  • Version 12.4
  • Plugins Alerting
  • Browser Chrome, Firefox, Safari

Do you have any screenshots?
Screen Shot 2022-08-03 at 11 30 42 PM

Screen Shot 2022-08-03 at 11 31 00 PM

Screen Shot 2022-08-03 at 11 31 26 PM

Screen Shot 2022-08-03 at 11 28 56 PM

Screen Shot 2022-08-04 at 12 03 42 AM

Do you have any additional context?
The problems with responsiveness seem to be amplified by introducing the dock navigation feature in the main menu in combination with the push flyout use. The pushMinBreakpoint for the overlay flyout might need to be reviewed along with the docked navigation behavior for the small screens.

@xeniatup xeniatup added bug Something isn't working untriaged labels Aug 4, 2022
@AWSHurneyt
Copy link
Collaborator

Good catch; there are a few elements that don't seem to resize gracefully. I'll look into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants