Skip to content

[ENH] Edit Domain: Record transformations#3231

Merged
lanzagar merged 12 commits intobiolab:masterfrom
ales-erjavec:oweditdomain-record-transformations
Sep 12, 2018
Merged

[ENH] Edit Domain: Record transformations#3231
lanzagar merged 12 commits intobiolab:masterfrom
ales-erjavec:oweditdomain-record-transformations

Conversation

@ales-erjavec
Copy link
Contributor

@ales-erjavec ales-erjavec commented Sep 5, 2018

Issue

Improve 'Edit Domain' widget.

Fixes gh-3188

Description of changes
  • Fix/change PyListModel.setData and setItemData
  • Add PyListModel.moveRows implementation
  • Improve 'Edit Domain' widgets GUI state hinting for variable items with applied changes
  • Record reordering of categories
  • Allow adding new and dropping of existing categories
Includes
  • Code changes
  • Tests
  • Documentation

Fix setData and setItemData so they no longer clear all other roles for
the item.

NOTE: Setting an item via the 'python sequence' interface
(`model[idx] = val`) still clears data for all roles.
Return False from data setter overides on failure.
@ales-erjavec ales-erjavec force-pushed the oweditdomain-record-transformations branch from 0530d91 to 52071cf Compare September 5, 2018 13:44
@codecov-io
Copy link

codecov-io commented Sep 5, 2018

Codecov Report

Merging #3231 into master will decrease coverage by 0.01%.
The diff coverage is 82.04%.

@@            Coverage Diff            @@
##           master   #3231      +/-   ##
=========================================
- Coverage   82.82%   82.8%   -0.02%     
=========================================
  Files         346     346              
  Lines       59629   60072     +443     
=========================================
+ Hits        49385   49745     +360     
- Misses      10244   10327      +83

@ales-erjavec ales-erjavec force-pushed the oweditdomain-record-transformations branch from 498d7e4 to 68f9fc3 Compare September 10, 2018 11:53
@ales-erjavec ales-erjavec force-pushed the oweditdomain-record-transformations branch from 68f9fc3 to 7da1764 Compare September 10, 2018 12:02
@ajdapretnar
Copy link
Contributor

This is really nice and works so well. I vote for the inclusion into the upcoming release. @lanzagar

@ajdapretnar
Copy link
Contributor

I have updated documentation for the widget.

@lanzagar lanzagar added this to the 3.16 milestone Sep 12, 2018
@lanzagar lanzagar merged commit 972ec52 into biolab:master Sep 12, 2018
@ales-erjavec ales-erjavec deleted the oweditdomain-record-transformations branch September 19, 2018 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants