Changes between Version 48 and Version 49 of GSoC/2021/SingleWindowLayout


Ignore:
Timestamp:
Jul 2, 2021, 5:48:54 AM (3 years ago)
Author:
lindakladivova
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • GSoC/2021/SingleWindowLayout

    v48 v49  
    313313== Week 4 ==
    314314'''1) What did I complete this week?'''\\
    315 As the last week, I focused on the GUI refactoring. I worked on the main refactoring PR (see ​https://github.com/OSGeo/grass/pull/1675). In terms of the main Map Display I am done here. However, me and mentors realized that changes influence also other Map Frames - particularly g.gui.rdigit, d.mon, Map Swipe, Ground Control Point and IClass frame.
    316 
    317 There were two ways to proceed. First, I started with the statusbars for all the frames that we need newly to be created based on wx.Statubar widget (see https://github.com/OSGeo/grass/pull/1689). Then we came up with another concept, namely delegating methods using metaprogramming. Although this path seemed cheap at first (we talked about it very intensively on videocall on Wednesday), after a few hours of studying, we came to the conclusion that it is inappropriate in our case. And this decision again opened PR https://github.com/OSGeo/grass/pull/1689.
    318 
    319 Meantime, I also discovered the bug related to hiding statusbar and toolbars that does not work properly for above-mentioned map frames as e.g. Map Swipe. I set up the issue for it (see https://github.com/OSGeo/grass/issues/1691).
     315Similarly as the last week, I focused on the GUI refactoring. I worked on the main refactoring PR (see ​https://github.com/OSGeo/grass/pull/1675). In terms of the main Map Display I am done here. However, me and my mentors realized that changes influence also other Map Frames - particularly g.gui.rdigit, d.mon, Map Swipe, Ground Control Point and IClass frame.
     316
     317There were two possible ways how to proceed. First, I started with the statusbars for all the frames that we need newly to be created based on wx.Statubar widget (see https://github.com/OSGeo/grass/pull/1689). Then we came up with another concept, namely delegating methods using metaprogramming. Although this path seemed cheap at first (we talked about it very intensively on videocall on Wednesday), after a few hours of studying, we came to the conclusion that it is inappropriate in our case. And this decision subsequently opened PR https://github.com/OSGeo/grass/pull/1689 again.
     318
     319Meantime, I also discovered the bug related to hiding statusbar and toolbars. Hiding does not work properly for above-mentioned map frames as e.g. Map Swipe. I set up the issue https://github.com/OSGeo/grass/issues/1691.
    320320
    321321To sum it up, this week was not so productive in terms of code, but key in determining what to do next.
     
    325325
    326326'''3) Is there any blocking issue?'''\\
    327 No, it is not. However, as I have already mentioned I am on holiday till Tuesday... and I am looking forward to new energy coming after the rest. :-)
     327No, it is not. However, as I have already mentioned I am on holiday till Tuesday.
     328I am looking forward to new energy coming after the rest.
    328329
    329330== Week 5 ==