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

ui: fix wrong path color indexing #33350

Merged
merged 4 commits into from
Aug 21, 2024
Merged

ui: fix wrong path color indexing #33350

merged 4 commits into from
Aug 21, 2024

Conversation

sshane
Copy link
Contributor

@sshane sshane commented Aug 21, 2024

Off by one index from the fix in #33026

This caused the path to be black when only a few points were showing in some situations.

Old:

image

New:

image

@github-actions github-actions bot added the ui label Aug 21, 2024
@sshane sshane added the bugfix label Aug 21, 2024
Copy link
Contributor

UI Screenshots

@sshane sshane merged commit d8c9822 into master Aug 21, 2024
20 checks passed
@sshane sshane deleted the fix-ui-path-wrong-pts branch August 21, 2024 03:49
Edison-CBS pushed a commit to Edison-CBS/openpilot that referenced this pull request Sep 15, 2024
* fix

* clean up

* clean up

* clean up
old-commit-hash: d8c9822
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant