Skip to content

tooltip error fix on mobile#210

Merged
GurinderRawala merged 1 commit intocoderabbit_micro_frontendfrom
tooltip_error_fix_on_mobile
Feb 24, 2026
Merged

tooltip error fix on mobile#210
GurinderRawala merged 1 commit intocoderabbit_micro_frontendfrom
tooltip_error_fix_on_mobile

Conversation

@GurinderRawala
Copy link
Copy Markdown

@GurinderRawala GurinderRawala commented Feb 24, 2026

Issue

Summary by CodeRabbit

  • Bug Fixes

    • Improved mobile tooltip event handling for better interaction behavior.
  • Chores

    • Updated build artifacts and optimized asset references.

@GurinderRawala GurinderRawala self-assigned this Feb 24, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 3f287df and 6670f78.

📒 Files selected for processing (2)
  • packages/grafana-ui/src/components/uPlot/plugins/TooltipPlugin2.tsx
  • public/microfrontends/fn_dashboard/index.html

📝 Walkthrough

Walkthrough

Two minor updates: the TooltipPlugin2 component no longer includes the view property when re-dispatching MouseEvents on mobile, and the fn_dashboard HTML updates an asset filename reference from 7223.20b4cb6131bf5aaf15e7.js to 7223.e70d1d928f30305f4693.js.

Changes

Cohort / File(s) Summary
Mobile Event Handling
packages/grafana-ui/src/components/uPlot/plugins/TooltipPlugin2.tsx
Removed view property from MouseEvent construction when re-dispatching on mobile.
Asset Reference Update
public/microfrontends/fn_dashboard/index.html
Updated script asset filename from 7223.20b4cb6131bf5aaf15e7.js to 7223.e70d1d928f30305f4693.js.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A whisker-thin tweak here, a fingerprint there,
Small changes that polish with technical care,
One view removed from events that compile,
One asset refreshed with a brand-new file!
🐰✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch tooltip_error_fix_on_mobile

Comment @coderabbitai help to get the list of available commands and usage tips.

@GurinderRawala GurinderRawala merged commit ce1f1cd into coderabbit_micro_frontend Feb 24, 2026
1 of 2 checks passed
@GurinderRawala GurinderRawala deleted the tooltip_error_fix_on_mobile branch February 24, 2026 18:15
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