Skip to content

Disallow tabbing to buttons when they're disabled#1154

Merged
demiankatz merged 2 commits into
UniversalViewer:devfrom
Geoffsc:1086-DisablePreviousButton
Oct 22, 2024
Merged

Disallow tabbing to buttons when they're disabled#1154
demiankatz merged 2 commits into
UniversalViewer:devfrom
Geoffsc:1086-DisablePreviousButton

Conversation

@Geoffsc

@Geoffsc Geoffsc commented Oct 21, 2024

Copy link
Copy Markdown
Contributor

Fix for #1086

@vercel

vercel Bot commented Oct 21, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
universalviewer βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Oct 21, 2024 3:59pm

@Geoffsc Geoffsc changed the title disallow tabbing to buttons when they're disabled Disallow tabbing to buttons when they're disabled Oct 21, 2024

@demiankatz demiankatz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Geoffsc -- I have one possible improvement to suggest:

Comment thread src/content-handlers/iiif/modules/uv-pdfheaderpanel-module/PDFHeaderPanel.ts Outdated

@demiankatz demiankatz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thanks, @Geoffsc!

@jamesmisson

Copy link
Copy Markdown
Contributor

This works for me and the code looks good.

@demiankatz demiankatz merged commit 841da32 into UniversalViewer:dev Oct 22, 2024
@demiankatz

Copy link
Copy Markdown
Contributor

Thanks, @Geoffsc and @jamesmisson -- merging now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Community Sprint COMPLETED

Development

Successfully merging this pull request may close these issues.

3 participants