Skip to content

Release 4.3.0 rc4#1715

Merged
demiankatz merged 4 commits intoUniversalViewer:release-4.3.0from
FalveyLibraryTechnology:release-4.3.0-rc4
Apr 2, 2026
Merged

Release 4.3.0 rc4#1715
demiankatz merged 4 commits intoUniversalViewer:release-4.3.0from
FalveyLibraryTechnology:release-4.3.0-rc4

Conversation

@demiankatz
Copy link
Copy Markdown
Contributor

@demiankatz demiankatz commented Mar 26, 2026

This is the fourth release candidate for 4.3.0, fixing more bugs discovered since RC3 (#1634) and updating further dependencies.

This RC also includes some further adjustments to the release workflow in an effort to get NPM publishing working again. Automatic publishing is still failing, however, so this release has been published manually.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 26, 2026

@demiankatz is attempting to deploy a commit to the Universal Viewer Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
universalviewer Ready Ready Preview, Comment Mar 26, 2026 1:42pm

Request Review

@LanieOkorodudu
Copy link
Copy Markdown
Collaborator

I noticed something while using the keyboard(Tab navigation).
After the Setting button gets highlighted, pressing Tab again takes the focus straight to the canvas. Just want wanted to check if this is the expected behavior?
Screenshot 2026-03-30 094820

Also, the Next and Previous buttons on the canvas show a highlight like in the screenshot. It feels a bit different compared to the previous version.
Not sure if this is an intentional change or something to look into.
Screenshot 2026-03-30 095201
Screenshot 2026-03-30 095216

@demiankatz
Copy link
Copy Markdown
Contributor Author

Thanks for taking a look, @LanieOkorodudu!

The focus on the canvas is intentional: when the canvas is focused, you can move the image with the arrow keys and zoom in and out with the -/+ keys. If you can't focus the canvas, this functionality is lost. It looks to me like the functionality already exists in 4.2.1, but the canvas doesn't show a focus outline, so it's less clear what is going on. That problem will be fixed in 4.3.0.

I believe the change to the next/previous focus outline is related to the CSS changes that added the canvas outline. @jamesmisson and I also noticed that. I don't particularly like it, but I haven't had time to investigate how difficult it would be to fix. If anybody has time to correct it, I'd welcome the fix -- but if not, I don't think it's bad enough to delay the release.

@LanieOkorodudu
Copy link
Copy Markdown
Collaborator

but if not, I don't think it's bad enough to delay the release.

Yeah, I agree, I don't think this is a blocker.

@demiankatz
Copy link
Copy Markdown
Contributor Author

but if not, I don't think it's bad enough to delay the release.

Yeah, I agree, I don't think this is a blocker.

Fortunately, it was easy to fix in #1725!

@jamesmisson jamesmisson self-requested a review April 2, 2026 16:29
@demiankatz demiankatz merged commit 784fb7a into UniversalViewer:release-4.3.0 Apr 2, 2026
4 of 5 checks passed
@demiankatz demiankatz deleted the release-4.3.0-rc4 branch April 2, 2026 16:32
@demiankatz demiankatz mentioned this pull request Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants