Opened 10 years ago

Last modified 4 years ago

#1048 new defect

wxgui: g.region: can't reset `rows=' nor `cols='

Reported by: msieczka Owned by: grass-dev@…
Priority: normal Milestone: 6.4.6
Component: wxGUI Version: svn-releasebranch64
Keywords: g.region, v.label, --ui Cc:
CPU: All Platform: All

Description

  1. Run g.region from wxgui.
  1. `Resolution' tab.
  1. Change the `Number of rows in region'.
  1. "Oops, I wanted to edit the resolution instead."
  1. Edit the `Grid resolution 2D (...)' field as you intended too.
  1. You can't reset `rows=' in the GUI now.

Change History (7)

comment:1 Changed 10 years ago by martinl

Component: defaultwxGUI
Keywords: g.region added

comment:2 Changed 7 years ago by neteler

Milestone: 6.4.06.4.4

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.

comment:3 Changed 7 years ago 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.

comment:4 Changed 7 years ago by hamish

Keywords: v.label --ui added

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

comment:5 Changed 6 years ago 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).

comment:6 in reply to:  5 Changed 6 years ago by mlennert

Replying to 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

comment:7 Changed 4 years ago by neteler

Milestone: 6.4.46.4.6
Note: See TracTickets for help on using tickets.