New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improves realtime widget #12884
Improves realtime widget #12884
Conversation
Feedback:
|
I decided to add the visitorid in order to have a bit more consistency. We already displayed the visitor profile icon and the user id before. But to be honest, I never recognized that it has been displayed. The small icon behind the time was quite invisible. Imho it's ugly from UI/UX perspective if the icon/link can be found on different places (behind the date or on new line depeding on the available space). Now it's always at the same position and easy to find and click.
The icons displayed here are generated with the same code as in visitor log and profile. I really would like to keep the consistency between all widgets/reports, so it's clear to the user which information can be found within the icon list. Imho that improves the UX a lot. Even if we already added various special handling for log/profile/widget so specific icons are not displayed in some parts, I would avoid changing more stuff here. |
👍 to show visitor id if no userid, would confuse me otherwise to see it one place and not the other. Though maybe it would be clearer to have '(Unknown user)' or something instead? Just some thoughts. |
Agreed with this. That said, we can't/don't want to display the visitor ID as this is meaningless and not human readable. There is no value in showing it in the widget. So maybe just show the User ID after the list of icons, when it's defined? And show the Visitor Profile icon in the list of icons? |
Alternatively we could show the text |
Can you try moving the icon either at the end of other icons, or on the date-time row? I think it will be fine... (it's not critical to have consistency between the widgets but rather we want to give the best possible design / data presentation for this widget) I wouldn't show the text "open visitor profile" because it's too wordy for the real time widget which is space constrained. |
It may still be useful to show the visitorId on hover for debugging purposes. Eg someone is visiting your website, and then you want to segment for that visitor, or use the GDPR tools to segment for that visitor etc. But not super important needed for sure. |
On hover sounds good 👍 |
Ok. So what exactly shall we do now? If we move the icon for the visitor profile somewhere else, the user id will not be displayed next to it anymore as there is not enough space to have it always in the same line. So the userid will be displayed in an extra line (if defined), but not linked with the visitor profile. And if so, where shall we display the visitorid on hover? The icon already has a tooltip saying "open visitor profile". And using a tooltip for the visitorid imho isn't a good solution as tooltip contents can't be copied. |
I'm not sure how it will look, but thinking to display the Visitor profile icon after the other icons (or on the date-time line...). For User ID, maybe display it after date-time, or on a new line would be fine (since it is an important info and will only be set quite rarely). Ideally the User ID also links to open the visitor profile (if possible.) Visitor ID on hover: in the same tooltip as the Visitor IP address |
@mattab I've done the changes |
@sgiehl the positioning of the icon looks odd when I tried it locally: Tried quickly fixing it, but couldn't figure out an easy way to do it (changing the size is easy, but couldn't find a way to move it down). |
And the list of icons should be on a separate line, right? @sgiehl |
sure. should be fixed now |
Looks really good 👍 |
* Improves realtime widget * readds provider information to flag tooltip * do not show visitor id & rearange visitor profile icon * show visitorid in tooltip * fix icon align & size * Tweak to have consistent padding
Note: the tooltip changes will also have effect in visitor log & profile
Changes will look like this:
fixes #12876
fixes #12887