Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

itemmodels.VariableListModel drag-and-drop updates #1843

Merged
merged 2 commits into from Jan 3, 2017

Conversation

Projects
None yet
6 participants
@kernc
Copy link
Member

commented Dec 29, 2016

Issue

VariableListModel drag and drop didn't work out of the box.

Description of changes

Make drag and drop work out of the box. Remove redundant duplicate implementations.
Expected by biolab/orange3-prototypes#57.

Includes
  • Code changes
  • Tests
  • Documentation
@codecov-io

This comment has been minimized.

Copy link

commented Dec 29, 2016

Current coverage is 89.20% (diff: 100%)

Merging #1843 into master will not change coverage

@@             master      #1843   diff @@
==========================================
  Files            86         86          
  Lines          9077       9077          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           8097       8097          
  Misses          980        980          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 48e7e1c...468096d

@ajdapretnar

This comment has been minimized.

Copy link
Contributor

commented Dec 29, 2016

This breaks Corpus completely.

kernc added a commit to kernc/orange3-text that referenced this pull request Dec 29, 2016

@kernc kernc referenced this pull request Dec 29, 2016

Merged

Fixups #171

@kernc kernc force-pushed the kernc:VarListModel_dnd branch from 468096d to e7cf8a6 Jan 2, 2017

@kernc

This comment has been minimized.

Copy link
Member Author

commented Jan 3, 2017

Thanks. Shouldn't break Load Corpus anymore.

kernc added a commit to kernc/orange3-text that referenced this pull request Jan 3, 2017

@lanzagar lanzagar merged commit f0e4ea3 into biolab:master Jan 3, 2017

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details

@astaric astaric added this to the 3.3.10 milestone Jan 6, 2017

astaric added a commit to astaric/orange3 that referenced this pull request Jan 6, 2017

Merge pull request biolab#1843 from kernc/VarListModel_dnd
itemmodels.VariableListModel drag-and-drop updates
(cherry picked from commit f0e4ea3)

astaric added a commit to astaric/orange3 that referenced this pull request Jan 10, 2017

Merge pull request biolab#1843 from kernc/VarListModel_dnd
itemmodels.VariableListModel drag-and-drop updates
(cherry picked from commit f0e4ea3)

astaric added a commit to astaric/orange3 that referenced this pull request Jan 11, 2017

Merge pull request biolab#1843 from kernc/VarListModel_dnd
itemmodels.VariableListModel drag-and-drop updates
(cherry picked from commit f0e4ea3)

astaric added a commit to astaric/orange3 that referenced this pull request Jan 13, 2017

Merge pull request biolab#1843 from kernc/VarListModel_dnd
itemmodels.VariableListModel drag-and-drop updates
(cherry picked from commit f0e4ea3)

astaric added a commit to astaric/orange3 that referenced this pull request Jan 18, 2017

Merge pull request biolab#1843 from kernc/VarListModel_dnd
itemmodels.VariableListModel drag-and-drop updates
(cherry picked from commit f0e4ea3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.