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

addition keep-aria-label-attribute option #416

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

Conversation

dar0xt
Copy link

@dar0xt dar0xt commented Jan 6, 2025

Add keep aria label attribute option.

@aravindkarnam
Copy link
Collaborator

aravindkarnam commented Jan 22, 2025

@dar0xt Thanks for your contribution. Could you please elaborate on the purpose/benefits of this PR?

Given that primary purpose of aria labels is to provide a text label for improving accessibility for users relying on assistive technologies like screen readers. And Crawl4AI's output is for consumption by LLMs, how does keeping aria labels benefit our users?

@aravindkarnam aravindkarnam added the ✊🏼 On-hold Currently put on pause, due to a blocker label Jan 22, 2025
@dar0xt
Copy link
Author

dar0xt commented Jan 22, 2025

@aravindkarnam Some sites contain important information in aria-labels, and without this option, information could be missing. So it would be helpful to have this option available as a selectable feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✊🏼 On-hold Currently put on pause, due to a blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants