<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>80445</bug_id>
          
          <creation_ts>2004-04-27 01:00:14 +0000</creation_ts>
          <short_desc>More customability to KWeather</short_desc>
          <delta_ts>2008-01-03 21:06:12 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>10</classification_id>
          <classification>Unmaintained</classification>
          <product>kweather-kde3</product>
          <component>general</component>
          <version>unspecified</version>
          <rep_platform>RedHat Enterprise Linux</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>wishlist</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Thomas Wolfe">blackphiber</reporter>
          <assigned_to>geiseri</assigned_to>
          <cc>dj</cc>
    
    <cc>pascal</cc>
          
          <cf_commitlink></cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>128</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>226854</commentid>
    <comment_count>0</comment_count>
    <who name="Thomas Wolfe">blackphiber</who>
    <bug_when>2004-04-27 01:00:14 +0000</bug_when>
    <thetext>Version:            (using KDE KDE 3.2.2)
Installed from:    RedHat RPMs

This is similar to another wish I reported:
http://bugs.kde.org/show_bug.cgi?id=80444
but this time I would like to be able to change the size of the text in KWeather and the color of the text, I know with the clock it is very easy to do this.  Right now for me its way to big.  Also I know many people would probably like to be able to choose between Celsius and fahrenheit.  Thanks again for listening.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>226891</commentid>
    <comment_count>1</comment_count>
    <who name="Christian Loose">christian.loose</who>
    <bug_when>2004-04-27 09:46:35 +0000</bug_when>
    <thetext>Changed product to kweather.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>227160</commentid>
    <comment_count>2</comment_count>
    <who name="Thomas Wolfe">blackphiber</who>
    <bug_when>2004-04-28 02:49:25 +0000</bug_when>
    <thetext>Thanks, I did not see that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280501</commentid>
    <comment_count>3</comment_count>
    <who name="">p92</who>
    <bug_when>2004-10-27 17:14:28 +0000</bug_when>
    <thetext>I also vote for this enhancement wish especially font color of the applet.

When kicker is transparent, a black font is really difficult to read !

Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>357654</commentid>
    <comment_count>4</comment_count>
    <who name="">p92</who>
    <bug_when>2005-07-13 21:13:55 +0000</bug_when>
    <thetext>still needed my wish in comment #3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>368118</commentid>
    <comment_count>5</comment_count>
    <who name="Bram Schoenmakers">me</who>
    <bug_when>2005-08-23 15:36:52 +0000</bug_when>
    <thetext>*** Bug 99526 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>368168</commentid>
    <comment_count>6</comment_count>
    <who name="Stefan Winter">swinter</who>
    <bug_when>2005-08-23 18:27:36 +0000</bug_when>
    <thetext>*** This bug has been confirmed by popular vote. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407519</commentid>
    <comment_count>7</comment_count>
    <who name="Reigo Reinmets">xatax</who>
    <bug_when>2006-01-19 20:25:46 +0000</bug_when>
    <thetext>This is definetly an issue that must be addressed! Im using a transparent taskbar also and my clock is white.. But kweather is black and mostly unreadble...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>451906</commentid>
    <comment_count>8</comment_count>
    <who name="Pupeno">pupeno</who>
    <bug_when>2006-07-06 11:41:20 +0000</bug_when>
    <thetext>I have the problem that it uses some normall font, which is too big for kicker so I can&apos;t read the temperature, it is too big and doesn&apos;t fit.
Being able to choose the size of the fonts used would be handy, automatically shrinking the label to fit would be awesome.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>504021</commentid>
    <comment_count>9</comment_count>
    <who name="Patrick">mail</who>
    <bug_when>2007-01-25 20:10:47 +0000</bug_when>
    <thetext>Yes, we _need_ configurable font display (colour, size). This missing feature renders kweather unusable for me :(
Also tooltip display with configurable information would be great, in case you just show the icon, but want more information while passing with the mouse cursor.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>511047</commentid>
    <comment_count>10</comment_count>
    <who name="Jeremy">jeremylb</who>
    <bug_when>2007-02-23 22:12:53 +0000</bug_when>
    <thetext>I agree! My background also is dark and my panel is transparent, thus making it almost impossible to read. A text configure option like the clock would make this applet so much better.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>511064</commentid>
    <comment_count>11</comment_count>
    <who name="Martin Koller">martin</who>
    <bug_when>2007-02-23 23:47:02 +0000</bug_when>
    <thetext>SVN commit 636720 by mkoller:

BUG: 80445
GUI:

Add possibility to define the text color


 M  +0 -1      dockwidget.h  
 M  +13 -0     kcmweather.cpp  
 M  +1 -0      kcmweather.h  
 M  +6 -1      kweather.cpp  
 M  +1 -0      kweather.h  
 M  +60 -1     prefdialogdata.ui  


--- branches/KDE/3.5/kdetoys/kweather/dockwidget.h #636719:636720
@@ -40,7 +40,6 @@
 
     void setLocationCode(const QString &amp;locationCode);
     void setViewMode(int);
-    void setFont(QFont newFont);
     void setOrientation(Orientation o) { m_orientation = o; }
     /** resize the view **/
     void resizeView(const QSize &amp;size);
--- branches/KDE/3.5/kdetoys/kweather/kcmweather.cpp #636719:636720
@@ -33,6 +33,7 @@
 #include &lt;klocale.h&gt;
 #include &lt;kurlrequester.h&gt;
 #include &lt;kcombobox.h&gt;
+#include &lt;kcolorbutton.h&gt;
 
 #include &quot;dockwidget.h&quot;
 #include &quot;prefdialogdata.h&quot;
@@ -68,6 +69,8 @@
                                 SLOT( changeViewMode( int ) ) );
   connect( mWidget-&gt;m_reportLocation, SIGNAL( activated( const QString&amp; ) ),
                                       SLOT( reportLocationChanged() ) );
