[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    ColorCollectionNameType Not standard convention?
From:       "=?ISO-8859-1?Q?Percy_Camilo_Trive=F1o_Aucahuasi?=" <orgyforever () gmail ! com>
Date:       2008-06-01 9:01:36
Message-ID: 579229230806010201x76dc1ff1q4ba434409c0b1d95 () mail ! gmail ! com
[Download RAW message or body]

Hi all I just looking in kcolordialog.cpp and I noticed that this const:
ColorCollectionNameType have 2 items without follow the standard file names
(to name pallete files)

//BEGIN
const ColorCollectionNameType colorCollectionName[] = {
    { "Recent_Colors", I18N_NOOP2("palette name", "* Recent Colors *") },
    { "Custom_Colors", I18N_NOOP2("palette name", "* Custom Colors *") },
//END

This should be like this?

//BEGIN
const ColorCollectionNameType colorCollectionName[] = {
    { "Recent.colors", I18N_NOOP2("palette name", "* Recent Colors *") },
    { "Custom.colors", I18N_NOOP2("palette name", "* Custom Colors *") },
//END

Percy

[Attachment #3 (text/html)]

Hi all I just looking in kcolordialog.cpp and I noticed that this const: \
ColorCollectionNameType have 2 items without follow the standard file names (to name \
pallete files)<br><br>//BEGIN<br>const ColorCollectionNameType colorCollectionName[] \
= {<br> &nbsp;&nbsp;&nbsp; { &quot;Recent_Colors&quot;, I18N_NOOP2(&quot;palette \
name&quot;, &quot;* Recent Colors *&quot;) },<br>&nbsp;&nbsp;&nbsp; { \
&quot;Custom_Colors&quot;, I18N_NOOP2(&quot;palette name&quot;, &quot;* Custom Colors \
*&quot;) },<br>//END<br> <br>This should be like this?<br><br>//BEGIN<br>const \
ColorCollectionNameType colorCollectionName[] = {<br>&nbsp;&nbsp;&nbsp; { \
&quot;Recent.colors&quot;, I18N_NOOP2(&quot;palette name&quot;, &quot;* Recent Colors \
*&quot;) },<br>&nbsp;&nbsp;&nbsp; { &quot;Custom.colors&quot;, \
I18N_NOOP2(&quot;palette name&quot;, &quot;* Custom Colors *&quot;) },<br> \
//END<br><br>Percy<br><br>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic