Bug 439414 - extremely large tooltips
Summary: extremely large tooltips
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: PackageKit
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Robin Norwood
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-03-28 16:13 UTC by Zack Cerza
Modified: 2008-04-07 09:23 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-04-07 09:23:40 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
what about something like this? (888 bytes, patch)
2008-03-28 19:29 UTC, Richard Hughes
no flags Details | Diff

Description Zack Cerza 2008-03-28 16:13:52 UTC
Description of problem:
If PK has enough pending transactions, gnome-pk will happily create a tooltip
that doesn't fit onscreen. 

For example, I cherry-picked maybe 30 updates from 200 or so by clicking "Update
Package" on each of them. They're added to the queue (along with tons of extra
"get update detail" and "get updates" items). Mousing over the gnome-pk icon
shows a tooltip that's taller than my screen itself.

Version-Release number of selected component (if applicable):
gnome-packagekit-0.1.9-4.fc9.i386

Comment 1 Richard Hughes 2008-03-28 18:06:55 UTC
Ahh, we've fixed the cherry pick problem by making them checkboxes you can
select. How many lines do you think a "too big" tooltip is (so we can add ...
for the rest).

Comment 2 Zack Cerza 2008-03-28 18:54:29 UTC
If we can't figure out how tall the tooltip is going the be (so we can compare
it to the screen height) I'm thinking the cap should be something like, I don't
know, 25? It's just a guess.

Also, instead of "..." it would be a good idea to put "(56 more transactions)"
or some such text.

Comment 3 Richard Hughes 2008-03-28 19:29:22 UTC
Created attachment 299515 [details]
what about something like this?

Comment 4 Zack Cerza 2008-03-28 19:57:05 UTC
Didn't actually compile/test it, but it looks like the right fix.

Comment 5 Richard Hughes 2008-03-28 20:19:14 UTC
I've added this into git, many thanks.


Note You need to log in before you can comment on or make changes to this bug.