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

Change Λrrow occurrences for Arrow #146

Merged
merged 1 commit into from
Apr 4, 2023

Conversation

calvellido
Copy link
Member

@calvellido calvellido commented Apr 4, 2023

This PR modifies all occurrences of referring to the library name as Λrrow for its proper name: Arrow.

While the usage of the uppercase lambda is cool, and it creates a nice effect on the text, it totally defeats the purpose of having accessible web content. And it also brings some inconsistencies and problems when writing the library name on different media, etc. It also creates the issue of the library name not being properly indexed by web crawlers and bots.

While some possible bypass for those problems could be the use of the aria-label property, it would really be a hacky shortcut misusing it, as that wouldn't be the goal of that HTML property. And all this while not really solving the underlying issue of having an improper (in terms of pure text) library name.

The usage of the uppercased lambda name, Λrrow, should only be used on display media like images, videos, and such, but never text content.

This closes #145

Copy link
Member

@raulraja raulraja left a comment

Choose a reason for hiding this comment

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

Thanks @calvellido !

@calvellido calvellido merged commit 5588652 into main Apr 4, 2023
@calvellido calvellido deleted the calvellido-accessible-library-name branch April 4, 2023 12:03
@nomisRev
Copy link
Member

nomisRev commented Apr 4, 2023

Awesome @calvellido!

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.

ARIA support for Λrrow
3 participants