| Summary: | chat window is raised across different desktops when message queue is off | ||
|---|---|---|---|
| Product: | [Unmaintained] kopete | Reporter: | Mihai RUSU <dizzy> |
| Component: | general | Assignee: | Kopete Developers <kopete-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version First Reported In: | 0.50.80 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: | Don't raise chat window | ||
|
Description
Mihai RUSU
2008-08-12 15:27:39 UTC
Created attachment 26814 [details]
Don't raise chat window
I didn't commit it because I'm not 100% sure about the change in
KopeteViewManager::slotEventDeleted. The KopeteView::raise is already called
when notify activated signal is emitted so I think that it should be ok if we
remove the activation from there.
Can anybody check if I'm right?
SVN commit 846257 by rjarosz: Fix bug 168978 chat window is raised across different desktops when message queue is off BUG: 168978 M +5 -2 kopeteviewmanager.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=846257 Comment on attachment 26814 [details]
Don't raise chat window
Mihai if you want to test it please use the patch from "WebSVN link" because
the committed patch is different then the one that is attached here.
Comment on attachment 26814 [details]
Don't raise chat window
Mihai if you want to test it please use the patch from "WebSVN link" because
the committed patch is different then the one that is attached here.
Thanks for the warning, I was going to test it right now and was going to use the attached patch but I got your message in time. I think I will just wait for kde anonymous trunk to update (I think there was a delay there) and just compile trunk kopete. Will let you know how it works. Seems to work fine (tested kopete trunk), thanks for your work. If there is/will be a KDE 4.1.x branch for bug fixes maybe you would consider porting it there too, thanks! I will backport it to KDE 4.1, but want to wait a little bit and see if it will not cause any regressions. |