Discussion:
D17493: Implementation of an Easy Mode in KCM fonts module
Nathaniel Graham
2018-12-10 19:44:59 UTC
Permalink
ngraham added a comment.


1. This needs screenshots. :)
2. Don't you think it's premature to submit code before we've settled on what it is we actually want? I don't see any consensus in T7927 <https://phabricator.kde.org/T7927>. Personally I am not at all sold on the idea of presenting users with ten font rendering options and expecting them to be able to choose their favorite. The differences between most of them are minuscule and seem all but impossible for someone who's not a font nerd to distinguish. The idea of visual previews seems sound, but I think we would need to limit them to choices that lay people actually have a chance of understanding and distinguishing between.

REPOSITORY
R119 Plasma Desktop

REVISION DETAIL
https://phabricator.kde.org/D17493

To: harmathy
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
Max Harmathy
2018-12-10 17:56:57 UTC
Permalink
harmathy created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
harmathy requested review of this revision.

REPOSITORY
R119 Plasma Desktop

REVISION DETAIL
https://phabricator.kde.org/D17493

AFFECTED FILES
CMakeLists.txt
kcms/fonts/CMakeLists.txt
kcms/fonts/fonts.cpp
kcms/fonts/freetype-renderer.cpp
kcms/fonts/freetype-renderer.h
kcms/fonts/menupreview.cpp
kcms/fonts/menupreview.h
kcms/fonts/package/contents/ui/SelectablePreview.qml
kcms/fonts/package/contents/ui/main.qml
kcms/fonts/renderpreviewimageprovider.cpp
kcms/fonts/renderpreviewimageprovider.h

To: harmathy
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
Loading...