> On March 6, 2014, 1:55 p.m., Mark Gaiser wrote: > > How will this work with other non KDE apps (like Chrome), will they simply > > pickup the Oxygen font? > > > > I'm asking because fonts are working just fine now. If i open a GTK app in > > KDE it shows the fonts in the same manner as a KDE app would show them. > > That is consistent and quite likable :) > > Sebastian Kügler wrote: > They won't pick up the font. Nothing we can do about that in Plasma. > Needs the distro or the Chrome developers to do it. The Oxygen Widget style > for GTK could probably pick it up, though. > > This is unrelated to this patch, however. > > Mark Gaiser wrote: > Well, if this patch gives the user a broken consistency (Chrome is being > used by lots of people, including KDE folks) then i think it might be best to > keep the defaults consistent and keep it at sans-serif. > > That would at least keep the default consistent. > Another approach might be (don't know if that's done already) to generate > a local user font.conf which changes the default sans-serif font to whatever > font is set in KDE's font settings. > > Note: Chrome is just a sane example here. Others would be gparted, > firefox, <fill in any gtk app> > > Martin Gräßlin wrote: > @Mark: well it's orthogonal to this change. We need to start somewhere > the transition. If we block the first patch because it introduces > inconsistancies, we will never be able to adress it. Thus I think it's the > right approach to start with the what we have control over. > > Aleix Pol Gonzalez wrote: > @Mark: Nobody assures Chrome will be using "Sans Serif" either, this > patch doesn't make a difference in this regard...
@Martin, @Aleix; To be clear, I'm not against this change at all. I'm only against the result it will have. Sure, you need to start somewhere. In this case that would be assuring that all applications follow the font settings that are in kde's font settings. How one should tackle that, i don't know. I guess it's possible with a local fonts.conf file. @Aleix not a strong argument. GUI interfaces almost always use the default font settings. My exact point is that those defaults _won't_ change for GTK apps with this patch. - Mark ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116633/#review52255 ----------------------------------------------------------- On March 6, 2014, 3:40 p.m., Sebastian Kügler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116633/ > ----------------------------------------------------------- > > (Updated March 6, 2014, 3:40 p.m.) > > > Review request for Plasma and Àlex Fiestas. > > > Repository: frameworkintegration > > > Description > ------- > > Change default font settings to Oxygen font > > > Depend on kde:oxygen-fonts being installed. oxygen-fonts installs a file > called OxygenFontConfig.cmake, which is used for checking the dependency is > available. > > > Diffs > ----- > > CMakeLists.txt e249554 > src/platformtheme/kfontsettingsdata.cpp 62990ce > > Diff: https://git.reviewboard.kde.org/r/116633/diff/ > > > Testing > ------- > > Started kate without a kdeglobals file being present, fonts are picked up > correctly. > > > Thanks, > > Sebastian Kügler > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
