From 3427f6f62f078ef21e0a1b5d518931e36b73a78d Mon Sep 17 00:00:00 2001 From: Maria Hutt Date: Thu, 30 Mar 2023 09:57:02 -0700 Subject: [PATCH] Revert "bug(breadcrumbs): color attribute shows on DOM for Vue (#27040)" This reverts commit dd419c0066d312f45c741bbe399d9764faf64841. --- core/api.txt | 2 +- .../components/breadcrumbs/breadcrumbs.tsx | 2 +- packages/vue/test/base/src/router/index.ts | 4 -- .../vue/test/base/src/views/Breadcrumbs.vue | 53 ------------------- .../vue/test/base/src/views/Components.vue | 3 -- .../base/tests/e2e/specs/breadcrumbs.cy.js | 9 ---- 6 files changed, 2 insertions(+), 71 deletions(-) delete mode 100644 packages/vue/test/base/src/views/Breadcrumbs.vue delete mode 100644 packages/vue/test/base/tests/e2e/specs/breadcrumbs.cy.js diff --git a/core/api.txt b/core/api.txt index e008c4afa4a..fbd1257f294 100644 --- a/core/api.txt +++ b/core/api.txt @@ -199,7 +199,7 @@ ion-breadcrumb,part,native ion-breadcrumb,part,separator ion-breadcrumbs,shadow -ion-breadcrumbs,prop,color,"danger" | "dark" | "light" | "medium" | "primary" | "secondary" | "success" | "tertiary" | "warning" | string & Record | undefined,undefined,false,true +ion-breadcrumbs,prop,color,"danger" | "dark" | "light" | "medium" | "primary" | "secondary" | "success" | "tertiary" | "warning" | string & Record | undefined,undefined,false,false ion-breadcrumbs,prop,itemsAfterCollapse,number,1,false,false ion-breadcrumbs,prop,itemsBeforeCollapse,number,1,false,false ion-breadcrumbs,prop,maxItems,number | undefined,undefined,false,false diff --git a/core/src/components/breadcrumbs/breadcrumbs.tsx b/core/src/components/breadcrumbs/breadcrumbs.tsx index 7b7bb2b7388..10850fab17b 100644 --- a/core/src/components/breadcrumbs/breadcrumbs.tsx +++ b/core/src/components/breadcrumbs/breadcrumbs.tsx @@ -30,7 +30,7 @@ export class Breadcrumbs implements ComponentInterface { * Default options are: `"primary"`, `"secondary"`, `"tertiary"`, `"success"`, `"warning"`, `"danger"`, `"light"`, `"medium"`, and `"dark"`. * For more information on colors, see [theming](/docs/theming/basics). */ - @Prop({ reflect: true }) color?: Color; + @Prop() color?: Color; /** * The maximum number of breadcrumbs to show before collapsing. diff --git a/packages/vue/test/base/src/router/index.ts b/packages/vue/test/base/src/router/index.ts index 236ec08afac..8ad0c0d4772 100644 --- a/packages/vue/test/base/src/router/index.ts +++ b/packages/vue/test/base/src/router/index.ts @@ -70,10 +70,6 @@ const routes: Array = [ path: '/components', component: () => import('@/views/Components.vue'), }, - { - path: '/components/breadcrumbs', - component: () => import('@/views/Breadcrumbs.vue') - }, { path: '/components/select', component: () => import('@/views/Select.vue') diff --git a/packages/vue/test/base/src/views/Breadcrumbs.vue b/packages/vue/test/base/src/views/Breadcrumbs.vue deleted file mode 100644 index 7e823056c05..00000000000 --- a/packages/vue/test/base/src/views/Breadcrumbs.vue +++ /dev/null @@ -1,53 +0,0 @@ - - - diff --git a/packages/vue/test/base/src/views/Components.vue b/packages/vue/test/base/src/views/Components.vue index 84c30f9b67c..de63bbf9905 100644 --- a/packages/vue/test/base/src/views/Components.vue +++ b/packages/vue/test/base/src/views/Components.vue @@ -2,9 +2,6 @@ - - Breadcrumbs - Select diff --git a/packages/vue/test/base/tests/e2e/specs/breadcrumbs.cy.js b/packages/vue/test/base/tests/e2e/specs/breadcrumbs.cy.js deleted file mode 100644 index a98f7853e69..00000000000 --- a/packages/vue/test/base/tests/e2e/specs/breadcrumbs.cy.js +++ /dev/null @@ -1,9 +0,0 @@ -describe("Components: Breadcrumbs", () => { - beforeEach(() => { - cy.visit("http://localhost:8080/components/breadcrumbs"); - }); - - it("should have color attribute", () => { - cy.get('ion-breadcrumbs#color').should('have.prop', 'color'); - }); -});