Review Request: Fix bug 234407 - systemsettings iconview wordwrapping cause narrow icon in zhcn locale

Yichao Yu yyc1992 at gmail.com
Wed Jan 9 14:42:57 GMT 2013



> On Jan. 9, 2013, 10:09 a.m., Ben Cooksley wrote:
> > systemsettings/icons/IconMode.cpp, line 183
> > <http://git.reviewboard.kde.org/r/108285/diff/1/?file=106110#file106110line183>
> >
> >     Not sure I like the idea of a hardcoded list of languages... is there a better way of determining if a language is CJK?
> 
> Christoph Feck wrote:
>     "CJK" is actually naming the languages which use CJK, so the list is hardcoded by definition.
> 
> Yichao Yu wrote:
>     I guess the question is not "which languages are CJK" but "which languages have this problem".
>

Plus, there may be english text even when the current locale is cjk[1], so I really don't think deciding from the current locale is a good idea.

Personally, I like what I am having now (larger spacing?). I don't know what it looks like without word wrap but I think multiple lines is better if the text is really too long (e.g. "?????????"). I did have a suggestion on how the word wrap should be done for cjk (see the last few lines in the description of this bug[2]) i.e. it may be better to keep each lines roughly the same length.

For detecting whether word wrapping should be used (and probably what method should be used in order to have a better appearance e.g. using the alternative method for CJK I mentioned above), I think it is probably a better way to detect blank space in the text. It might be a better idea to increase the threshold (maximum length) if there is not a single space in the text and use some better method to do word wrapping in such case. This may work for any language that allow word wrapping (I personally don't know any language that does not, correct me if I am wrong.....) (including English (for extremely long words) if you add "-" correctly).

[1] http://wstaw.org/m/2013/01/09/plasma-desktopr20016.png
[2] https://bugs.kde.org/show_bug.cgi?id=309780


- Yichao


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108285/#review25038
-----------------------------------------------------------


On Jan. 9, 2013, 4:33 a.m., Chao Feng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108285/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2013, 4:33 a.m.)
> 
> 
> Review request for kde-workspace.
> 
> 
> Description
> -------
> 
> CJK languages do not use space as words seperator. 
> 
> And a CJK translation of the text in Systemsettings are very short. A single line is enough for them.
> 
> 
> This addresses bug 234407.
>     http://bugs.kde.org/show_bug.cgi?id=234407
> 
> 
> Diffs
> -----
> 
>   systemsettings/icons/IconMode.cpp 37cfc4bed42e4d05fc4c01008f8ca2c63b287b5e 
> 
> Diff: http://git.reviewboard.kde.org/r/108285/diff/
> 
> 
> Testing
> -------
> 
> 1. Apply patch
> 2. Systemsetting show ok on CJK
> 
> 
> Thanks,
> 
> Chao Feng
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130109/19f1e65e/attachment.htm>


More information about the kde-core-devel mailing list