fix: use cross-only fullscreen diagram close control
Build and publish Eldorado documentation / build (pull_request) Successful in 33s
Build and publish Eldorado documentation / build (pull_request) Successful in 33s
This commit is contained in:
1 parent
577117e19c
commit
d3b9f7af14
3 files changed
+12
-2
No files matched your search
@@ -65,6 +65,16 @@ test('Archify is pinned, licensed and rebuilt, not a remote embed', () => {
|
||||
assert(wrapper.includes('aria-hidden="true"'));
|
||||
assert(read('src/css/custom.css').includes('justify-content: flex-end'));
|
||||
});
|
||||
test('full-screen close is cross-only with an accessible name and square touch target', () => {
|
||||
const wrapper = read('src/components/ArchifyDiagram/index.tsx');
|
||||
const button = wrapper.match(/<button[^>]*aria-label="Close full-screen diagram"[^>]*>(.*?)<\/button>/s);
|
||||
assert(button, 'close button retains its accessible name');
|
||||
assert.equal(button[1], '<span aria-hidden="true">×</span>');
|
||||
assert(button[0].includes('autoFocus') && button[0].includes('onClick={close}'));
|
||||
const styles = read('src/css/custom.css').match(/\.archify-modal-toolbar button \{([^}]+)\}/)[1];
|
||||
assert(styles.includes('width: 44px') && styles.includes('height: 44px'));
|
||||
assert(styles.includes('padding: 0') && styles.includes('flex-shrink: 0'));
|
||||
});
|
||||
test('Product owns two teams and Technology owns five separate teams', () => {
|
||||
const spec = JSON.parse(read('diagrams/organization-structure.architecture.json'));
|
||||
const page = read('docs/organization/structure.mdx');
|
||||
|
||||
Reference in new issue
Block a user