[Bug]: Guided review: “Also in this stop” files lack inline comment controls
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 62/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- react, typescript
- Domaine
- frontend
Piste de recherche
Start with GuideExcerpt.tsx, which enables gutter comment controls, line selection and the comment editor for highlighted excerpts. Compare it with GuideFileDiff.tsx, which renders the files under "Also in this stop" without that wiring. Done when hovering a changed line in an expanded additional file shows the + button and submits a comment through the same flow as excerpts.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
When reviewing a pull request using the guide, highlighted code excerpts show a + button when hovering over a line, allowing me to add a review comment. Files listed under “Also in this stop” do not offer this control when expanded.
Steps to reproduce
- Open a pull request’s review guide.
- Navigate to a stop containing highlighted excerpts and additional files under “Also in this stop”.
- Hover over a line in a highlighted excerpt: the + comment button appears.
- Expand one of the additional files and hover over a changed line: no comment button appears.
Expected behavior
Changed lines in expanded “Also in this stop” files should support inline review comments, just like highlighted excerpts.
Actual behavior
The additional files display their diffs, but inline commenting is unavailable.
Environment
- T3 Code desktop:
0.1.10-nightly.20261113.4630 - macOS:
27.2.1(build26B131)
Investigation
GuideExcerpt.tsx enables gutter controls and line selection and connects them to the comment editor. GuideFileDiff.tsx, used for the additional files, does not have that integration.
I found no exact duplicate. Open PR #35324 touches this viewer, but its current changes do not add commenting support there.
I’d be interested in contributing a focused fix that reuses the existing commenting flow, if this limitation is unintended.
- Langage dominant
- TypeScript
- Étoiles
- 24.8k
- Forks
- 6.4k
- Merge moyen
- 59 min
- PR mergées (30 j)
- 418
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de pingdotgg/t3code
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 1 jour
-
via-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: MCP App inline frames never go below 80px, so compact apps get blank space under themOuverte
Difficulté 2/5 Une demi-journée Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Question card options show LaTeX as raw source, though the question text renders itPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: Settings inheritance popover shows raw slugs for branch naming, default permissions and response streamingPeut-être pris @juliusmarminge l’a pris il y a 1 jour. Ouvertevouch:denounced
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de pingdotgg/t3code
Issues similaires
-
skill fork: kickoff message "Run the X skill now" makes the child call the skill tool on itselfOuvertearea: skills bug priority: medium
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
griffinwork40/agent-afk#4874 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
paperclipai/paperclip#21503 ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug] List: the reorder hint's Move up stays enabled on the first row, and Move down on the lastOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
-
Blocklist addition requestOuverteblocklist addition
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
MetaMask/eth-phishing-detect#312343 ·
Les mainteneurs répondent en général sous 1 jour
-
packages:player
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour