Skip to content

feat(Scrubber): add labels to beacons when scrubbing - #933

Merged
aquelemiguel merged 22 commits into
mainfrom
dls-1104
Oct 28, 2026
Merged

aquelemiguel merged 22 commits into
mainfrom
dls-1104

Conversation

@aquelemiguel

@aquelemiguel aquelemiguel commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

@aquelemiguel aquelemiguel self-assigned this Oct 22, 2026
@vercel

vercel Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ldls Ready Ready Preview Oct 27, 2026 4:45pm UTC
ldls-react-native Ready Ready Preview Oct 27, 2026 4:45pm UTC

Request Review

@aquelemiguel
aquelemiguel marked this pull request as ready for review October 22, 2026 14:57
@aquelemiguel
aquelemiguel requested a review from a team as a code owner October 22, 2026 14:57
Copilot AI balanced review requested due to automatic review settings October 22, 2026 14:57

This comment was marked as resolved.

@gamegee gamegee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just noticed after few tests, that each label position is computed on its own.
So based on the content it can either appears on the right / left

To me its not blocking, but just to make sure we are aware of this

Image

@aquelemiguel

Copy link
Copy Markdown
Contributor Author

[low/nit]: I would suggest to use regular smooth line instead of square one for the example of Beacons but also of Pulse.

yeah agreed, i dislike the square chart in general, here specifically it doesn't work

@sonarqubecloud

Copy link
Copy Markdown

@aquelemiguel
aquelemiguel merged commit 469b43b into main Oct 28, 2026
24 checks passed
@aquelemiguel
aquelemiguel deleted the dls-1104 branch October 28, 2026 12:43

This branch was successfully deployed

2 active deployments
Preview – ldls — d7e749db Deployed Oct 27, 2026 by vercel[bot]
Preview – ldls-react-native — d7e749db Deployed Oct 27, 2026 by vercel[bot]
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.

3 participants