TODO list (again)

[Roland Krause]

Jens, thanks for your patch. 
Do you have a chance to test this for KDE-3? This is really what I
meant with the bugfix in the ToDo list, as you said, KDE-2 is becoming
irrelevant rather quickly now.

Roland

--- Jens Zurheide <jens.zurheide at gmx.de> wrote:
> Sunday 20 January 2002 11:20 Jens Zurheide
> >>2 BugFix:    Fix browser font selection
> 
> Here we go...
> 
> The patch was tested on a KDE-2.2.1 (SuSE 7.3) with KHTML from
> KDE-2.2.2 
> (cvs) with Anti-Aliasing on.
> 
> AS mentioned before, the colors settings are not configurable any
> more but 
> they could probably be made configurable. The defaults are taken form
> the 
> khtmlrc configuration file.
> 
> I have exchanged the font selection combo boxes (QComboBox) with
> KComboBox. 
> If anybody has a problem concering speed this could be reverted.
> 
> As most X11-Window systems display the characters in DPI I have
> changed 
> small to 8, medium to 10 and large to 12 DPI with a minimum font size
> of 8 
> DPI (hardcoded).
> 
> This patch contains the patches from yesterday except for the am_edit
> patch 
> for the templates.
> 
> Something forgotten? Maybe. Just look at the patch and drop me a note
> if 
> you have questions or something should be changed.
> 
> Regards,
> Jens
> 
> PS: the other patch is for khtml_settings of KDE-2.2.2 but as this is
> a 
> dying branch it is more or less not too interesting (maybe for
2.2.3)> Index: khtml_settings.cc
> ===================================================================
> RCS file: /home/kde/kdelibs/khtml/khtml_settings.cc,v
> retrieving revision 1.65.2.1
> diff -u -3 -w -p -r1.65.2.1 khtml_settings.cc
> --- khtml_settings.cc 2001/09/07 15:05:44 1.65.2.1
> +++ khtml_settings.cc 2002/01/20 02:22:17
> @@ -472,7 +472,7 @@ QString KHTMLSettings::fantasyFontName()
>  
>  void KHTMLSettings::setFont(const QFont::CharSet &charset, int i,
> const QString &n)
>  {
> -    QStringList fontList = fontsForCharset[charset];
> +    QStringList &fontList = fontsForCharset[charset];
>      while (fontList.count() <= (uint)i)
>        fontList.append(QString::null);
>      fontList[i] = n;
> 

> ATTACHMENT part 3 application/x-gzip name=kdevelop_docbrowser.diff.gz


__________________________________________________
Do You Yahoo!?
Send FREE video emails in Yahoo! Mail!
http://promo.yahoo.com/videomail/