-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128680/
-----------------------------------------------------------

Review request for KDE Frameworks and KDE Usability.


Bugs: 97420
    https://bugs.kde.org/show_bug.cgi?id=97420


Repository: kwidgetsaddons


Description
-------

Adds a toolbutton to the right of the font combobox to control the "font 
merging" property of QFont. When selected, characters which are not available 
in the font are dimmed (i.e. look disabled).

Seeking feedback about:
- placement of the button
- used icon (currently "format-text-strikethrough"; it shows an S character 
which could stand for "substitution")
- action name and tooltip (see line 477 and 479)
- and code changes ;)

I was unsure if the toolbutton could have a popup menu showing three options:
1) show characters from all fonts
2) dim characters not available in font
3) hide characters not available in font

but I did not implement it, because it felt odd not being able to see _which_ 
characters are not available in the font.


Diffs
-----

  src/kcharselect.cpp 45c1e99 

Diff: https://git.reviewboard.kde.org/r/128680/diff/


Testing
-------


Thanks,

Christoph Feck

Reply via email to