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

Operating System of tool does not appear in the list #229

Closed
josehilera opened this issue Apr 8, 2024 · 4 comments · Fixed by #447
Closed

Operating System of tool does not appear in the list #229

josehilera opened this issue Apr 8, 2024 · 4 comments · Fixed by #447
Assignees

Comments

@josehilera
Copy link
Collaborator

The info about the operating system included in the json about a tool, does not appear in the list.
Example json of "Color Contrast" includes:
"desktop": [
"iOS"
],
But in the list the info does not appear:
color-contrast
So, an user of the list cannot know that this tool is for iOS and not for Android.

@josehilera josehilera added the bug Something isn't working label Apr 8, 2024
@remibetin remibetin removed the bug Something isn't working label Feb 12, 2025
@remibetin
Copy link
Member

Thanks @josehilera

@iadawn @shawna-slh The bug is caused by a naming error. I have fixed it locally in just one minute, see screenshot below. If you are happy with the result, I can open a PR and fix it.

Image

@shawna-slh
Copy link
Contributor

Thanks, @remibetin ! Go ahead and fix it. No need to wait for Kevin's OK.

@josehilera
Copy link
Collaborator Author

In the screenshot appears Android, Windows, Linux. But the tool is only for iOS.

@remibetin
Copy link
Member

remibetin commented Feb 13, 2025

I picked a random tool for the screenshot, to show how the OS were rendered. Sorry for the confusion.

Here is the preview for the Color Contrast tool.

Screenshot of the "Color Contrast" tool entry. The entry now displays an "Operating system" line with the value "iOS", between the "purpose" and "scope" values

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 a pull request may close this issue.

4 participants