Re: [E-devel] forecast's popup, to popup on a click (configurable)...

[Ravenlock <rav...@ra...>]

On 03/22/2007 13:18, Виктор Кожухаров wrote:
> В чт, 2007-03-22 в 09:26 -0500, Ravenlock написа:
>> On 03/21/2007 01:05, Виктор Кожухаров wrote:
>>> В вт, 2007-03-20 в 22:07 -0500, Ravenlock написа:
>>>> Hello,
>>>>
>>>> I, just this evening, started using the forecast module.  I like it very 
>>>> much.
>>>>
>>>> However, I find that the popup... popping up on a hover is terribly 
>>>> annoying.  Its annoying for me because I like to keep it in the upper 
>>>> right corner of my screen.  Being there, every time I minimize a 
>>>> window... I am then hovering over it, and it pops up.  :(
>>>>
>>>> So I wanted to have an option to have the popup window popup on either a 
>>>> mouse in or a mouse down.
>>>>
>>>> The attached patch alters the config panel, and provides a way to 
>>>> disable the "popup on hover".  If "popup on hover" is unchecked, the 
>>>> popup will appear on a mouse down, and stay pinned.  A second mouse down 
>>>> will hide it.
>>>>
>>>> PLEASE NOTE:
>>>> This patch also changes the default popup behavior to popup on the mouse 
>>>> down.  This is by design.  My argument for doing it this way is...  The 
>>>> original was not a popup on *hover*... but popup immediately on a mouse 
>>>> in.  This meant that if you just passed over it carelessly... it popped 
>>>> up.  That bothered me too. A timer, to create a short "hover" delay 
>>>> would be necessary to make the hover work properly, imho.
>>>>
>>>> I hope someone finds this useful.  :)
>>> Thanks. I'm going to take a look at it. However, the default behavior is
>>> going to stay, so I'll have to change the logic of the patch to turn off
>>> the popup at mouse in, instead of turning it on. And an immediate popup
>>> at mouse in is important, since people tend to want information
>>> immediately, instead of having to wait for a timer to time out :)
>>>
>> Having never worked on a 'module' before.  I realize after a second 
>> glance that I forgot some init code.  Would you like me to provide an 
>> updated patch?  Its probably the same code you would wind up adding to 
>> make sure the module exhibits the original default behavior.
>>
>> Thanks.
>>
> Sure. Patches are always welcome :)

Here is version 2.  It simply has a few extra lines of init code.  I 
believe it will maintain the original behavior now.

-Ravenlock

>>> p.s. And practice your mouse control, it sounded like you triggered the
>>> popup one too many times :)
>>>
>>>
>>> ------------------------------------------------------------------------
>>>
>>> -------------------------------------------------------------------------
>>> Take Surveys. Earn Cash. Influence the Future of IT
>>> Join SourceForge.net's Techsay panel and you'll get the chance to share your
>>> opinions on IT & business topics through brief surveys-and earn cash
>>> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
>>>
>>>
>>> ------------------------------------------------------------------------
>>>
>>> _______________________________________________
>>> enlightenment-devel mailing list
>>> enl...@li...
>>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>>


-- 
Regards,
Ravenlock