refactor: CTabs: merge 'pills' and 'tabs' props to 'variant' prop

Author: woothu

src/components/index.d.ts

@@ -579,8 +579,7 @@ export declare class CDataTable extends Vue {
 export declare class CTabs extends Vue {
   fill: boolean
   justified: boolean
-  pills: boolean
-  tabs: boolean
+  variant: string
   fade: boolean
   vertical: [boolean, object]
   addNavWrapperClasses: [string, Array<any>, object]

src/components/tabs/CTabs.vue

@@ -48,10 +48,10 @@ export default {
   props: {
     fill: Boolean,
     justified: Boolean,
-    pills: Boolean,
-    tabs: {
-      type: Boolean,
-      default: true
+    variant: {
+      type: String,
+      default: 'tabs',
+      validator: val => ['', 'tabs', 'pills'].includes(val)
     },
     fade: {
       type: Boolean,
@@ -81,9 +81,9 @@ export default {
       return [
         this.addNavClasses,
         {
-          'nav' : true,
-          'nav-tabs': this.tabs && !this.pills,
-          'nav-pills': this.pills,
+          'nav': true,
+          'nav-tabs': this.variant === 'tabs',
+          'nav-pills': this.variant === 'pills',
           'flex-column': this.vertical,
           'nav-fill': this.fill,
           'nav-justified': this.justified,

src/components/tabs/tests/CTabs.spec.js

@@ -10,7 +10,7 @@ const defaultWrapper = mount(CTabs)
 //   propsData: {
 //     fill: true,
 //     justified: true,
-//     pills: true,
+//     variant: 'pills',
 //     fade: true,
 //     vertical: true,
 //     addNavWrapperClasses: 'additional-nav-wrapper-class',
@@ -33,7 +33,7 @@ const App = Vue.extend({
         props: {
           fill: true,
           justified: true,
-          pills: true,
+          variant: 'pills',
           fade: true,
           vertical: { navs: 'col-sm-3', content: 'col-sm-9' },
           addNavWrapperClasses: 'additional-nav-wrapper-class',