[Semi-PATCH] Dolphin - Previews in Tooltips

[Simon St James]

Hi Peter,

Firstly, if you are currently on vacation: stop reading now, and get some 
rest ;)

Secondly, I've noticed a slight annoyance with the tooltip placement: if I 
want to hover over an item to get some Nepomuk metadata information (which is 
not shown in a tooltip), the tooltip often obscures the info panel, which I 
place on the right of the Dolphin window:

http://img171.imageshack.us/my.php?image=dolphintooltipobscurednq1.png

I was wondering what you thought of the strategy in the attached patch? 
Basically, it attempts to move the x-coordinate of the tooltip so that it is 
close to the mouse pointer rather than the edge of the highlighted item (but 
still, of course, in a way that does not overlap the item).  

I find this works much better for me as a) I have more control over where the 
tooltip appears and so can stop it from overlapping the metadata panel; b) it 
mirrors the KDE3 behaviour a little better (creature of habit, me :)) and c) 
the tooltip appears where the user is currently looking (I personally am 
mildly surprised each time I have to move my eyes to the right to see the 
newly-spawned tooltip).

Also: what do you think about hiding the tooltip upon initiating a drag? 
Currently the tooltip still hangs around, which is a bit awkward looking.

Best Wishes,
Simon

PS 

My hosting is currently down so I won't able to receive e-mails (personal or 
mailing lists) for a while :/

On Tuesday 19 August 2008 20:43:10 you wrote:
> Hi Simon,
>
> Am Tuesday, 19. August 2008 19:37:27 schrieb Simon St James:
> [...]
>
> > > > Issue https://bugs.kde.org/show_bug.cgi?id=167205 has a detailed
> > > > description + history. If I remember correctly, it should be no big
> > > > deal merging svn commit 824927 with 838041 (tooltipmanager.cpp). I
> > > > can take over this task for you if you want to concentrate on the
> > > > previews
> > > >
> > > > :-)
> >
> > I'm not sure if I have a full appreciation of the issues: will just
> > keeping within the desktop bounds (now that desktop =
> > QApplication::desktop()->screenGeometry(...)) solve the dual screen
> > issues implicitly?
>
> Yes :-)
>
> > Anyway, here's my attempt - let me know if I'm missing
> > anything :)
>
> Thanks a lot, this seems to work perfectly. I've just committed your patch
> (I've done only some minor coding style changes).
>
> Best regards,
> Peter
>
> PS: I just tried your preview-tooltips and really like them :-)
>
> > Best Wishes,
> > Simon


-------------- next part --------------
A non-text attachment was scrubbed...
Name: tooltip-place-near-pointer.patch
Type: text/x-diff
Size: 2345 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20080920/7437b57c/attachment.patch>