[GRASS-dev] [GRASS GIS] #1048: wxgui: g.region: can't reset `rows=' nor `cols='

GRASS GRASS developer
1

#1048: wxgui: g.region: can't reset `rows=' nor `cols='
----------------------+-----------------------------------------------------
Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.0
Component: default | Version: svn-releasebranch64
Keywords: | Platform: All
      Cpu: All |
----------------------+-----------------------------------------------------
1. Run g.region from wxgui.

2. `Resolution' tab.

3. Change the `Number of rows in region'.

4. "Oops, I wanted to edit the resolution instead."

5. Edit the `Grid resolution 2D (...)' field as you intended too.

6. You can't reset `rows=' in the GUI now.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1048&gt;
GRASS GIS <http://grass.osgeo.org>

2

#1048: wxgui: g.region: can't reset `rows=' nor `cols='
-----------------------+----------------------------------------------------
  Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: normal | Milestone: 6.4.0
Component: wxGUI | Version: svn-releasebranch64
Resolution: | Keywords: g.region
  Platform: All | Cpu: All
-----------------------+----------------------------------------------------
Changes (by martinl):

  * keywords: => g.region
  * component: default => wxGUI

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1048#comment:1&gt;
GRASS GIS <http://grass.osgeo.org>

3

#1048: wxgui: g.region: can't reset `rows=' nor `cols='
----------------------+-----------------------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.4
Component: wxGUI | Version: svn-releasebranch64
Keywords: g.region | Platform: All
      Cpu: All |
----------------------+-----------------------------------------------------
Changes (by neteler):

  * milestone: 6.4.0 => 6.4.4

Comment:

Indeed, no way to reset the "accidentally" hit rows= in the GUI.
I fear that there is no (easy) way to avoid this.

A nice idea would be to offer here the region dialog from
the Location wizard rather than the simply g.region interface.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1048#comment:2&gt;
GRASS GIS <http://grass.osgeo.org>

4

#1048: wxgui: g.region: can't reset `rows=' nor `cols='
----------------------+-----------------------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.4
Component: wxGUI | Version: svn-releasebranch64
Keywords: g.region | Platform: All
      Cpu: All |
----------------------+-----------------------------------------------------

Comment(by annakrat):

The same for v.label fontsize option. Easy way is to replace the spin
control with a text field, but we probably don't want this.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1048#comment:3&gt;
GRASS GIS <http://grass.osgeo.org>

5

#1048: wxgui: g.region: can't reset `rows=' nor `cols='
-------------------------------------+--------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.4
Component: wxGUI | Version: svn-releasebranch64
Keywords: g.region, v.label, --ui | Platform: All
      Cpu: All |
-------------------------------------+--------------------------------------
Changes (by hamish):

  * keywords: g.region => g.region, v.label, --ui

Comment:

n.b. this isn't a bug with any particular module, rather the module GUIs
generated by the parser.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1048#comment:4&gt;
GRASS GIS <http://grass.osgeo.org>

6

#1048: wxgui: g.region: can't reset `rows=' nor `cols='
-------------------------------------+--------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.4
Component: wxGUI | Version: svn-releasebranch64
Keywords: g.region, v.label, --ui | Platform: All
      Cpu: All |
-------------------------------------+--------------------------------------

Comment(by annakrat):

How about changing the spinctrl to text field with integer validator (the
same we did for floatspin)? In this case there is now sense in having the
spinctrl anyway. We can use the text field only in cases when there is no
required answer (and maybe no default answer).

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1048#comment:5&gt;
GRASS GIS <http://grass.osgeo.org>

7

#1048: wxgui: g.region: can't reset `rows=' nor `cols='
-------------------------------------+--------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.4
Component: wxGUI | Version: svn-releasebranch64
Keywords: g.region, v.label, --ui | Platform: All
      Cpu: All |
-------------------------------------+--------------------------------------

Comment(by mlennert):

Replying to [comment:5 annakrat]:
> How about changing the spinctrl to text field with integer validator
(the same we did for floatspin)? In this case there is now sense in having
the spinctrl anyway. We can use the text field only in cases when there is
no required answer (and maybe no default answer).

+1. Values can be so different that spin control does not really make
sense.

Moritz

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1048#comment:6&gt;
GRASS GIS <http://grass.osgeo.org>