On Wednesday 19 November 2014 16:17:52 šumski wrote:
I've tried the to forward port the patch to Qt5, but there it doesn't really work good. I.e. no text on hovering, the text thumbnail paints no text, etc.
Furthermore, i think the patch is incorrect, or at least an overkill. Original Qt code already checks error codes from freetype (qfontengine_ft.cpp L960-965), and if subpixel is not available it fallbacks to internal implementation (IOW, with that patch, the check will be done twice).
So, IMO wanted effect is best achieved with the current method of removing the FT_CONFIG_SUBPIXEL condition (or just passing the QT_USE_FREETYPE_LCDFILTER define). The former could be even integrated into Qt i think, as there is nothing different how the code would be compiled w/ and w/o the freetype define considering there is already runtime check implemented..
Yes, you're right. I've just debugged the packages, they really behave like that. Which leads me to another question: maybe it will make sense to do the same change for libqt4 instead of the patch I suggested earlier? -- Regards, Stas -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org