Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typos #7337

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix typos #7337

wants to merge 1 commit into from

Conversation

task-jp
Copy link
Contributor

@task-jp task-jp commented Jan 12, 2025

Ran typos . and fixed all typos that do make sense.

https://crates.io/crates/typos

Ran `typos .` and fixed all typos that do make sense.
https://crates.io/crates/typos
@@ -2172,7 +2172,7 @@ extern "C" fn supported_accessibility_actions(
.accessibility_props
.0
.keys()
.filter_map(|x| x.strip_prefix("acessible-action-"))
.filter_map(|x| x.strip_prefix("accessible-action-"))
Copy link
Member

Choose a reason for hiding this comment

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

This one makes me shake my head - how did this work before?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought there would be the counter-part but haven't found it.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah I think your change is entirely correct. This probably never worked before....

Copy link
Member

Choose a reason for hiding this comment

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

When I test accessible actions, I use the accessibility inspector, which requires an app bundle. So I always end up compiling rust code, instead of using the interpreter. But evidently we're missing automated test coverage.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the confirmation.

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