Opened 8 years ago

Closed 7 years ago

#7726 closed bug (fixed)

Time Preference preflet show time zone tooltip outside of list

Reported by: Ziusudra Owned by: mmadia
Priority: normal Milestone: R1
Component: Preferences/Time & Date Version: R1/alpha4.1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

See attached screenshot.

Attachments (2)

TimePrefletTooltip.png (18.5 KB ) - added by Ziusudra 8 years ago.
0001-Create-TimeZoneListView-class-and-move-GetToolTipAt-.patch (7.4 KB ) - added by Ziusudra 7 years ago.
This patch creates a TimeZoneListView and moves the tooltip code from ZoneView into it. Rebased to hrev44901

Download all attachments as: .zip

Change History (8)

by Ziusudra, 8 years ago

Attachment: TimePrefletTooltip.png added

comment:1 by Ziusudra, 7 years ago

Has a Patch: set

by Ziusudra, 7 years ago

This patch creates a TimeZoneListView and moves the tooltip code from ZoneView into it. Rebased to hrev44901

comment:2 by diver, 7 years ago

Version: R1/alpha3R1/alpha4.1

comment:3 by mmadia, 7 years ago

Can anyone comment if this patch is OK to commit? It applies cleanly to master (currently at hrev45326). WIth the patch applied, the tooltip will appear only when hovering over a specific timezone. The contents of the tooltip will change, depending on which timezone the mouse is hovering over.

comment:4 by leavengood, 7 years ago

It looks great to me. A solid and well implemented solution to this bug! I say commit it.

comment:5 by mmadia, 7 years ago

Owner: changed from axeld to mmadia
Status: newassigned

comment:6 by mmadia, 7 years ago

Resolution: fixed
Status: assignedclosed

Applied in hrev45330. Thanks for the contribution, Ziusudra!

Note: See TracTickets for help on using tickets.