Skip to content

Rm support label sentence fluff #129

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

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

Conversation

ugtthis
Copy link
Contributor

@ugtthis ugtthis commented Jul 31, 2025

  • users are in the context of the compatibility page already
  • having "openpilot requires the car to come equipped with" is verbose and doesn't add much value
  • easier to scan
  • having just the package also prevents cutting off package names, like the below
  • I kept openpilot will work with <b>all packages and trims</b> of this car. since it makes clearer that "all" trims and package work vs just a certain one

Go to this page to see changes -> https://comma-web--pr129-b2yqnfls.web.app/vehicles

Related:
PR - commaai/opendbc#2592
incognitojam's comment here

Before After
Before After

Copy link
Contributor

deployed preview: https://comma-web--pr129-b2yqnfls.web.app

Welcome! Make sure to:

  • read the contributing guidelines
  • mark your PR as a draft until it's ready to review
  • post the preview on Discord; feedback from users will speedup the PR review

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.

1 participant