OpenBazaar-Client icon indicating copy to clipboard operation
OpenBazaar-Client copied to clipboard

Hovering an avatar in the chat panel is no longer displaying the tooltip with their guid/handle in it.

Open morebrownies opened this issue 9 years ago • 3 comments

morebrownies avatar Feb 29 '16 14:02 morebrownies

I removed the tooltips, because it caused the layout to break.

jjeffryes avatar Feb 29 '16 18:02 jjeffryes

Ok. Let's keep this open. We don't need to figure it out right now, but the tooltips are helpful.

morebrownies avatar Feb 29 '16 18:02 morebrownies

Here' what's happening: in order for the chat heads to be scrollable an overflow-y is being set on .chatConversationHeads. This essentially prevents overflow-x from being visible (even though you can set it to be so, it is ignored by the browser). It would need to be visible otherwise it prevents the tooltip from being visible.

So, with the current implementation of the tooltip, you basically have to choose. Do you want chat heads to be scrollable or for tooltips to be available? I personally think the scrollable chat heads are more important because if someone has more convos than will show on the screen, they'll never be able to get to the bottom ones. And, since as of now, we can't remove those darn things, that person would be screwed without a scroll bar.

If we want to support both, we would need a JS tooltip implementation where the tooltip element is placed outside of the .chatConversationHeads container. Obviously, the current CSS implementation is much more simpler to manage, but this is a case where it doesn't work for us.

rmisio avatar Feb 29 '16 18:02 rmisio