Skip to content

Fix structure nav for multi-sectioned IA manifests - #876

Merged
Dananji merged 1 commit into
mainfrom
multi-sectioned-ia-manifest
Oct 29, 2025
Merged

Fix structure nav for multi-sectioned IA manifests#876
Dananji merged 1 commit into
mainfrom
multi-sectioned-ia-manifest

Conversation

@Dananji

@Dananji Dananji commented Oct 20, 2025

Copy link
Copy Markdown
Collaborator

Related issue: #818

Following changes are made in this PR to improve structured navigation for Canvas ranges without mediafragment information. The IA Manifest in question has multiple canvases and are referenced in the structures property with the relevant Canvas ids. However, these ids don't have mediafragment information attached as expected by Ramp, e.g. url#t=0,2.0.
While working on this, I came across that if such ranges referencing to sections has children the collapsible UI is not operable by using left/right arrow keys to collapse/expand the sections as expected. This breaks accessibility for such structure displays. So, this PR has an additional fix to that as well.

Changes in the PR:

@cjcolvar cjcolvar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great!

@Dananji
Dananji merged commit 793f7ad into main Oct 29, 2025
2 checks passed
@Dananji
Dananji deleted the multi-sectioned-ia-manifest branch October 29, 2025 16:32
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.

2 participants