Conversation
dltacube
marked this pull request as ready for review
June 12, 2026 13:22
Contributor
|
I like this change, when I was testing my web ui PR I never tested more than a handful of stores. I was more concerned about the functionality of changing stores while the program is running. |
Owner
|
thanks for the input @Mr-Tech-13 ! |
jef
approved these changes
Jun 30, 2026
jef
enabled auto-merge (squash)
June 30, 2026 21:21
Author
|
@jef not sure what to do about the lint pull request title workflow. Can that be overlooked? |
Author
|
@jef the title is conventional now. Could you approve/rerun the Pull Request Linter? The latest run is stuck as |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Reworks the dashboard matrix from a single fixed table into responsive per-series cards with a store grid. Store cells now wrap within the viewport instead of compressing into one row or disappearing off the right edge when many stores are selected.
Why
With a large
STORESselection, the current table layout puts every store in one table row. In the screenshot harness, the rendered table was 4019 px wide inside a 998 px matrix area, making the dashboard hard to scan.The new layout keeps each series in a bounded panel and lets store cells wrap within the available width.
Scope
Visual-only web dashboard change. The PR touches only
web/index.htmland does not change store polling, notification behavior, checkout behavior, or the matrix API contract.Validation
git diff --check/api/matrixresponse.Screenshot artifacts captured locally:
/tmp/streetmerchant-webui-pr-screenshots/before-dashboard.png/tmp/streetmerchant-webui-pr-screenshots/after-dashboard.png/tmp/streetmerchant-webui-pr-screenshots/layout-metrics.jsonMeasured at a 1024x768 viewport:
Screenshots are intentionally kept out of the commit so the PR remains a visual-only source change.