Skip to content

Add selectable key notation with persistence, introduce conversion/format unit tests, and run tests in PR/main CI - #5

Merged
samsta merged 5 commits into
mainfrom
copilot/add-key-representation-options
May 21, 2026
Merged

samsta merged 5 commits into
mainfrom
copilot/add-key-representation-options

Conversation

Copilot AI commented May 21, 2026 •

Copy link
Copy Markdown
Contributor
  • Reproduce rotated heatmap label spacing/cropping issue
  • Apply minimal chart layout adjustment to remove excess gap and prevent bottom cropping
  • Run targeted validation for heatmap behavior
  • Capture updated UI screenshot
  • Reply to the new review comment with commit hash and screenshot
  • Run final lint/build/test and validation

@samsta
samsta marked this pull request as ready for review May 21, 2026 07:55
Comment thread src/App.tsx
Comment thread src/App.tsx
Comment thread src/App.tsx
Comment on lines 620 to 611

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot Just noticed in passing that this list doesn't fill the article vertically if it's long, it only uses about half of the vertical space within the article

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adjusted in commit 8e52647: the sparse/missing regions list now grows to fill the insight panel’s available vertical space (with scrolling), instead of being capped to about half height.

Screenshot: /tmp/keybpmmap-sparse-list-fill.png.

Copilot finished work on behalf of samsta May 21, 2026 08:13
Comment on lines +87 to +99
const label = formatKeyLabel(camelotKey)
const rotateLabel = label.length > 3
const labelY = height - margin.bottom + 28
return (
<text
key={camelotKey}
x={x}
y={height - margin.bottom + 28}
textAnchor="middle"
y={labelY}
textAnchor={rotateLabel ? 'start' : 'middle'}
className="chart-label"
transform={rotateLabel ? `rotate(90 ${x} ${labelY})` : undefined}
>
{formatKeyLabel(camelotKey)}
{label}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot There's a large gap between the rotated labels and the bottom of the heatmap and some of the labels are cropped at the bottom of the container because they are too far down

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adjusted in commit 6db0ed5: rotated heatmap key labels now anchor upward and use a tighter y-offset, which removes the extra gap and prevents bottom clipping.

Screenshot: /tmp/keybpmmap-heatmap-rotated-labels-spacing.png.

Copilot AI requested a review from samsta May 21, 2026 08:24
Copilot finished work on behalf of samsta May 21, 2026 08:24
@samsta
samsta merged commit 70e176e into main May 21, 2026
2 of 3 checks passed
@samsta
samsta deleted the copilot/add-key-representation-options branch May 27, 2026 07:30
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.

2 participants