+  connect( mWidget-&gt;m_textColor, SIGNAL( changed(const QColor &amp;) ),
+                                 SLOT( textColorChanged(const QColor &amp;) ) );
 
   KAboutData *about = new KAboutData(
       I18N_NOOP( &quot;kcmweather&quot; ),
@@ -151,6 +154,11 @@
   emit changed( true );
 }
 
+void KCMWeather::textColorChanged(const QColor &amp;)
+{
+  emit changed( true );
+}
+
 void KCMWeather::load()
 {
   kdDebug() &lt;&lt; &quot;Load&quot; &lt;&lt; endl;
@@ -161,6 +169,10 @@
   mWidget-&gt;m_enableLog-&gt;setChecked( enabled );
   enableLogWidgets( enabled );
 
+  static QColor black(Qt::black);
+  QColor textColor = config.readColorEntry(&quot;textColor&quot;, &amp;black);
+  mWidget-&gt;m_textColor-&gt;setColor(textColor);
+
   QString loc =  config.readEntry( &quot;report_location&quot; );
 
   mWidget-&gt;m_logFile-&gt;setURL( config.readPathEntry( &quot;log_file_name&quot; ) );
@@ -181,6 +193,7 @@
   config.setGroup( &quot;General Options&quot; );
   config.writeEntry( &quot;logging&quot;, mWidget-&gt;m_enableLog-&gt;isChecked() );
   config.writeEntry( &quot;log_file_name&quot;, mWidget-&gt;m_logFile-&gt;url() );
+  config.writeEntry( &quot;textColor&quot;, mWidget-&gt;m_textColor-&gt;color() );
 
   // Station idx to local idx; if nothing selected yet, keep it empty
   QString loc;
--- branches/KDE/3.5/kdetoys/kweather/kcmweather.h #636719:636720
@@ -47,6 +47,7 @@
     void enableLogWidgets( bool value );
     void changeViewMode( int mode );
     void reportLocationChanged();
+    void textColorChanged(const QColor &amp;);
 
   private:
     prefsDialogData *mWidget;
--- branches/KDE/3.5/kdetoys/kweather/kweather.cpp #636719:636720
@@ -52,7 +52,7 @@
         QWidget *parent, const char *name):
         KPanelApplet(configFile, t, actions, parent, name), weatherIface(),
         mFirstRun( false ), mReport( 0 ), mClient( 0 ),
-        mContextMenu( 0 ), mWeatherService( 0 ), settingsDialog( 0 )
+        mContextMenu( 0 ), mWeatherService( 0 ), settingsDialog( 0 ), mTextColor(Qt::black)
 {
 	kdDebug(12004) &lt;&lt; &quot;Constructor &quot; &lt;&lt; endl;
 	setObjId(&quot;weatherIface&quot;);
@@ -64,6 +64,7 @@
 	dockWidget = new dockwidget(reportLocation, this, &quot;dockwidget&quot;);
 	connect(dockWidget, SIGNAL(buttonClicked()), SLOT(doReport()));
 	dockWidget-&gt;setViewMode(mViewMode);
+	dockWidget-&gt;setPaletteForegroundColor(mTextColor);
 
 	timeOut = new QTimer(this, &quot;timeOut&quot; );
 	connect(timeOut, SIGNAL(timeout()), SLOT(timeout()));
@@ -212,6 +213,9 @@
     fileName = kcConfig-&gt;readPathEntry(&quot;log_file_name&quot;);
     reportLocation = kcConfig-&gt;readEntry(&quot;report_location&quot;);
     mViewMode = kcConfig-&gt;readNumEntry(&quot;smallview_mode&quot;, dockwidget::ShowAll);
+
+    static QColor black(Qt::black);
+    mTextColor = kcConfig-&gt;readColorEntry(&quot;textColor&quot;, &amp;black);
 }
 
 /** Save the application mPrefs. */
@@ -324,6 +328,7 @@
 
     dockWidget-&gt;setLocationCode(reportLocation);
     dockWidget-&gt;setViewMode(mViewMode);
+    dockWidget-&gt;setPaletteForegroundColor(mTextColor);
     emit updateLayout();
 
     if (logOn &amp;&amp; !fileName.isEmpty())
--- branches/KDE/3.5/kdetoys/kweather/kweather.h #636719:636720
@@ -78,6 +78,7 @@
 	KPopupMenu *mContextMenu;
 	WeatherService_stub *mWeatherService;
 	KCMultiDialog *settingsDialog;
