| 232 | |
| 233 | == Week 5 == |
| 234 | '''1) What did I complete this week?''' |
| 235 | The PR https://github.com/OSGeo/grass/pull/2682 implementing the Export history button in the Console pane was merged. |
| 236 | |
| 237 | The first part of the week I worked on PR https://github.com/OSGeo/grass/pull/2718 which adds an input prompt sign to the command prompt widget. However, although it was our suggestion from https://github.com/OSGeo/grass/issues/2585, the intervention into the command prompt showed programmatically as very unfortunate - it was really hard to cover all cases of how the text in the command prompt could be managed. So, we decided to implement the task differently by showing a hint in the case that the command prompt is not focused. I think it is an even more user-friendly solution. It is already prepared for merging (see https://github.com/OSGeo/grass/pull/2728). |
| 238 | |
| 239 | Meanwhile, I worked on adding "isDocked" parameter to workspace handling in the PR https://github.com/OSGeo/grass/pull/2667 dealing with dock/undock map display functionality. It was the thing I forget to consider. However, the workspace issue revealed another error in https://github.com/OSGeo/grass/pull/2667 related to closing undocked map display frame. |
| 240 | |
| 241 | '''2) What am I going to achieve for next week?''' |
| 242 | |
| 243 | The goal for the next week is to merge both - https://github.com/OSGeo/grass/pull/2728 and https://github.com/OSGeo/grass/pull/2667. Then I plan to create a PR on the Console reorganization according to the suggestion in https://github.com/OSGeo/grass/issues/2585. |
| 244 | |
| 245 | '''3) Is there any blocking issue?''' |
| 246 | There is one blocking error in https://github.com/OSGeo/grass/pull/2667 related to the erroneous closing of undocked map display frame (wrapped C/C++ object of type MapPageFrame has been deleted). I have already spent several hours to fix it but unsuccessfully. I will probably put it off now and go back later, with more ideas in my mind. |