[GRASS-dev] [GRASS GIS] #1100: Trim mapset name from maps list in wxGUI

#1100: Trim mapset name from maps list in wxGUI
-------------------------+--------------------------------------------------
Reporter: aghisla | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Keywords: | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
as a complete aside, for a while I've had the wish to clean that up a bit
in the wx GUI as I find it rather cluttering and hard to read.

perhaps: roads @ PERMANENT or roads (PERMANENT)

both in the map layer-list and in the list of maps selector. or maybe just
omit it in the map-selector pull-down list as it is already given by the
higher level mapset name in the mapset tree (blue text).

... and pack the (opacity: 100%) [:] stuff on the right edge of the row.

thoughts?

Hamish

[from #1051]

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

#1100: Trim mapset name from maps list in wxGUI
-------------------------+--------------------------------------------------
Reporter: aghisla | Owner: martinl
     Type: enhancement | Status: assigned
Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Keywords: | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by martinl):

* cc: grass-dev@… (added)
  * owner: grass-dev@… => martinl
  * status: new => assigned

Comment:

Please try out r44610

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: martinl
      Type: enhancement | Status: closed
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: fixed | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by martinl):

  * status: assigned => closed
  * resolution: => fixed

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: martinl
      Type: enhancement | Status: reopened
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by hamish):

  * status: closed => reopened
  * resolution: fixed =>

Comment:

this hasn't been addressed in the layer manager AFAICT.
are we talking about the same thing?

Hamish

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: martinl
      Type: enhancement | Status: reopened
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------

Comment(by hamish):

see attached screenshots.

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: martinl
      Type: enhancement | Status: reopened
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------

Comment(by martinl):

Replying to [comment:3 hamish]:

The first part of the wish has been implemented (list of maps selector), I
have overlooked the second part. Leaving open.

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------
Changes (by martinl):

* cc: grass-dev@… (removed)
* cc: martinl (added)
  * owner: martinl => grass-dev@…
  * status: reopened => new

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------

Comment(by martinl):

Replying to [comment:3 hamish]:
> this hasn't been addressed in the layer manager AFAICT.
> are we talking about the same thing?

please check out r45573 (devbr6) - `CT.TR_ALIGN_WINDOWS` aligns windows at
least vertically. Better? Enough for this ticket?

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------

Comment(by hamish):

Replying to [comment:7 martinl]:
> please check out r45573 (devbr6) - `CT.TR_ALIGN_WINDOWS` aligns windows
at least vertically.

I'm not sure exactly what windows are supposed to be aligning to what, but
trying the change I see the preferences button is now packed to the right
side, but hovering only 80% to the right side. seems locked at the bar
after the attribute manager icon, not packed to the right side.
better, but looks weird.

see new screenshot.

> Better? Enough for this ticket?

yes better, but not yet perfect :slight_smile:

also, thoughts on changing '@mapset' to ' (mapset)' in the layer entry to
make the list a bit less cluttered? For the screenshot mockup in this
report I just did that using right-click -> rename, so seems technically
possible.

thanks,
Hamish

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

#1100: Trim mapset name from maps list in wxGUI
--------------------------+-------------------------------------------------
  Reporter: aghisla | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: minor | Milestone: 6.5.0
Component: wxGUI | Version: unspecified
Resolution: | Keywords:
  Platform: Unspecified | Cpu: Unspecified
--------------------------+-------------------------------------------------

Comment(by martinl):

Replying to [comment:8 hamish]:
> Replying to [comment:7 martinl]:
> > please check out r45573 (devbr6) - `CT.TR_ALIGN_WINDOWS` aligns
windows at least vertically.
>
> I'm not sure exactly what windows are supposed to be aligning to what,
but trying the change I

Naturally windows (widgets) assigned to the tree items (topic of this
ticket)

> see the preferences button is now packed to the right side, but hovering
only 80% to the right side. seems locked at the bar after the attribute
manager icon, not packed to the right side.
> better, but looks weird.

I know, AFAIU there is only possibility to align these windows vertically
based on the first added item. Small trick
[source:grass/trunk/gui/wxpython/gui_modules/layertree.py#L783 here].

> > Better? Enough for this ticket?
>
> yes better, but not yet perfect :slight_smile:
>
> also, thoughts on changing '@mapset' to ' (mapset)' in the layer entry
to make the list a bit less cluttered? For the screenshot mockup in this
report I just did that using right-click -> rename, so seems technically
possible.

Not sure, '@' as name/mapset separator seems to be quite standard naming
convention in GRASS. Any opinion/preferences from other users?

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