KDE/kdebase/workspace/plasma/applets/systemtray

[Jason Stubbs]

On Monday 09 June 2008 10:15:21 JST, Sebastian Sauer wrote:
> SVN commit 818620 by sebsauer:
>
> fix GTK icons are painted bigger than KDE icons in the systemtray.
>
> ---
> trunk/KDE/kdebase/workspace/plasma/applets/systemtray/systemtraycontainer.c
>@@ -42,9 +42,7 @@ 
>      updateBackground();
>
>      // Tray icons have a fixed size of 22x22
> -    setMinimumSize(22, 22);
>      setMaximumSize(22, 22);
> -    resize(22, 22);
>  }
>
>  void SystemTrayContainer::embedSystemTrayClient( WId clientId )
> ---
> trunk/KDE/kdebase/workspace/plasma/applets/systemtray/systemtraywidget.cpp
> @@ -125,7 +125,9 @@ 
>  void SystemTrayWidget::addWidgetToLayout(QWidget *widget)
>  {
>      // Add the widget to the layout
> -    m_mainLayout->addWidget(widget, m_nextRow, m_nextColumn);
> +    m_mainLayout->setRowMinimumHeight(m_nextRow, 22);
> +    m_mainLayout->setColumnMinimumWidth(m_nextColumn, 22);
> +    m_mainLayout->addWidget(widget, m_nextRow, m_nextColumn, 1, 1,
> Qt::AlignCenter);
>
>      // Figure out where the next widget should go
>      if (m_orientation == Qt::Horizontal) {

Just out of interest, don't these additions and removals lead to exactly the 
same behaviour? Also, the centering indicates that the widget can have a size 
less than 22x22; is that the case?

-- 
Jason Stubbs