Closed pavoljuhas closed 5 years ago
Merging #32 into maint will increase coverage by
2.63%
. The diff coverage is100%
.
@@ Coverage Diff @@
## maint #32 +/- ##
==========================================
+ Coverage 25.22% 27.86% +2.63%
==========================================
Files 88 89 +1
Lines 11634 11717 +83
==========================================
+ Hits 2935 3265 +330
+ Misses 8699 8452 -247
Impacted Files | Coverage Δ | |
---|---|---|
src/diffpy/pdfgui/tests/TestParametersPanel.py | 100% <100%> (ø) |
|
src/diffpy/pdfgui/gui/parameterspanel.py | 54.5% <100%> (+54.5%) |
:arrow_up: |
src/diffpy/pdfgui/gui/errorreportdialog.py | 22.01% <0%> (+22.01%) |
:arrow_up: |
src/diffpy/pdfgui/gui/errorwrapper.py | 38.46% <0%> (+38.46%) |
:arrow_up: |
...fpy/pdfgui/gui/wxExtensions/autowidthlabelsgrid.py | 66.66% <0%> (+66.66%) |
:arrow_up: |
src/diffpy/pdfgui/gui/debugoptions.py | 71.42% <0%> (+71.42%) |
:arrow_up: |
src/diffpy/pdfgui/gui/pdfpanel.py | 73.33% <0%> (+73.33%) |
:arrow_up: |
src/diffpy/pdfgui/gui/pdfguiglobals.py | 91.66% <0%> (+91.66%) |
:arrow_up: |
... and 1 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update fe47600...d9d39c6. Read the comment docs.
@dragonyanglong - can you please test the new behavior of Parameters Panel checkboxes and post any problems or comments here? If you can test on either of Linux or Windows platforms it would be really helpful.
@dragonyanglong - thank you for quick and detailed response.
The PR should fix the following misbehaviors of the Fixed-column checkboxes.
Thanks @pavoljuhas for the instructions. I finished the testing on linux Ubuntu 18.04.1 LTS and Mac macOS Sierra 10.12.4. Sorry I don't have windows computer.
[x] missed clicks when flipping one value
- deselect all checkboxes and then use arrow keys to select R1 C2
- click two times R2 C2 with about 2 second delay. The first click flips on, but the second click is ignored. It takes the 3rd click to flip checkbox off.
- comment: in macOS system, I noticed an additional misbehavior related to this. After finishing the two steps above, use arrow keys to deselect R1 C2, it will perform like clicked twice (equivalent as not changing anything). This PR fixes the problem.
[x] undesired flips when selecting parameters for "Fix / Free" menu
- click R1 C1 then shift-click R3 C2. The checkbox in third row flips, but it would be more intuitive if it remained the same.
[x] the Fix/Free behavior above
- "Fix / Free" menu should set the same flag in all selected rows, but it is somehow broken as you observed.
In addition, the src/diffpy/pdfgui/gui/parameterspanel.py
is not consistent with wxglade-generated code, which will be fixed by issue #31 (Sorry I didn't find time to work on that issue yet, I will do in near future, sorry about the delay)
Overall, the PR looks good to me. Thanks Pavol!
@dragonyanglong - thank you for checking!
This improves handling of the fixed checkboxes in the Parameters panel, especially when several cells are selected for the Fixed / Free context menu.