Skip to content

Conversation

mosinve
Copy link
Member

@mosinve mosinve commented Oct 21, 2017

Move height prop from progress-bar to progress according to current Bootstrap4-beta2.
fixes #1216

Move height prop from progress-bar to progress according to current Bootstrap4-beta2.
fixes #1216
@mosinve mosinve requested a review from tmorehouse October 21, 2017 14:08
@codecov-io
Copy link

codecov-io commented Oct 21, 2017

Codecov Report

Merging #1217 into dev will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1217      +/-   ##
==========================================
+ Coverage   32.86%   32.87%   +0.01%     
==========================================
  Files         109      109              
  Lines        2866     2865       -1     
  Branches      920      890      -30     
==========================================
  Hits          942      942              
+ Misses       1547     1546       -1     
  Partials      377      377
Impacted Files Coverage Δ
lib/components/progress-bar.vue 0% <ø> (ø) ⬆️
lib/components/progress.vue 0% <ø> (ø) ⬆️

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 cc3916c...a80d3c7. Read the comment docs.

@tmorehouse
Copy link
Member

Will need to adjust b-progress-bar as it also sets it's own height based on parent b-progress height value.

Basically just need to remove it as new CSS sets height === 100% on the progress-bar

@tmorehouse tmorehouse changed the title fix(Progress) height prop fix(Progress): Bootstrap V4.beta.2 CSS height prop change Oct 21, 2017
@tmorehouse tmorehouse merged commit a963ea3 into dev Oct 21, 2017
@tmorehouse tmorehouse deleted the mosinve-patch-1 branch October 21, 2017 15:14
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.

b-progress should apply the heigth prop to itself
3 participants