Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-11-18 Thread Christoph Feck

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

(Updated Nov. 18, 2016, 3:51 p.m.)


Status
--

This change has been discarded.


Review request for KDE Frameworks, KDE Usability and Andreas Kainz.


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 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) disable 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 30ddd34 
  src/kcharselect_p.h db0259c 

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


Testing
---


File Attachments


Character table with font substitution (default)
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
Character table with dimming substituted characters
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png


Thanks,

Christoph Feck



Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-14 Thread Christoph Feck


> On Sept. 4, 2016, 4:34 a.m., Christoph Feck wrote:
> > Andreas, any idea which icon the action could use?
> 
> Andreas Kainz wrote:
> I would use visibility and hint to show/hide the font's that are not 
> available in the selected font. will the font change to "not selected" grayed 
> out when you show all characters also the ones that aren't available?
> 
> Christoph Feck wrote:
> "visibility" and "hint" is the same icon, but one of them crossed-out. If 
> I use both of them, it is confusing, because the tool button itself has a 
> selected state (sunken frame and selected background).
> 
> So I see two options:
> 1) use the "hint" icon, and make the button unselected by default. 
> Selecting it, the unavailable characters are then grayed out.
> 2) use the "visibility" icon, and make the button selected by default.
> Unselecting it, the unavailable characters are then grayed out.
> 
> Which would you prefer? The code currently uses the unselected button for 
> the default (do not gray-out unavailable characters) but I can reverse it.
> 
> Regarding your question, I have difficulties to parse it. Let me try to 
> answer from what I understood: If font-merging is enabled, but none of the 
> installed fonts have glyphs for a specific character, then this character is 
> rendered as an rectangle box. The character cell is not disabled, so you will 
> be able to select the character (e.g. to copy/paste it), even if you have no 
> font with this character.
> 
> Heiko Tietze wrote:
> Third option is to have a less obtrusive visualization (going back to 
> your first proposal) that makes the toggle button obsolete. But I have no 
> idea how to do so since graying out is the opposite of unobtrusive. It 
> depends heavily on the workflow meaning whether or not users should be able 
> to deal with those characters at all.
> 
> Andreas Kainz wrote:
> personally I wouldn't grayed out by default cause I want to have this 
> symbol and I don't care if this is available or not.
> 
> on the other hand I think your app is really nice and usefull but you 
> need some time to understand the different drop downs, ... so I didn't know 
> your initial proposal. I will update some mockup where there is written all 
> fonts and if someone select a font (arial) you see the not available chars 
> grayed out as in your app.
> 
> Andreas Kainz wrote:
> https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/kcharselect.png

If font merging is enabled, Qt must know which font you prefer, because the 
replacement fonts are selected to be as similar as possible to the requested 
font. In other words, simply having an "All Fonts" choice does not allow the 
user to chose the base font.

I checked your proposal, and indeed having a separate list of scripts is on my 
TODO list, but this is orthogonal to this feature request.


- Christoph


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


On Sept. 1, 2016, 6:45 p.m., Christoph Feck wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128680/
> ---
> 
> (Updated Sept. 1, 2016, 6:45 p.m.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Andreas Kainz.
> 
> 
> 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 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) disable 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 30ddd34 
>   src/kcharselect_p.h db0259c 
> 
> Diff: https://git.reviewboard.kde.org/r/128680/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Character table with font substitution (default)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
> Character table with dimming substituted characters
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b

Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-13 Thread Andreas Kainz


> On Sept. 4, 2016, 2:34 vorm., Christoph Feck wrote:
> > Andreas, any idea which icon the action could use?
> 
> Andreas Kainz wrote:
> I would use visibility and hint to show/hide the font's that are not 
> available in the selected font. will the font change to "not selected" grayed 
> out when you show all characters also the ones that aren't available?
> 
> Christoph Feck wrote:
> "visibility" and "hint" is the same icon, but one of them crossed-out. If 
> I use both of them, it is confusing, because the tool button itself has a 
> selected state (sunken frame and selected background).
> 
> So I see two options:
> 1) use the "hint" icon, and make the button unselected by default. 
> Selecting it, the unavailable characters are then grayed out.
> 2) use the "visibility" icon, and make the button selected by default.
> Unselecting it, the unavailable characters are then grayed out.
> 
> Which would you prefer? The code currently uses the unselected button for 
> the default (do not gray-out unavailable characters) but I can reverse it.
> 
> Regarding your question, I have difficulties to parse it. Let me try to 
> answer from what I understood: If font-merging is enabled, but none of the 
> installed fonts have glyphs for a specific character, then this character is 
> rendered as an rectangle box. The character cell is not disabled, so you will 
> be able to select the character (e.g. to copy/paste it), even if you have no 
> font with this character.
> 
> Heiko Tietze wrote:
> Third option is to have a less obtrusive visualization (going back to 
> your first proposal) that makes the toggle button obsolete. But I have no 
> idea how to do so since graying out is the opposite of unobtrusive. It 
> depends heavily on the workflow meaning whether or not users should be able 
> to deal with those characters at all.
> 
> Andreas Kainz wrote:
> personally I wouldn't grayed out by default cause I want to have this 
> symbol and I don't care if this is available or not.
> 
> on the other hand I think your app is really nice and usefull but you 
> need some time to understand the different drop downs, ... so I didn't know 
> your initial proposal. I will update some mockup where there is written all 
> fonts and if someone select a font (arial) you see the not available chars 
> grayed out as in your app.

