|Re: Grid - onMouseExit hover question [message #998175 is a reply to message #997912]
||Wed, 09 January 2013 13:59
| Thomas Schindl
Registered: July 2009
Your conclusion sounds correct. File a bug and attach a patch. If you'd
like to share your Win7-Rendering I'm more than happy to add them to the
Just create a bug for them as well and attach the sources. I guess we'll
have to go through the IP-Process for them.
Am 09.01.13 14:25, schrieb Sean Ruff:
> I had a quick question about the behavior of the hovering logic of the
> grid component.
> So I've implemented my own GridHeaderRenderer which provides a more Win7
> native look and feel. It looks great and works great (almost). My
> renderer supports 3 different states
> (mouseover/hover,mousedown/selected,normal). Most of the logic I need
> is already supported out of the box by extending
> DefaultColumnHeaderRenderer. The one issue I'm having is detecting then
> the column header is hovered and the mouse exits the Grid directly from
> the column header (by moving up).
> I believe I've tracked the issue down to one particular piece of logic
> which I wanted to question here before I open a bug request to make sure
> my understanding is correct. In the onHoverCell the Grid initializes
> the hoveringColumnHeader to a valid column (along with other fields).
> While in the onMouseExist those same hover fields are reset, with the
> exception of hoveringColumnHeader. Then, when the following logic in
> the paintHeader method is executed:
> == column);
> and the setHover for the renderer is set to true (in my use case).
> So my question is
> Shouldn't hoveringColumnHeader be set to null in the onMouseExit method?
> Again, I can open a bug report if changes are needed and even include a
> patch, if needed.
> Additionally I can share the Header Renderer that I have implemented if
> interested (IIRC there was a request for more "Win7 native" headers here
> on the forums), while obviously not native it does give the a closer
> "win7" look.
> Thanks for the nebula widgets!
Powered by FUDForum
. Page generated in 0.01894 seconds