[GRASS-dev] [GRASS GIS] #2514: G_OPT_C_BG and G_OPT_C_FG differ in support of color_none

GRASS GIS trac at osgeo.org
Sat Dec 27 20:37:27 PST 2014


#2514: G_OPT_C_BG and G_OPT_C_FG differ in support of color_none
-------------------------------------+--------------------------------------
 Reporter:  annakrat                 |       Owner:  grass-dev@…              
     Type:  defect                   |      Status:  new                      
 Priority:  normal                   |   Milestone:  7.0.0                    
Component:  Default                  |     Version:  svn-trunk                
 Keywords:  color, standard options  |    Platform:  All                      
      Cpu:  Unspecified              |  
-------------------------------------+--------------------------------------

Comment(by annakrat):

 Replying to [comment:1 mmetz]:
 > Replying to [ticket:2514 annakrat]:
 > > In the attempt to standardize options for fg and bg colors, we ignored
 the gisprompt where some commands have `old,color,color` and some
 `old,color_none,color`. So this for example results in d.vect gui dialog
 not supporting transparent feature color, because G_OPT_C_FG doesn't use
 `color_none`.
 > >
 > > Beside the gisprompt, also the description must be different. Should
 we have 4 versions of color standard options? Something like G_OPT_C_BG,
 G_OPT_C_FG, G_OPT_CN_BG, G_OPT_CN_FG?
 >
 > Too many just slightly different standard options contradict the idea of
 having standard options. You can not make every possible occurrence of an
 option a standard option. Standard options can be easily modified by
 modules to meet their needs. Therefore I would opt to drop G_OPT_C_FG and
 G_OPT_C_BG, and have instead G_OPT_C for `old,color,color` and G_OPT_CN
 for `old,color_none,color`. That would also fix the wrong description for
 G_OPT_C_FG.

 That makes sense, committed in r63797.

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/2514#comment:2>
GRASS GIS <http://grass.osgeo.org>



More information about the grass-dev mailing list