+	QColor mTextColor;
 };
 
 #endif
--- branches/KDE/3.5/kdetoys/kweather/prefdialogdata.ui #636719:636720
@@ -10,7 +10,7 @@
             &lt;x&gt;0&lt;/x&gt;
             &lt;y&gt;0&lt;/y&gt;
             &lt;width&gt;429&lt;/width&gt;
-            &lt;height&gt;289&lt;/height&gt;
+            &lt;height&gt;341&lt;/height&gt;
         &lt;/rect&gt;
     &lt;/property&gt;
     &lt;property name=&quot;sizePolicy&quot;&gt;
@@ -271,8 +271,66 @@
                 &lt;/widget&gt;
             &lt;/grid&gt;
         &lt;/widget&gt;
+        &lt;widget class=&quot;QGroupBox&quot;&gt;
+            &lt;property name=&quot;name&quot;&gt;
+                &lt;cstring&gt;groupBox3&lt;/cstring&gt;
+            &lt;/property&gt;
+            &lt;property name=&quot;title&quot;&gt;
+                &lt;string&gt;Text&lt;/string&gt;
+            &lt;/property&gt;
+            &lt;hbox&gt;
+                &lt;property name=&quot;name&quot;&gt;
+                    &lt;cstring&gt;unnamed&lt;/cstring&gt;
+                &lt;/property&gt;
+                &lt;widget class=&quot;QLayoutWidget&quot;&gt;
+                    &lt;property name=&quot;name&quot;&gt;
+                        &lt;cstring&gt;layout1&lt;/cstring&gt;
+                    &lt;/property&gt;
+                    &lt;hbox&gt;
+                        &lt;property name=&quot;name&quot;&gt;
+                            &lt;cstring&gt;unnamed&lt;/cstring&gt;
+                        &lt;/property&gt;
+                        &lt;widget class=&quot;QLabel&quot;&gt;
+                            &lt;property name=&quot;name&quot;&gt;
+                                &lt;cstring&gt;textLabel1&lt;/cstring&gt;
+                            &lt;/property&gt;
+                            &lt;property name=&quot;text&quot;&gt;
+                                &lt;string&gt;Color:&lt;/string&gt;
+                            &lt;/property&gt;
+                        &lt;/widget&gt;
+                        &lt;widget class=&quot;KColorButton&quot;&gt;
+                            &lt;property name=&quot;name&quot;&gt;
+                                &lt;cstring&gt;m_textColor&lt;/cstring&gt;
+                            &lt;/property&gt;
+                            &lt;property name=&quot;text&quot;&gt;
+                                &lt;string&gt;&lt;/string&gt;
+                            &lt;/property&gt;
+                        &lt;/widget&gt;
+                    &lt;/hbox&gt;
+                &lt;/widget&gt;
+                &lt;spacer&gt;
+                    &lt;property name=&quot;name&quot;&gt;
+                        &lt;cstring&gt;spacer3&lt;/cstring&gt;
+                    &lt;/property&gt;
+                    &lt;property name=&quot;orientation&quot;&gt;
+                        &lt;enum&gt;Horizontal&lt;/enum&gt;
+                    &lt;/property&gt;
+                    &lt;property name=&quot;sizeType&quot;&gt;
+                        &lt;enum&gt;Expanding&lt;/enum&gt;
+                    &lt;/property&gt;
+                    &lt;property name=&quot;sizeHint&quot;&gt;
+                        &lt;size&gt;
+                            &lt;width&gt;170&lt;/width&gt;
+                            &lt;height&gt;20&lt;/height&gt;
+                        &lt;/size&gt;
+                    &lt;/property&gt;
+                &lt;/spacer&gt;
+            &lt;/hbox&gt;
+        &lt;/widget&gt;
     &lt;/vbox&gt;
 &lt;/widget&gt;
+&lt;customwidgets&gt;
+&lt;/customwidgets&gt;
 &lt;connections&gt;
     &lt;connection&gt;
         &lt;sender&gt;m_enableLog&lt;/sender&gt;
@@ -297,5 +355,6 @@
     &lt;includehint&gt;kurlrequester.h&lt;/includehint&gt;
     &lt;includehint&gt;klineedit.h&lt;/includehint&gt;
     &lt;includehint&gt;kpushbutton.h&lt;/includehint&gt;
+    &lt;includehint&gt;kcolorbutton.h&lt;/includehint&gt;
 &lt;/includehints&gt;
 &lt;/UI&gt;
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>567908</commentid>
    <comment_count>12</comment_count>
    <who name="">p92</who>
    <bug_when>2008-01-03 11:48:51 +0000</bug_when>
    <thetext>Not only text color (currently implemented) but also background color and transparency implementation would allow kweather to better fit in modern kicker panel.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>568070</commentid>
    <comment_count>13</comment_count>
    <who name="Pupeno">pupeno</who>
    <bug_when>2008-01-03 21:06:12 +0000</bug_when>
    <thetext>At this moment I would prioritize have it working with KDE 4 if it is not already. But that&apos;s just a user&apos;s opinion.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>