Opened 10 years ago

Last modified 3 years ago

#7730 new enhancement

Please add tooltips for status modifier icons

Reported by: rrt Owned by: deryni
Milestone: Component: pidgin (gtk)
Version: 2.5.2 Keywords:
Cc: latinsud

Description

Some of the status modifier icons (such as blocked, logged on to MSN via webmessenger &c.) are less than obvious. It would be good if all such icons had tooltips; there's plenty of space to point elsewhere to get the regular tooltip for showing details of the current buddy.

Attachments (1)

pidgin258-tooltip-msn-clientid.patch (1001 bytes) - added by latinsud 10 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 10 years ago by rekkanoryo

  • Type changed from defect to enhancement

comment:2 Changed 10 years ago by lschiere

  • Owner lschiere deleted

Changed 10 years ago by latinsud

comment:3 Changed 10 years ago by bernmeister

Move to Patches Needing Review?

comment:4 Changed 10 years ago by darkrain42

  • Milestone set to Patches Needing Review
  • Type changed from enhancement to patch

comment:5 follow-up: Changed 10 years ago by deryni

This patch has little to do with the original request.

comment:6 in reply to: ↑ 5 Changed 10 years ago by latinsud

Replying to deryni:

This patch has little to do with the original request.

It has, but instead of creating a new tooltip for the icon, it adds the tip to the existing generic tooltip. That is, not being perfect, but not altering UI logic either.

If you think there should be a standalone tooltip, i can try to code it (i'm no expert). Else imho this is the only and required solution.

comment:7 Changed 10 years ago by rrt

I don't think this would remove my confusion, as why should I associate this bit of text with the icon at the side? The principle for tooltips is simple: one UI element, one tooltip.

Some icons (e.g. "musical note") don't seem to be covered by this patch.

Finally, inline in the code is not the place to put a mapping from icon types to textual explanations for them.

comment:8 follow-up: Changed 10 years ago by deryni

The patch has merit but does not really mitigate the original report, not without also including the emblem on the info line.

This patch is also MSN specific and so does nothing for emblems appearing for buddies on any other protocol, whereas an emblem tooltip would.

Given our current tooltip code I'm not sure that it is possible to have emblem specific tooltips (and our tooltip code is complicated enough that I can't quickly tell by looking at it).

I am in no way indicating that I think this patch isn't a good idea (I do), just that it doesn't solve the original problem.

comment:9 in reply to: ↑ 8 Changed 10 years ago by latinsud

Replying to deryni:

The patch has merit but does not really mitigate the original report, not without also including the emblem on the info line.

According to that, what could be done, apart from the patch is:

  • Do a similar patch for all protocols that use emblems.
  • Add emblem icon to tooltip, just like status, protocol, and buddy icon are already shown.

The only problem for user is that, inside tooltip, emblem icon won't appear exactly beside the emblem description, but this currently happens for status icon vs status text. Images inside tooltip are currently in quite fixed locations.

comment:10 Changed 10 years ago by deryni

This shouldn't need to be done for each protocol, that information is already marked on the buddy (that's how the emblem is shown in the first place). So, it would make sense for pidgin to show that information in the tooltip already (possibly with the emblem icon next to it, or at least next to the protocol icon).

I still think if it is possible the icon in the buddy list entry itself should have a descriptive tooltip, but with the information in the general tooltip that will certainly matter less.

comment:11 Changed 10 years ago by deryni

  • Owner set to deryni

comment:12 Changed 3 years ago by dx

  • Component changed from unclassified to pidgin (gtk)
  • Milestone Patches Needing Review deleted
  • Type changed from patch to enhancement

Removing patch tags because that patch is obsolete since it's msn specific and even if msn is updated it won't have that feature. I'm tempted to close this ticket since the main motivation seemd to be msn too, but the comments say otherwise.

Note: See TracTickets for help on using tickets.
All information, including names and email addresses, entered onto this website or sent to mailing lists affiliated with this website will be public. Do not post confidential information, especially passwords!