fix(card): Drop img-fluid property (#2548)

jacobmllr95 · tmorehouse · commit cfc685fab151 · 2019-02-13T10:21:14.000-04:00
diff --git a/src/components/card/README.md b/src/components/card/README.md
@@ -499,22 +499,22 @@ When using card groups with footers, their content will automatically line up.
 ```html
 <div>
   <b-card-group>
-    <b-card title="Title" img-src="https://placekitten.com/g/300/450" img-alt="Img" img-top>
+    <b-card title="Title" img-src="https://placekitten.com/g/300/450" img-alt="Image" img-top>
       <b-card-text>
         This is a wider card with supporting text below as a natural lead-in to additional content.
         This content is a little bit longer.
       </b-card-text>
       <div slot="footer"><small class="text-muted">Last updated 3 mins ago</small></div>
     </b-card>
 
-    <b-card title="Title" img-src="https://placekitten.com/g/300/450" img-alt="Img" img-top>
+    <b-card title="Title" img-src="https://placekitten.com/g/300/450" img-alt="Image" img-top>
       <b-card-text>
         This card has supporting text below as a natural lead-in to additional content.
       </b-card-text>
       <div slot="footer"><small class="text-muted">Last updated 3 mins ago</small></div>
     </b-card>
 
-    <b-card title="Title" img-src="https://placekitten.com/g/300/450" img-alt="Img" img-top>
+    <b-card title="Title" img-src="https://placekitten.com/g/300/450" img-alt="Image" img-top>
       <b-card-text>
         This is a wider card with supporting text below as a natural lead-in to additional content.
         This card has even longer content than the first to show that equal height action.
@@ -536,22 +536,22 @@ automatically line up.
 ```html
 <div>
   <b-card-group deck>
-    <b-card title="Title" img-src="https://picsum.photos/300/300/?image=41" img-alt="Img" img-top>
+    <b-card title="Title" img-src="https://picsum.photos/300/300/?image=41" img-alt="Image" img-top>
       <b-card-text>
         This is a wider card with supporting text below as a natural lead-in to additional content.
         This content is a little bit longer.
       </b-card-text>
       <div slot="footer"><small class="text-muted">Last updated 3 mins ago</small></div>
     </b-card>
 
-    <b-card title="Title" img-src="https://picsum.photos/300/300/?image=41" img-alt="Img" img-top>
+    <b-card title="Title" img-src="https://picsum.photos/300/300/?image=41" img-alt="Image" img-top>
       <b-card-text>
         This card has supporting text below as a natural lead-in to additional content.
       </b-card-text>
       <div slot="footer"><small class="text-muted">Last updated 3 mins ago</small></div>
     </b-card>
 
-    <b-card title="Title" img-src="https://picsum.photos/300/300/?image=41" img-alt="Img" img-top>
+    <b-card title="Title" img-src="https://picsum.photos/300/300/?image=41" img-alt="Image" img-top>
       <b-card-text>
         This is a wider card with supporting text below as a natural lead-in to additional content.
         This card has even longer content than the first to show that equal height action.
@@ -579,8 +579,7 @@ set them to display: inline-block as column-break-inside: avoid isn’t a bullet
     <b-card
       title="Card title that wraps to a new line"
       img-src="https://placekitten.com/g/400/450"
-      img-fluid
-      img-alt="image"
+      img-alt="Image"
       img-top
     >
       <b-card-text>
@@ -601,8 +600,7 @@ set them to display: inline-block as column-break-inside: avoid isn’t a bullet
     <b-card
       title="Title"
       img-src="https://placekitten.com/500/350"
-      img-fluid
-      img-alt="image"
+      img-alt="Image"
       img-top
     >
       <b-card-text>
@@ -628,9 +626,9 @@ set them to display: inline-block as column-break-inside: avoid isn’t a bullet
       <b-card-text class="small text-muted">Last updated 3 mins ago</b-card-text>
     </b-card>
 
-    <b-card img-src="https://picsum.photos/400/400/?image=41" img-fluid img-alt="image" overlay />
+    <b-card img-src="https://picsum.photos/400/400/?image=41" img-alt="Image" overlay />
 
-    <b-card img-src="https://picsum.photos/400/200/?image=41" img-fluid img-alt="image" img-top>
+    <b-card img-src="https://picsum.photos/400/200/?image=41" img-alt="Image" img-top>
       <b-card-text>
         This is a wider card with supporting text below as a natural lead-in to additional content.
         This card has even longer content than the first.
diff --git a/src/components/card/card-img.js b/src/components/card/card-img.js
@@ -43,10 +43,6 @@ export const props = {
   width: {
     type: String,
     default: null
-  },
-  fluid: {
-    type: Boolean,
-    default: false
   }
 }
 
@@ -70,7 +66,7 @@ export default {
     return h(
       'img',
       mergeData(data, {
-        class: [baseClass, props.fluid ? 'img-fluid' : null],
+        class: [baseClass],
         attrs: {
           src: props.src,
           alt: props.alt,
diff --git a/src/components/card/card-img.spec.js b/src/components/card/card-img.spec.js
@@ -115,20 +115,6 @@ describe('card-image', async () => {
     expect(wrapper.classes().length).toBe(1)
   })
 
-  it('has class "img-fluid" when fluid=true', async () => {
-    const wrapper = mount(CardImg, {
-      context: {
-        props: {
-          src: 'https://picsum.photos/600/300/?image=25',
-          fluid: true
-        }
-      }
-    })
-    expect(wrapper.classes()).toContain('card-img')
-    expect(wrapper.classes()).toContain('img-fluid')
-    expect(wrapper.classes().length).toBe(2)
-  })
-
   it('has attribute alt when prop alt set', async () => {
     const wrapper = mount(CardImg, {
       context: {
