Properly fix task tray background glitches

Ensure session is fully locked before switching to new VT when lock is requested

2 files changed, 6 insertions, 20 deletions

diff --git a/kicker/applets/systemtray/systemtrayapplet.cpp b/kicker/applets/systemtray/systemtrayapplet.cpp
index 542ac42d0..b91dafd27 100644
--- a/kicker/applets/systemtray/systemtrayapplet.cpp
+++ b/kicker/applets/systemtray/systemtrayapplet.cpp
@@ -1130,10 +1130,8 @@ void SystemTrayApplet::setBackground()
 
 
 TrayEmbed::TrayEmbed( bool kdeTray, TQWidget* parent )
-    : QXEmbed( parent ), kde_tray( kdeTray ), m_ensureBackgroundSetTimerCount(0)
+    : QXEmbed( parent ), kde_tray( kdeTray )
 {
-    m_ensureBackgroundSetTimer = new TQTimer();
-    connect(m_ensureBackgroundSetTimer, TQT_SIGNAL(timeout()), this, TQT_SLOT(ensureBackgroundSet()));
     hide();
     m_scaledWidget = new TQWidget(parent);
     m_scaledWidget->hide();
@@ -1141,7 +1139,7 @@ TrayEmbed::TrayEmbed( bool kdeTray, TQWidget* parent )
 
 TrayEmbed::~TrayEmbed()
 {
-    delete m_ensureBackgroundSetTimer;
+    //
 }
 
 void TrayEmbed::getIconSize(int defaultIconSize)
@@ -1164,20 +1162,19 @@ void TrayEmbed::setBackground()
 {
     const TQPixmap *pbg = parentWidget()->backgroundPixmap();
 
-    if (pbg)
-    {
+    if (pbg) {
         TQPixmap bg(width(), height());
         bg.fill(parentWidget(), pos());
         setPaletteBackgroundPixmap(bg);
         setBackgroundOrigin(WidgetOrigin);
     }
-    else
+    else {
         unsetPalette();
+    }
 
     if (!isHidden())
     {
         XClearArea(x11Display(), embeddedWinId(), 0, 0, 0, 0, True);
-        m_ensureBackgroundSetTimerCount = 0;
         ensureBackgroundSet();
     }
 }
@@ -1211,8 +1208,7 @@ void TrayEmbed::ensureBackgroundSet()
 				int g = int( tqGreen( l ) );
 				int b = int( tqBlue( l ) );
 				int a = int( tqAlpha( l ) );
-				ls[x] = tqRgba( r, g, b, a );
-				XSetForeground(x11Display(), gc, (r << 16) | (g << 8) | b );
+				XSetForeground(x11Display(), gc, (a << 24) | (r << 16) | (g << 8) | b );
 				XDrawPoint(x11Display(), argbpixmap, gc, x, y);
 			}
 		}
@@ -1223,13 +1219,5 @@ void TrayEmbed::ensureBackgroundSet()
 
 		// Repaint
 		XClearArea(x11Display(), embeddedWinId(), 0, 0, 0, 0, True);
-
-		// HACK
-		// Clear background artifacts in first available timeslot after initial icon display
-		if (m_ensureBackgroundSetTimerCount < 1) {
-			m_ensureBackgroundSetTimerCount++;
-			m_ensureBackgroundSetTimer->stop();
-			m_ensureBackgroundSetTimer->start(0, TRUE);
-		}
 	}
 }
diff --git a/kicker/applets/systemtray/systemtrayapplet.h b/kicker/applets/systemtray/systemtrayapplet.h
index 06b42c23d..250480b65 100644
--- a/kicker/applets/systemtray/systemtrayapplet.h
+++ b/kicker/applets/systemtray/systemtrayapplet.h
@@ -135,8 +135,6 @@ public:
 private:
     bool kde_tray;
     TQWidget *m_scaledWidget;
-    TQTimer* m_ensureBackgroundSetTimer;
-    int m_ensureBackgroundSetTimerCount;
 private slots:
     void ensureBackgroundSet();
 };