Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/1630-accordion-adapt-native-stan…
Browse files Browse the repository at this point in the history
…dard' into 1630-accordion-adapt-native-standard

# Conflicts:
#	showcases/e2e/accordion/accordion-a11y.spec.ts
  • Loading branch information
nmerget committed Nov 19, 2024
2 parents f9d2ab6 + 3a5058f commit 2d3c79f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions showcases/e2e/accordion-item/accordion-item-a11y.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import { runA11yCheckerTest, runAxeCoreTest } from '../default.ts';
import { lvl3 } from '../fixtures/variants';

// Showcase uses <li> outside of <ul> in this case
// TODO: Let's investigate whether we could prevent this deactivation later on
const axeDisableRules = ['listitem'];

test.describe('DBAccordionItem', () => {
Expand Down
1 change: 1 addition & 0 deletions showcases/e2e/accordion/accordion-a11y.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import {
import { lvl3 } from '../fixtures/variants';

// https://stackoverflow.com/questions/78129019/can-you-have-an-li-in-a-autonomous-custom-element-with-the-parent-ul-not
// TODO: Let's investigate whether we could prevent this deactivation later on
const skipAxe = hasWebComponentSyntax(process.env.showcase);

test.describe('DBAccordion', () => {
Expand Down

0 comments on commit 2d3c79f

Please sign in to comment.