Skip to content

Conversation

@xristianstefanov
Copy link
Contributor

@xristianstefanov xristianstefanov self-assigned this Jul 13, 2025
@xristianstefanov xristianstefanov requested review from a team as code owners July 13, 2025 21:41
Copy link
Contributor

@ikoevska ikoevska left a comment

Choose a reason for hiding this comment

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

Right now, this is one of those articles that are practically "we have a feature, here's a demo". There's no real explanation of how to use it, tweak it, etc. There are no links or mentions of the API ref, to applicable properties, methods, interfaces, classes, etc. With our recurring theme of a11y, it might be worth mentioning if screen readers and keyboard navigation are supported. How does adaptiveness tie into this new feature?

There's also no crosslinking from other pages TO the new resource, which impedes discoverability and SEO.

I highly recommend adding more technical details and flesh to it and improving the crosslinking.

@xristianstefanov xristianstefanov merged commit 5a4d177 into master Jul 17, 2025
@github-actions github-actions bot deleted the pdfviewer-form-filling branch July 17, 2025 12:08
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.

4 participants