D27785: [Fonts KCM] Change how nearestExistingFonts() finds a matching font

2020-04-01 Thread Ahmad Samir
ahmadsamir abandoned this revision. ahmadsamir added a comment. Not needed with D27808 , which will use KFontChooserDialog everywhere. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27785 To: ahmadsamir, #plasma, davidedmu

D27785: [Fonts KCM] Change how nearestExistingFonts() finds a matching font

2020-03-16 Thread Ahmad Samir
ahmadsamir added a comment. In D27785#628110 , @bport wrote: > Ok for me but need to wait approval on font dialog review (to ensure correct font style is selected when editing) Thanks. The font dialog review, D27808

D27785: [Fonts KCM] Change how nearestExistingFonts() finds a matching font

2020-03-16 Thread Benjamin Port
bport accepted this revision. bport added a comment. This revision is now accepted and ready to land. Ok for me but need to wait approval on font dialog review (to ensure correct font style is selected when editing) REPOSITORY R119 Plasma Desktop BRANCH arcpatch-D27785 (branched from mas

D27785: [Fonts KCM] Change how nearestExistingFonts() finds a matching font

2020-03-11 Thread Ahmad Samir
ahmadsamir updated this revision to Diff 77442. ahmadsamir retitled this revision from "[Fonts KCM] Change setNearestExistingFonts() to set the fonts only when necessary" to "[Fonts KCM] Change how nearestExistingFonts() finds a matching font". ahmadsamir edited the summary of this revision. ahma

D27785: [Fonts KCM] Change how nearestExistingFonts() finds a matching font

2020-03-09 Thread Ahmad Samir
ahmadsamir updated this revision to Diff 77265. ahmadsamir edited the summary of this revision. ahmadsamir added a comment. Tweak commit message REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27785?vs=77264&id=77265 BRANCH arcpatch-D27785 (branche

D27785: [Fonts KCM] Change how nearestExistingFonts() finds a matching font

2020-03-09 Thread Ahmad Samir
ahmadsamir updated this revision to Diff 77264. ahmadsamir marked 3 inline comments as done. ahmadsamir retitled this revision from "[Fonts KCM] Change setNearestExistingFonts() to set the fonts only when necessary" to "[Fonts KCM] Change how nearestExistingFonts() finds a matching font". ahmadsa