Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

Add setActiveSearch function to tab storybook#3565

Merged
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
amelzer:fix-tab-storybook
Aug 4, 2017
Merged

Add setActiveSearch function to tab storybook#3565
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
amelzer:fix-tab-storybook

Conversation

@amelzer
Copy link
Copy Markdown
Contributor

@amelzer amelzer commented Aug 4, 2017

Associated Issue: #3557

Summary of Changes

  • add setActiveSearch property to storybook

Test Plan

Tell us a little a bit about how you tested your patch.

Example test plan:

  • Clicking “+” triggers a setActiveSearch action output in action logger

Screenshots/Videos (OPTIONAL)

storybook-tabs

@jasonLaster jasonLaster merged commit bc21b7f into firefox-devtools:master Aug 4, 2017
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 4, 2017

Codecov Report

Merging #3565 into master will increase coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3565      +/-   ##
==========================================
+ Coverage   52.52%   52.62%   +0.09%     
==========================================
  Files         119      119              
  Lines        4645     4652       +7     
  Branches      958      958              
==========================================
+ Hits         2440     2448       +8     
+ Misses       2205     2204       -1
Impacted Files Coverage Δ
src/components/shared/Button/PaneToggle.js 7.69% <0%> (ø) ⬆️
src/components/Editor/index.js 22.22% <0%> (+1.78%) ⬆️
src/components/Editor/Footer.js 3.84% <0%> (+1.88%) ⬆️

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 f60265b...5bc9c88. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants