<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'Bitstream Vera Sans'; font-size:10pt; font-weight:400; font-style:normal;">On Tuesday 25 November 2008 02:41:38 Parker Coates wrote:<br>
> Hello folks and Dmitry in particular,<br>
> Is it not protocol to leave the default behaviour intact when making a<br>
> change like this? I know an email was sent to the list, but I had read<br>
> it and dismissed it, as I thought it didn't apply to me since Killbots<br>
> used the click-to-close behaviour. I also realise the change was made<br>
> last April and I just noticed the problem today, so obviously it<br>
> didn't have that big an effect.<br>
Hmm, not sure what you're talking about :)<br>
You mean someone has changed default behavior and haven't post a note about it in the list (i wonder if that someone would be me, though I don't recall me doing this change :))<br>
Well, if you're talking about that kind of thing, i think it should have been noted, yes.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>> I'm not sure how to fix this. The text item has its own mouse event<br>
> handlers to properly handle the link functionality, so any attempt to<br>
> grab all mouse release events would break the link handling, and as<br>
> far as I know, there's no way to grab only the clicks on regular text<br>
> the the text item ignores. Does anyone have any idea of how to do this<br>
> correctly.<br>
I have an idea that some bool (m_linkHovered) can be added to this class and that it can be set to true once linkHovered signal arrives, and false otherwise. Then, you can override mouseReleasEvent and if m_linkHovered is true, call the parent handler, otherwise do usual hiding stuff.<br>
If that'd work out, we have a solution.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>This is purely from head, I didn't look at the code yet.<br>
(time to go to bed now...)<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>Cheers,<br>
Dmitry.</p></body></html>