[GRASS-dev] Final report for Mini project 2022: Redesigning a map display status bar combo box into a new settings dialog

2022-03-02 Thread Linda Kladivová
Hello everyone,



I am sending my final Mini Project report. The more detailed version with 
permanent links on GitHub PRs and with several screenshots can be found at
the project wiki [1].





Abstract:


Especially for Single-Window GUI, the Map Display status bar needs to be 
lightened up a bit since in the case of opening more Map Display notebook 
pages the check box on the right side of the status bar gets shrunk. The 
solution is to remove the combo box in the status bar occupying too much 
space and implement a new Map Display settings dialog accessible from the 
top map display toolbar.




The state of the art BEFORE the start of Mini Project:


At this moment, the status bar at the bottom of the Map Display window
consists of the text/widget section, the combo box, the mask and the check
box. For the Single-Window GUI, the number of widgets must be reduced so 
that the check box on the right side of the status bar is not shrunk.




The state of the art AFTER Mini Project:


The new Map Display settings dialog is introduced. The main task of this 
dialog is to manage General Map Display settings and Statusbar settings. It
implies that the status bar combobox is not needed anymore because we can 
find all Map Display settings including status bar customization in the new
dialog.




Next Steps:
Other possible improvements related to status bar stuff could be:

   * Moving Projection page from GUI settings to the Map Display status bar
   settings
   * Adding context menu for status bar customization
   * Adding EPSG code label of current location to the main status bar
   * Merging „Go to coordinates X, Y“ and „Coordinates“ item into one widget


Permanent links and screenshots can be found at [2] and [3].




[1] https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/RedesigningStatusbar
(https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/RedesigningStatusbar)


[2] https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/RedesigningStatusbar#
FinalReport
(https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/RedesigningStatusbar#FinalReport)


[3] https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/RedesigningStatusbar#
BrandnewScreenshotsofMapDisplaySettingsdialog
(https://trac.osgeo.org/grass/wiki/wxGUIDevelopment/RedesigningStatusbar#BrandnewScreenshotsofMapDisplaySettingsdialog)





Have a nice week,
Linda___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev


[GRASS-dev] Question regarding GRASS_NOTIFY

2022-03-02 Thread Nicklas Larsson via grass-dev
Hi Devs,

Is there anyone out there who know about usage of the environment variable 
GRASS_NOTIFY, historically and in the present?

In addressing an otherwise trivial compiler warning, I stumbled into this for 
me strange piece of code and usage of GRASS_NOTIFY in the Raster Display 
Library. I put up a PR [1] suggesting the removal of this, but I agree with 
Vaclav I’d better ask here on the ML.

The only use of GRASS_NOTIFY is in “D_close_driver()” [2] and the value of that 
env variable (if set) is directly inserted in a “system()” call, without any 
sanitation!

Furthermore, GRASS_NOTIFY is not documented [3] other than very briefly in API 
docs [4].
There is a note, dated 2009, suggesting its use with ximgview [5].


I’d love to hear of your thoughts, experiences on this.

Best,
Nicklas



[1] https://github.com/OSGeo/grass/pull/2135
[2] 
https://github.com/OSGeo/grass/blob/28a86c92761bdeeba199fc7e2ee159f018b6d529/lib/display/r_raster.c#L164
[3] https://grass.osgeo.org/grass80/manuals/variables.html
[4] 
https://grass.osgeo.org/programming8/defs_2display_8h.html#ab0d0949ed77467d353a47bef17b97318
[5] https://lists.osgeo.org/pipermail/grass-dev/2009-December/047194.html
___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev