Display tag and performer image on hover. on the scene edit page#5739
Conversation
|
I reverted the change to show the hover popover when mousing over options in the drop down. I personally find the hover popover behaviour for tags highly irritating. I tried making the popover only occur when mousing over the performer image, but it still had issues overflowing the window. To me it seems unnecessary: most relevant information is available in the option already, and you can navigate to the performer page by clicking on the performer image (which you can't currently do for tags). In any case, I don't want that to prevent merging in the rest of it, so feel free to submit that part as a separate PR for discussion if you wish. It's not ideal that the popover components make a graphql request for the tag/performer object, when we have the objects available in the select control. I tried changing the popover components to accept an object, but it appears at least for tags, the returned data object is missing information, which caused a UI crash when hovering. I've left it as is for now. |
…shapp#5739) * add component for PerformerPopover * show PerformerPopover for performer select values * show TagPopover for tag select values
closes #4320