https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/kcharselect.png


- Andreas


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


On Sept. 1, 2016, 4:45 nachm., Christoph Feck wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128680/
> ---
> 
> (Updated Sept. 1, 2016, 4:45 nachm.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Andreas Kainz.
> 
> 
> 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 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) disable 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 30ddd34 
>   src/kcharselect_p.h db0259c 
> 
> Diff: https://git.reviewboard.kde.org/r/128680/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Character table with font substitution (default)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
> Character table with dimming substituted characters
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>



Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-13 Thread Andreas Kainz


> On Sept. 4, 2016, 2:34 vorm., Christoph Feck wrote:
> > Andreas, any idea which icon the action could use?
> 
> Andreas Kainz wrote:
> I would use visibility and hint to show/hide the font's that are not 
> available in the selected font. will the font change to "not selected" grayed 
> out when you show all characters also the ones that aren't available?
> 
> Christoph Feck wrote:
> "visibility" and "hint" is the same icon, but one of them crossed-out. If 
> I use both of them, it is confusing, because the tool button itself has a 
> selected state (sunken frame and selected background).
> 
> So I see two options:
> 1) use the "hint" icon, and make the button unselected by default. 
> Selecting it, the unavailable characters are then grayed out.
> 2) use the "visibility" icon, and make the button selected by default.
> Unselecting it, the unavailable characters are then grayed out.
> 
> Which would you prefer? The code currently uses the unselected button for 
> the default (do not gray-out unavailable characters) but I can reverse it.
> 
> Regarding your question, I have difficulties to parse it. Let me try to 
> answer from what I understood: If font-merging is enabled, but none of the 
> installed fonts have glyphs for a specific character, then this character is 
> rendered as an rectangle box. The character cell is not disabled, so you will 
> be able to select the character (e.g. to copy/paste it), even if you have no 
> font with this character.
> 
> Heiko Tietze wrote:
> Third option is to have a less obtrusive visualization (going back to 
> your first proposal) that makes the toggle button obsolete. But I have no 
> idea how to do so since graying out is the opposite of unobtrusive. It 
> depends heavily on the workflow meaning whether or not users should be able 
> to deal with those characters at all.

personally I wouldn't grayed out by default cause I want to have this symbol 
and I don't care if this is available or not.

on the other hand I think your app is really nice and usefull but you need some 
time to understand the different drop downs, ... so I didn't know your initial 
proposal. I will update some mockup where there is written all fonts and if 
someone select a font (arial) you see the not available chars grayed out as in 
your app.


- Andreas


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


On Sept. 1, 2016, 4:45 nachm., Christoph Feck wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128680/
> ---
> 
> (Updated Sept. 1, 2016, 4:45 nachm.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Andreas Kainz.
> 
> 
> 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 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) disable 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 30ddd34 
>   src/kcharselect_p.h db0259c 
> 
> Diff: https://git.reviewboard.kde.org/r/128680/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Character table with font substitution (default)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
> Character table with dimming substituted characters
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>



Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-12 Thread Heiko Tietze


> On Sept. 4, 2016, 2:34 vorm., Christoph Feck wrote:
> > Andreas, any idea which icon the action could use?
> 
> Andreas Kainz wrote:
> I would use visibility and hint to show/hide the font's that are not 
> available in the selected font. will the font change to "not selected" grayed 
> out when you show all characters also the ones that aren't available?
> 
> Christoph Feck wrote:
> "visibility" and "hint" is the same icon, but one of them crossed-out. If 
> I use both of them, it is confusing, because the tool button itself has a 
> selected state (sunken frame and selected background).
> 
> So I see two options:
> 1) use the "hint" icon, and make the button unselected by default. 
> Selecting it, the unavailable characters are then grayed out.
> 2) use the "visibility" icon, and make the button selected by default.
> Unselecting it, the unavailable characters are then grayed out.
> 
> Which would you prefer? The code currently uses the unselected button for 
> the default (do not gray-out unavailable characters) but I can reverse it.
> 
> Regarding your question, I have difficulties to parse it. Let me try to 
> answer from what I understood: If font-merging is enabled, but none of the 
> installed fonts have glyphs for a specific character, then this character is 
> rendered as an rectangle box. The character cell is not disabled, so you will 
> be able to select the character (e.g. to copy/paste it), even if you have no 
> font with this character.

