D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Phabricator]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:21d962fe33c8: [kfontinst] Port to use icons from the icon 
theme and make HiDPI aware (authored by Guo Yunhe i...@guoyunhe.me).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D23868?vs=66229=66232

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

AFFECTED FILES
  kcms/kfontinst/apps/Installer.cpp
  kcms/kfontinst/kcmfontinst/16-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/16-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/22-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/22-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/CMakeLists.txt
  kcms/kfontinst/kcmfontinst/FontFilter.cpp
  kcms/kfontinst/kcmfontinst/FontFilter.h
  kcms/kfontinst/kcmfontinst/FontList.cpp
  kcms/kfontinst/kcmfontinst/GroupList.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.h
  kcms/kfontinst/viewpart/FontPreview.cpp

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  LGTM!

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe updated this revision to Diff 66229.
guoyunhe added a comment.


  Rename and move the find duplicate button

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D23868?vs=66193=66229

BRANCH
  master

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

AFFECTED FILES
  kcms/kfontinst/apps/Installer.cpp
  kcms/kfontinst/kcmfontinst/16-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/16-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/22-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/22-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/CMakeLists.txt
  kcms/kfontinst/kcmfontinst/FontFilter.cpp
  kcms/kfontinst/kcmfontinst/FontFilter.h
  kcms/kfontinst/kcmfontinst/FontList.cpp
  kcms/kfontinst/kcmfontinst/GroupList.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.h
  kcms/kfontinst/viewpart/FontPreview.cpp

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Nathaniel Graham]

ngraham added a comment.


  Rename Scan Duplicates... to Find Duplicates... and move the button to the 
left of Install from File... and then I think this will be good.

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe updated this revision to Diff 66193.
guoyunhe added a comment.


  Fix size policy

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D23868?vs=66191=66193

BRANCH
  master

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

AFFECTED FILES
  kcms/kfontinst/apps/Installer.cpp
  kcms/kfontinst/kcmfontinst/16-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/16-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/22-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/22-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/CMakeLists.txt
  kcms/kfontinst/kcmfontinst/FontFilter.cpp
  kcms/kfontinst/kcmfontinst/FontFilter.h
  kcms/kfontinst/kcmfontinst/FontList.cpp
  kcms/kfontinst/kcmfontinst/GroupList.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.h
  kcms/kfontinst/viewpart/FontPreview.cpp

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe updated this revision to Diff 66191.
guoyunhe added a comment.


  Avoid scrollbar. Put progressbar and font status to another row.

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D23868?vs=66157=66191

BRANCH
  master

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

AFFECTED FILES
  kcms/kfontinst/apps/Installer.cpp
  kcms/kfontinst/kcmfontinst/16-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/16-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/22-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/22-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/CMakeLists.txt
  kcms/kfontinst/kcmfontinst/FontFilter.cpp
  kcms/kfontinst/kcmfontinst/FontFilter.h
  kcms/kfontinst/kcmfontinst/FontList.cpp
  kcms/kfontinst/kcmfontinst/GroupList.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.h
  kcms/kfontinst/viewpart/FontPreview.cpp

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Nathaniel Graham]

ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.


  Uh-oh, moving the button down to the lower row now causes the layout to get 
so wide with English that it overflows the default System Settings window size: 
F7357577: Screenshot_20190915_150638.png .

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe updated this revision to Diff 66157.
guoyunhe added a comment.


  Similar filter layout as Colors KCM. Move "Scan Duplicates..." button to the 
bottom button group.

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D23868?vs=66101=66157

BRANCH
  master

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

AFFECTED FILES
  kcms/kfontinst/apps/Installer.cpp
  kcms/kfontinst/kcmfontinst/16-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/16-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/22-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/22-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/CMakeLists.txt
  kcms/kfontinst/kcmfontinst/FontFilter.cpp
  kcms/kfontinst/kcmfontinst/FontFilter.h
  kcms/kfontinst/kcmfontinst/FontList.cpp
  kcms/kfontinst/kcmfontinst/GroupList.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.h
  kcms/kfontinst/viewpart/FontPreview.cpp

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Nathaniel Graham]

ngraham added a comment.


  In D23868#532065 , @guoyunhe wrote:
  
  > Now I can get something like this (the QLineEdit and QPushButton still have 
different height):
  >
  > F7357403: image.png 
  
  
  That's better, and we should just fix 
https://bugs.kde.org/show_bug.cgi?id=411811

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe added a comment.


  Now I can get something like this (the QLineEdit and QPushButton still have 
different height):
  
  F7357403: image.png 

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Nathaniel Graham]

ngraham added a comment.


  In D23868#532046 , @guoyunhe wrote:
  
  > In D23868#532016 , @ngraham 
wrote:
  >
  > > Maybe in the interests of #consistency 
, let's make the filter feature 
look the same way that it does in the Colors KCM: F7357040: 
Screenshot_20190915_131425.png 
  >
  >
  > There is a major difference: the color filter is a combobox but the font 
filter is a menu with checkbox and child menu. I cannot change it to a combobox 
but can swap the order and try to add borders.
  
  
  Radio buttons, not checkboxes. Which means the entries are all mutually 
exclusive, which means it could actually be a combobox, but for the fact that 
some of its items have sub-menus.

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe added a comment.


  In D23868#532016 , @ngraham wrote:
  
  > Maybe in the interests of #consistency 
, let's make the filter feature 
look the same way that it does in the Colors KCM: F7357040: 
Screenshot_20190915_131425.png 
  
  
  There is a major difference: the color filter is a combobox but the font 
filter is a menu with checkbox and child menu. I cannot change it to a combobox 
but can swap the order and try to add borders.

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Nathaniel Graham]

ngraham added a comment.


  Maybe in the interests of #consistency 
, let's make the filter feature 
look the same way that it does in the Colors KCM: F7357040: 
Screenshot_20190915_131425.png 

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe updated this revision to Diff 66101.
guoyunhe added a comment.


  Unify button look

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D23868?vs=66074=66101

BRANCH
  master

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

AFFECTED FILES
  kcms/kfontinst/apps/Installer.cpp
  kcms/kfontinst/kcmfontinst/16-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/16-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/16-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/22-actions-disablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-enablefont.png
  kcms/kfontinst/kcmfontinst/22-actions-font-disable.png
  kcms/kfontinst/kcmfontinst/22-actions-font-enable.png
  kcms/kfontinst/kcmfontinst/CMakeLists.txt
  kcms/kfontinst/kcmfontinst/FontFilter.cpp
  kcms/kfontinst/kcmfontinst/FontFilter.h
  kcms/kfontinst/kcmfontinst/FontList.cpp
  kcms/kfontinst/kcmfontinst/GroupList.cpp
  kcms/kfontinst/kcmfontinst/KCmFontInst.cpp
  kcms/kfontinst/viewpart/FontPreview.cpp

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Yunhe Guo]

guoyunhe added a comment.


  I changed it to QToolButton, but it still look different:
  
  F7352361: image.png 

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D23868: [kfontinst] Port to use icons from the icon theme and make HiDPI aware

[Nathaniel Graham]

ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.


  Either use a ToolButton for the filter feature, or turn "Scan for duplicate 
fonts" into a PushButton, so they match one way or another.

REPOSITORY
  R119 Plasma Desktop

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

To: guoyunhe, #plasma, ngraham
Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart