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

[ENH] Open draged files on CSV Import, Load Model and Distance File widgets #6747

Merged
merged 4 commits into from
Mar 1, 2024

Conversation

ales-erjavec
Copy link
Contributor

Issue

Open dragged files on CSV Import, Load Model and Distance File widgets similar to how existing File works.

Description of changes
  • owcsvimport.py Handle drops on the widget. Implement common OWUrlDropBase abstract class.
  • Use OWUrlDropBase in File, LoadMode, Distance File widgets.
Includes
  • Code changes
  • Tests
  • Documentation

@ales-erjavec ales-erjavec force-pushed the owcsvimport-drop-handler branch 3 times, most recently from a5a35e8 to 5dd73c9 Compare February 29, 2024 10:55
Copy link

codecov bot commented Feb 29, 2024

Codecov Report

Merging #6747 (cfb79c4) into master (ba951fa) will increase coverage by 0.04%.
The diff coverage is 96.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6747      +/-   ##
==========================================
+ Coverage   88.13%   88.18%   +0.04%     
==========================================
  Files         322      322              
  Lines       70589    70665      +76     
==========================================
+ Hits        62216    62317     +101     
+ Misses       8373     8348      -25     

@ales-erjavec ales-erjavec force-pushed the owcsvimport-drop-handler branch 2 times, most recently from f75d0a1 to adeb968 Compare February 29, 2024 14:13
@janezd janezd self-assigned this Feb 29, 2024
@janezd janezd merged commit 0487f58 into biolab:master Mar 1, 2024
21 of 27 checks passed
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.

2 participants