Change and document popover position for charts
Description
https://design.gitlab.com/components/popover#appearance states that popover elements should appear on top of hovered/focused elements, however this may not be correct for charts, I opened this issue to decide on an approach.
https://gitlab-org.gitlab.io/gitlab-ui/?path=/story/charts-column-chart--default
placement top (default) | placement right (pre- !2742 (merged)) |
---|---|
![]() |
![]() |
Original Discussion
The following discussion from !2742 (merged) should be addressed:
-
@pgascouvaillancourt started a discussion: (+1 comment) @mrincon this change affected chart tooltips as well because they use
GlPopover
internally. Do you think that we should:- Let chart tooltips use
GlPopover
's default position? - Or override their positioning so that they still use the
right
positioning?
- Let chart tooltips use
Edited by Miguel Rincon