-
Notifications
You must be signed in to change notification settings - Fork 59
Fixed viz tooltip position. #48
Fixed viz tooltip position. #48
Conversation
The "fixes" header needs to have the colon: "Fixes: #7778" (otherwise the link from redmine -> repo doesn't happen, and that's quite handy). Please fix that. Also, please add a bit of explanatory text to the bug about what you found, why it was wrong, and why your fix fixes the issue, as it's not very obvious from the code change (even factoring in my inexperience with Javascript). Otherwise, nice job. |
Comment Updated. |
And the bug?
|
bug detail added. |
I'm sorry to be a pest about this, karnan, but I don't see any addition to http://tracker.ceph.com/issues/7778 at all. ??? |
It looks like the absolute positioning doesn't work on smaller windows see screen shot in #7778 |
269557e
to
31646cf
Compare
@GregMeno it is because of bug in rapheal js. workaround made by adding a wrapper html div. now absolute positioning works relatively to the new div. |
07ce7b7
to
3d39909
Compare
Fixes the position of tooltip that appears on top of the hostgroups in workbench (sort by host). Fixes #7778 Signed-off-by: Karnan TC <kchidamb@redhat.com>
The connectivity of hostgroup background is not proper. Now, the position of last row osds are right aligned if it is an even row. Fixes: #9521 Signed-off-by: Karnan TC <kchidamb@redhat.com>
3d39909
to
7a3ac92
Compare
updates Mon to Monitors, Updates PG to Placement Group. Fixes: #9329 Signed-off-by: Karnan TC <kchidamb@redhat.com>
Fixed viz tooltip position.
Fixes the position of tooltip that appears on top of the
hostgroups in workbench (sort by host).
Initially the tooltip always appears in the center of the screen and not
near the hostgroup, which is a ui bug.
Now the tooltip is placed exactly on the top of the host group using js absolute positioning.
Fixes: #7778
Signed-off-by: Karnan TC kchidamb@redhat.com