Skip to content

fix: small tooltip bug in spreadsheet#2821

Merged
HarshMN2345 merged 4 commits intomainfrom
fix-small-tooltip-bug-spreadsheet
Feb 2, 2026
Merged

fix: small tooltip bug in spreadsheet#2821
HarshMN2345 merged 4 commits intomainfrom
fix-small-tooltip-bug-spreadsheet

Conversation

@HarshMN2345
Copy link
Member

@HarshMN2345 HarshMN2345 commented Feb 2, 2026

What does this PR do?

before

image

after

image

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • Style
    • Widened the key column in the database table interface for clearer display (increased minimum width).
    • Improved column-size tooltips: Min and Max values now appear on separate lines, tooltips are positioned at the top, and their width/whitespace handling was adjusted for better readability and to prevent overflow.

@appwrite
Copy link

appwrite bot commented Feb 2, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Preview deployments create instant URLs for every branch and commit

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

Walkthrough

This pull request updates the columns page template for the database UI: key column width and minimumWidth are increased from 300 to 380. The min/max tooltip content now places Min and Max on separate lines (newline inserted) and is rendered with white-space: pre-line. Tooltip presentation is adjusted to maxWidth="20rem" and positioned at the top.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: small tooltip bug in spreadsheet' directly describes the main change: improving tooltip presentation in a spreadsheet-like interface by adjusting how min/max values are displayed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-small-tooltip-bug-spreadsheet

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@HarshMN2345 HarshMN2345 merged commit 140bd83 into main Feb 2, 2026
3 of 4 checks passed
@HarshMN2345 HarshMN2345 deleted the fix-small-tooltip-bug-spreadsheet branch February 2, 2026 08:34
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