[ZBX-19531] Data overview head vertical aligin issue Created: 2021 Jun 09 Updated: 2025 Feb 18 Resolved: 2023 Nov 03 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | Frontend (F) |
Affects Version/s: | 5.4.1 |
Fix Version/s: | 6.0.24rc1, 6.4.9rc1, 7.0.0alpha8, 7.0 (plan) |
Type: | Problem report | Priority: | Minor |
Reporter: | Priechodsky Dušan | Assignee: | Alexander Shubin |
Resolution: | Fixed | Votes: | 2 |
Labels: | dataoverview, widget | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: |
![]() ![]() ![]() ![]() ![]() |
||||||||||||||||||||
Issue Links: |
|
||||||||||||||||||||
Team: | |||||||||||||||||||||
Sprint: | Sprint 77 (Jun 2021), Sprint 78 (Jul 2021), Sprint 79 (Aug 2021), Sprint 80 (Sep 2021), Sprint 81 (Oct 2021), Sprint 82 (Nov 2021), Sprint 83 (Dec 2021), Sprint 84 (Jan 2022), Sprint 85 (Feb 2022), Sprint 86 (Mar 2022), Sprint 87 (Apr 2022), Sprint 88 (May 2022), Sprint 89 (Jun 2022), Sprint 90 (Jul 2022), Sprint 91 (Aug 2022), Sprint 92 (Sep 2022), Sprint 93 (Oct 2022), Sprint 94 (Nov 2022), Sprint 95 (Dec 2022), Sprint 96 (Jan 2023), Sprint 97 (Feb 2023), Sprint 98 (Mar 2023), Sprint 99 (Apr 2023), Sprint 100 (May 2023), Sprint 101 (Jun 2023), Sprint 102 (Jul 2023), Sprint 103 (Aug 2023), Sprint 104 (Sep 2023), Sprint 105 (Oct 2023), Sprint 106 (Nov 2023) | ||||||||||||||||||||
Story Points: | 0.5 |
Description |
Hi data overview widget head text no in vertical aligin |
Comments |
Comment by Aigars Kadikis [ 2021 Jun 21 ] |
I can confirm the behaviour is as described: Host titles are now represented horizontally. Class of HTML 'th' element still comes with name 'vertical_rotation'. |
Comment by Janis Freibergs [ 2021 Aug 25 ] |
Fixed in development branches: |
Comment by Ivan [ 2021 Nov 26 ] |
In all versions 5.4.x the problem continues, including 5.4.7 |
Comment by Mark Oudsen [ 2022 Jan 24 ] |
I can confirm the problem continues - running 5.4.9 but still have horizontal instead of vertical headers leading to all kinds of serious dashboard overviews. |
Comment by Mark Oudsen [ 2022 Jan 24 ] |
Reported again in |
Comment by Janis Freibergs [ 2022 Jan 24 ] |
Hi, Mark, the status Resolved means a fix has been found, but the changes await approval from colleagues to ensure it is correct (which it wasn't fully here in the first approach, though technically working); the fix has not yet been added to the mainstream release. You can try out switching to the development branch from my comment above (code based on 5.4.10rc1) to see if that works for you, or await when status changes to _Done _- that means the code was approved and merged with the official branch. If hesitant to upgrade, a temporary workaround (albeit it does not look 100% as intended), could be adding the style: .list-table thead .vertical_rotation { min-width: 1.4em; writing-mode: vertical-lr; transform: rotate(180deg); } to your selected theme(s), but be highly advised to reset it when upgrading/changing versions in the future. |
Comment by Mark Oudsen [ 2022 Jan 25 ] |
Appreciated! Will use the workaround for now and awaiting 5.4.10 to resolve this permanently. Thx! |
Comment by Mark Oudsen [ 2022 Jan 30 ] |
I've tested this extensively and was not able to use the workaround unless I dictated a fixed row height for the labels to be displayed in. There is some CSS styling missing to have a dynamic height row (where one should considering to allow wrapping at a certain maximum height to avoid huge sized headers). .list-table thead .vertical_rotation { min-width: 1.4em; line-height: normal; writing-mode: vertical-lr; transform: rotate(180deg); height: 280px; } While fixing the height did the job for me, still awaiting a more permanent (and dynamic) styling? (PS: Is this the reason the manual test at your end failed as well?) |
Comment by Mark Oudsen [ 2022 Jan 30 ] |
Just noticed the fix is pushed forward to 5.4.11 (rc1). If you want to test any styling adjustments in the mean time please share any new adjustements here - very happy to try this in my environment. While we are at it: the bold line under the column header is also interrupted ... |
Comment by Mark Oudsen [ 2022 Feb 18 ] |
I've picked up the blue-theme.css from source 5.4.11rc11 (at moment of the timestamp of this log entry) and have applied the theme to my server. After analysis I've observed that behavior is still incorrect. Did some root cause analysis and found that "vertical_rotation" is used in the header, but it not defined in the CSS anywhere.
# grep rotation * blue-theme.css: .list-table .vertical_rotation_inner { dark-theme.css: .list-table .vertical_rotation_inner { hc-dark.css: .list-table .vertical_rotation_inner { hc-light.css: .list-table .vertical_rotation_inner { When observing the generated HTML code: When applying "vertical_rotation_inner" to the header item, rotation works, but it also rotates the bold line between the header and the items and does not respect header space as I've observed per my earlier comments. Feels like either:
Fixing manually (again) for now at my end, but please consider re-checking/testing your updates in 5.4.11rc1 (not sure about v6.x, but guessing it has the same issue or perhaps v6.x generates the "vertical_rotation_inner" class as well? I do not have a lab setup for v6.x to test). |
Comment by Mark Oudsen [ 2022 Feb 18 ] |
Checked v6.1 code in Github ==> v6.1 auses "vertical_rotation_inner", but cannot check if Zabbix v6.1 generates "vertical_rotation" or "vertical_rotation_inner". |
Comment by Mark Oudsen [ 2022 Feb 18 ] |
Customer now starting to complain as the overviews become unworkable when having moderate to long names in the header. Any way this item can be bumped to a higher prio/faster fix? |
Comment by Mark Oudsen [ 2022 Mar 16 ] |
Same issue observed in 6.0.2 LTS.
Waiting for official 6.0.3 release (high confidence based on observed code changes). |
Comment by Mark Oudsen [ 2022 Nov 04 ] |
Please bump the priority a bit. We're truly struggling to get the dataviews in a readable format. Also how can we keep the dataview as this is widely used in our deployments? |
Comment by Alexander Shubin [ 2023 Nov 02 ] |
Fixed in:
|
Comment by Mark Oudsen [ 2023 Dec 04 ] |
Confirmed working in our environments for 6.0 and 6.4. You may not be able to imagine how happy we are having the right views again Regards, Mark |