Skip to content

Fixes issue with indexing after search filter.#427

Merged
stefanoborini merged 3 commits into
switch_to_vuefrom
fix_selected_index
May 9, 2017
Merged

Fixes issue with indexing after search filter.#427
stefanoborini merged 3 commits into
switch_to_vuefrom
fix_selected_index

Conversation

@martinRenou

@martinRenou martinRenou commented May 8, 2017

Copy link
Copy Markdown
Member

Fixes #426

@martinRenou martinRenou requested a review from stefanoborini May 8, 2017 12:14
@codecov-io

codecov-io commented May 8, 2017

Copy link
Copy Markdown

Codecov Report

Merging #427 into switch_to_vue will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##           switch_to_vue     #427   +/-   ##
==============================================
  Coverage          95.33%   95.33%           
==============================================
  Files                 88       88           
  Lines               4077     4077           
  Branches             259      259           
==============================================
  Hits                3887     3887           
  Misses               138      138           
  Partials              52       52

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fed2eca...2eff7d5. Read the comment docs.

@stefanoborini stefanoborini changed the title Fixes #426 Fixes issue with indexing after search filter. May 8, 2017
},

methods: {
index_of: function(app) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indexOf

@stefanoborini stefanoborini merged commit c85c168 into switch_to_vue May 9, 2017
@stefanoborini stefanoborini deleted the fix_selected_index branch May 9, 2017 17:23
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.

3 participants