Third option is to have a less obtrusive visualization (going back to your 
first proposal) that makes the toggle button obsolete. But I have no idea how 
to do so since graying out is the opposite of unobtrusive. It depends heavily 
on the workflow meaning whether or not users should be able to deal with those 
characters at all.


- Heiko


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


On Sept. 1, 2016, 4:45 nachm., Christoph Feck wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128680/
> ---
> 
> (Updated Sept. 1, 2016, 4:45 nachm.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Andreas Kainz.
> 
> 
> 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 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) disable 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 30ddd34 
>   src/kcharselect_p.h db0259c 
> 
> Diff: https://git.reviewboard.kde.org/r/128680/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Character table with font substitution (default)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
> Character table with dimming substituted characters
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>



Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-12 Thread Christoph Feck


> On Sept. 4, 2016, 4:34 a.m., Christoph Feck wrote:
> > Andreas, any idea which icon the action could use?
> 
> Andreas Kainz wrote:
> I would use visibility and hint to show/hide the font's that are not 
> available in the selected font. will the font change to "not selected" grayed 
> out when you show all characters also the ones that aren't available?

"visibility" and "hint" is the same icon, but one of them crossed-out. If I use 
both of them, it is confusing, because the tool button itself has a selected 
state (sunken frame and selected background).

So I see two options:
1) use the "hint" icon, and make the button unselected by default. Selecting 
it, the unavailable characters are then grayed out.
2) use the "visibility" icon, and make the button selected by default.
Unselecting it, the unavailable characters are then grayed out.

Which would you prefer? The code currently uses the unselected button for the 
default (do not gray-out unavailable characters) but I can reverse it.

Regarding your question, I have difficulties to parse it. Let me try to answer 
from what I understood: If font-merging is enabled, but none of the installed 
fonts have glyphs for a specific character, then this character is rendered as 
an rectangle box. The character cell is not disabled, so you will be able to 
select the character (e.g. to copy/paste it), even if you have no font with 
this character.


- Christoph


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


On Sept. 1, 2016, 6:45 p.m., Christoph Feck wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128680/
> ---
> 
> (Updated Sept. 1, 2016, 6:45 p.m.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Andreas Kainz.
> 
> 
> 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 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) disable 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 30ddd34 
>   src/kcharselect_p.h db0259c 
> 
> Diff: https://git.reviewboard.kde.org/r/128680/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Character table with font substitution (default)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
> Character table with dimming substituted characters
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>



Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-10 Thread Andreas Kainz


> On Sept. 4, 2016, 2:34 vorm., Christoph Feck wrote:
> > Andreas, any idea which icon the action could use?

I would use visibility and hint to show/hide the font's that are not available 
in the selected font. will the font change to "not selected" grayed out when 
you show all characters also the ones that aren't available?


- Andreas


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


On Sept. 1, 2016, 4:45 nachm., Christoph Feck wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128680/
> ---
> 
> (Updated Sept. 1, 2016, 4:45 nachm.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Andreas Kainz.
> 
> 
> 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 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) disable 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 30ddd34 
>   src/kcharselect_p.h db0259c 
> 
> Diff: https://git.reviewboard.kde.org/r/128680/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Character table with font substitution (default)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
> Character table with dimming substituted characters
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>



Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-03 Thread Christoph Feck

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



Andreas, any idea which icon the action could use?

- Christoph Feck


On Sept. 1, 2016, 4:45 p.m., Christoph Feck wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128680/
> ---
> 
> (Updated Sept. 1, 2016, 4:45 p.m.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Andreas Kainz.
> 
> 
> 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 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) disable 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 30ddd34 
>   src/kcharselect_p.h db0259c 
> 
> Diff: https://git.reviewboard.kde.org/r/128680/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Character table with font substitution (default)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
> Character table with dimming substituted characters
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>



Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-01 Thread Christoph Feck

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

(Updated Sept. 1, 2016, 4:45 p.m.)


Review request for KDE Frameworks, KDE Usability and Andreas Kainz.


Changes
---

- disable characters, instead of just dimming them
- rebase on current master


Summary (updated)
-

[KCharSelect] Add ToolButton for disabling characters not available in selected 
font


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


Repository: kwidgetsaddons


Description (updated)
---

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 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) disable 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 (updated)
-

  src/kcharselect.cpp 30ddd34 
  src/kcharselect_p.h db0259c 

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


Testing
---


File Attachments


Character table with font substitution (default)
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/7220dddb-ca1b-42a2-966c-791925156baf__snapshot1.png
Character table with dimming substituted characters
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/08/15/67c02866-b12c-4080-b6a4-22e29006448b__snapshot2.png


Thanks,

Christoph Feck