On Sun, Dec 03, 2006 at 01:00:08AM +0100, Enrico Forestieri wrote: > On Sat, Dec 02, 2006 at 10:08:15PM +0100, Georg Baum wrote: > > > Am Samstag, 2. Dezember 2006 17:01 schrieb John Levon: > > > On Sat, Dec 02, 2006 at 05:00:33PM +0100, Enrico Forestieri wrote: > > > > > > > > That is OK with me, provided you find out that the comment about > > lying > > > > > QFontInfo is not true anymore. > > > > > > > > Seemingly, this was a sort of bug in older Qt versions. See for example > > > > http://lists.trolltech.com/qt-interest/2000-02/thread00025-0.html > > > > where the same problem is illustrated (and a possible workaround > > given). > > > > > > > > However, trusting the documentation at > > > > http://doc.trolltech.com/4.2/qfontinfo.html > > > > > > > > The QFontInfo class provides the same access functions as QFont, > > > > e.g. family(), pointSize(), italic(), weight(), fixedPitch(), > > > > styleHint() etc. But whilst the QFont access functions return the > > > > values that were set, a QFontInfo object returns the values that > > > > apply to the font that will actually be used to draw the text. > > > > > > It always said that, and it's been broken for god knows how long. > > > Hopefully they really did fix it in Qt4. It should obvious if they did, > > > you can remove this logic, and you still get the right fonts on Linux. > > > > Then it looks like it is fixed indeed. Enrico, which qt version did you > > test it against? We should make sure that it does not only work with qt > > 4.2, but also with our minimum required version (which one is that BTW?) > > I tested it against Qt 4.1.4. As regards the minimum required version, > from the thread here: > http://thread.gmane.org/gmane.editors.lyx.devel/67343/focus=67810 > it seems that Qt >= 4.1.2 is to be preferred, 4.1.3 being the minimum > recommended.
LyX can only be compiled using Qt 4.1.0 or higher because we use the QSyntaxHighlighter class and it is not available in older releases. AFAICS, before giving up on this missing class, the only (easily solvable) problem with 4.0.1 would have been missing definitions for Q_SLOTS and Q_SIGNALS in the Qt headers. I've tested the patch against 4.1.0 and it worked, so I am going to shove it in. -- Enrico
