Allow selecting text in the query Messages output#74
Conversation
The app sets user-select: none on body for a desktop feel, and only inputs/ textarea/pre opt back in, so the Messages tab's error/print text couldn't be selected or copied. Add `select-text` to the Messages container to re-enable selection for that subtree. The results grid keeps its own cell-selection and copy behaviour. Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: GitButler <gitbutler@gitbutler.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds the select-text class to the messages container in QueryResultsTable.tsx to allow users to select and copy message/error text. The reviewer pointed out that using a single-line comment (//) inside parenthesized JSX expressions is risky and suggested replacing it with a standard JSX comment ({/* ... */}) inside the div to avoid potential syntax errors during future refactoring.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
There was a problem hiding this comment.
Pull request overview
This PR restores standard text selection/copy behavior specifically for the Query “Messages” output area, while keeping the rest of the app’s desktop-like user-select: none behavior intact.
Changes:
- Added Tailwind’s
select-textutility to the Messages container so message/error text can be selected and copied. - Added an inline explanatory comment near the Messages container.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Address review feedback: the explanatory comment was a // line comment in JSX
expression position, which a formatter could merge into the element and comment
it out. Fold it into the existing {/* Messages */} JSX comment, matching the
file's comment style.
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: GitButler <gitbutler@gitbutler.com>
Summary
user-select: noneonbody(desktop chrome feel) and onlyinput/textarea/preopt back in, so the Messages tab's error/print text couldn't be selected or copied.select-textto the Messages container so its subtree re-enables text selection. The results grid keeps its own cell-selection and copy behaviour.Test plan
npx tsc --noEmitcleannpm run buildclean