Skip to content

[ENH] Replace listViews with searchable listViews#4924

Merged
janezd merged 2 commits intobiolab:masterfrom
PrimozGodec:list-view-search
Sep 18, 2020
Merged

[ENH] Replace listViews with searchable listViews#4924
janezd merged 2 commits intobiolab:masterfrom
PrimozGodec:list-view-search

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