GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||
GeezerVison is showing github.com — Fix tick prefixes and suffixes with explicit tick values by davrens101 · Pull Request #8160 · plotly/plotly.js Conversation
|
| Source, with the new regression | Passed | Failed | Total |
|---|---|---|---|
| Pristine upstream source | 398 | 5 | 403 |
| Patched source | 399 | 4 | 403 |
The new regression fails before the fix and passes after the fix. Four pre-existing failures remain in each run:
should make room for the inside labels of the counter axescan relayout insiderangeshould rotate labels to avoid overlapsinsiderange react to new dataThe numerical errors match across those runs. The suite is not fully green.
The local evidence runner excludes unrelated MapLibre CSS. The installed esbuild-plugin-inline-css throws at path.resolve(undefined, args.path). No upstream configuration or dependency changes form part of this patch.
These results come from the preparation logs, not a new submission-time run. Lint, typecheck, syntax, schema-drift, and full CI results are not available. No schema, public API, dependency, mock, or generated type changes occur. No local image baselines were generated. Maintainer CI must confirm the standard checks and any affected baselines.
The source-built reproduction uses tickvals: [0, 1, 2], tickprefix: '$', and ticksuffix: ' USD':
Before: 0, 1, 2
After: $0 USD, $1 USD, $2 USD
The submission agent inspected the saved before/after screenshots. Screenshots and complete logs remain in the local evidence package. This CLI submission does not attach images or claim public screenshot URLs. The issue contains the original visual report.
AI assisted the patch, regression test, review, and submission. The user approved submission: "go ahead and submit it looks good to me". That approval is not a claim of independent maintainer review. 🐦
Commit c8ecfd27a4e52229b17d598ad18243189a2f221e replaces the placeholder with draftlogs/8160_fix.md and the actual PR link.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @davrens101 ! It's working well on my end.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
[BUG] [REGRESSION]:tickprefix/ticksuffix ignored when tickvals provided
There was an error while loading. Please reload this page.