Skip to content

fix: improve foreign stock chart readability#445

Open
AlexTzib wants to merge 1 commit into
Manuito83:developfrom
AlexTzib:fix/foreign-stock-chart-readability
Open

fix: improve foreign stock chart readability#445
AlexTzib wants to merge 1 commit into
Manuito83:developfrom
AlexTzib:fix/foreign-stock-chart-readability

Conversation

@AlexTzib
Copy link
Copy Markdown
Contributor

@AlexTzib AlexTzib commented May 5, 2026

This is a small follow-up to make the foreign stock history graph easier to read without changing the rest of the foreign stock flow.

What this changes:

  • plots the chart using real timestamp spacing instead of treating every sample as the same distance apart
  • adds subtle vertical markers when stock appears after being empty, so restock moments are easier to spot
  • reduces the bottom axis labels and keeps them horizontal, which should make the chart less crowded on smaller screens
  • adds a short chart context label so it is clear whether the chart is showing the full recent history or a shorter high-activity window

Why:
The old graph could be a bit misleading because a 3 minute gap and a 30 minute gap were drawn the same way. That made the history harder to reason about, especially around restocks. This keeps the existing chart and data, but presents the time axis closer to what actually happened.

Local verification:

  • Checked the changed chart APIs against the pinned fl_chart 1.0.0 package in the local pub cache.
  • Ran git diff --check.

I could not run Flutter analyzer/build locally because flutter/dart are not available on this PATH, so I expect GitHub CI to be the real compile check here.

@AlexTzib AlexTzib force-pushed the fix/foreign-stock-chart-readability branch from c9cf02b to 653ecfd Compare May 5, 2026 09:29
@AlexTzib AlexTzib force-pushed the fix/foreign-stock-chart-readability branch from 653ecfd to f4d7ac7 Compare May 5, 2026 09:32
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