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

feat(progress): Support for multiple progress bars #889

Merged
merged 20 commits into from
Aug 20, 2017
Merged

Conversation

tmorehouse
Copy link
Member

@tmorehouse tmorehouse commented Aug 19, 2017

Adds support for multiple progress bars by introducing a new b-progress-bar sub component.

sub component b-progress-bar will inherit many of the props from the parent b-progress component, but my be overridden at the sub component level.

PR includes updated docs and examples

@codecov-io
Copy link

codecov-io commented Aug 19, 2017

Codecov Report

Merging #889 into 1.x will increase coverage by 0.05%.
The diff coverage is 37.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##              1.x     #889      +/-   ##
==========================================
+ Coverage   40.74%   40.79%   +0.05%     
==========================================
  Files          88       89       +1     
  Lines        2236     2255      +19     
  Branches      644      653       +9     
==========================================
+ Hits          911      920       +9     
- Misses       1169     1179      +10     
  Partials      156      156
Impacted Files Coverage Δ
lib/components/form-input.vue 50% <100%> (ø) ⬆️
lib/components/progress-bar.vue 30.43% <30.43%> (ø)
lib/components/progress.vue 69.23% <50%> (+28.05%) ⬆️

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 07f4b43...8226b73. Read the comment docs.

.progress-bar {
transition: all .5s;
}
</style>
Copy link
Member Author

@tmorehouse tmorehouse Aug 19, 2017

Choose a reason for hiding this comment

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

Bootstrap V4.beta CSS includes transition for-progress-bar, so this CSS is not needed

@tmorehouse tmorehouse merged commit 76c613c into 1.x Aug 20, 2017
@tmorehouse tmorehouse deleted the tmorehouse/progress branch August 20, 2017 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants