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] Replace listViews with searchable listViews #4924

Merged
merged 2 commits into from
Sep 18, 2020

Conversation

PrimozGodec
Copy link
Contributor

@PrimozGodec PrimozGodec commented Jul 28, 2020

Issue

ListViews do not have a filter

Description of changes

Replacing listViews with a searchable list view

It must not merged before https://github.com/biolab/orange-widget-base is released.

Includes
  • Code changes
  • Tests
  • Documentation

@PrimozGodec PrimozGodec changed the title [ENH] Replace listViews with searchable listViews [WIP][ENH] Replace listViews with searchable listViews Jul 28, 2020
@PrimozGodec PrimozGodec force-pushed the list-view-search branch 2 times, most recently from 09182e6 to d065e5a Compare September 7, 2020 07:26
@codecov
Copy link

codecov bot commented Sep 11, 2020

Codecov Report

Merging #4924 into master will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #4924      +/-   ##
==========================================
+ Coverage   84.47%   84.63%   +0.15%     
==========================================
  Files         283      284       +1     
  Lines       58344    58819     +475     
==========================================
+ Hits        49289    49781     +492     
+ Misses       9055     9038      -17     

@janezd
Copy link
Contributor

janezd commented Sep 17, 2020

This can now be reviewed and possibly merged, right? (Answer "no" or just remove "WIP".)

@PrimozGodec PrimozGodec changed the title [WIP][ENH] Replace listViews with searchable listViews [ENH] Replace listViews with searchable listViews Sep 17, 2020
@janezd janezd merged commit 7f6c51d into biolab:master Sep 18, 2020
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