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

require-baseline: Property 'cursor' is not a widely available baseline feature #51

Closed
nzakas opened this issue Feb 17, 2025 · 4 comments · Fixed by #53
Closed

require-baseline: Property 'cursor' is not a widely available baseline feature #51

nzakas opened this issue Feb 17, 2025 · 4 comments · Fixed by #53
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion bug Something isn't working

Comments

@nzakas
Copy link
Member

nzakas commented Feb 17, 2025

It appears that the cursor property is not considered baseline even though it has been around for a while:
https://github.com/web-platform-dx/web-features/blob/main/features/cursor.yml#L9

Options

We have a couple of ways of addressing this:

  1. We could manually override the settings for cursor. We'd then need to make this determination on a case-by-case basis for any other similar properties.
  2. We could add an option to the rule that lets people specifically allow specific properties. This would put the responsibility on users to decide how strictly they want to follow baseline.

Discussed in #50

Originally posted by sysmat February 17, 2025
From where is this rule, and why?

@nzakas nzakas added accepted There is consensus among the team that this change meets the criteria for inclusion bug Something isn't working labels Feb 17, 2025
@nzakas nzakas added this to Triage Feb 17, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Feb 17, 2025
@nzakas nzakas moved this from Needs Triage to Feedback Needed in Triage Feb 17, 2025
@nzakas
Copy link
Member Author

nzakas commented Feb 17, 2025

Ping @eslint/eslint-tsc

@mdjermanovic
Copy link
Member

I'm fine with implementing both solutions at the same time. We could override the data ourselves when it seems incorrect and also add an option to let users specify properties they'd like to allow regardless of our data.

@fasttime
Copy link
Member

It seems that cursor is not supported in Safari for iOS, and that explains why it's not included in the baseline. This isn't really surprising because iOS is designed to run on mobile devices that are used without a mouse and have little need for a mouse pointer. It's possible that Safari for iOS intentionally decided not to adopt the cursor standard for usability purposes, in which case that feature won't be added to the baseline anytime soon.

I think it's fine to override the baseline defaults in this case. Providing an additional option to selectively override certain baseline features could be also useful.

@nzakas
Copy link
Member Author

nzakas commented Feb 18, 2025

It looks like they're aware of the problem and are working towards a solution:
web-platform-dx/web-features#1038

In the near-term, I'll just manually update cursor. We can leave this open to determine if we need to add a new option to the rule as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion bug Something isn't working
Projects
Status: Complete
Development

Successfully merging a pull request may close this issue.

3 participants