Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
Accelerator
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Type
/
to search
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
Uh oh!
There was an error while loading.
Please reload this page
.
diffplug
/
gitcasso
Public
Notifications
You must be signed in to change notification settings
Fork
0
Star
57
Code
Issues
22
Pull requests
1
Actions
Projects
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Actions
Projects
Security and quality
Insights
Appending comments
- #91
#91
Merged
nedtwigg
merged 8 commits into
main
diffplug/gitcasso:main
from
fix/add-comment-bug
diffplug/gitcasso:fix/add-comment-bug
Copy head branch name to clipboard
Sep 26, 2025
Conversation
Commits
8
(8)
Checks
Files changed
Merged
Appending comments
#91
nedtwigg
merged 8 commits into
main
diffplug/gitcasso:main
from
fix/add-comment-bug
diffplug/gitcasso:fix/add-comment-bug
Copy head branch name to clipboard
Commits
Commits on Sep 25, 2025
Re-enhance when submitted.
WebsByTodd
committed
cf3ada0
View commit details
Copy full SHA for cf3ada0
Browse repository at this point
Commits on Sep 26, 2025
Merge branch 'main' into fix/add-comment-bug
nedtwigg
committed
08a4cf3
View commit details
Copy full SHA for 08a4cf3
Browse repository at this point
move submit handler to be after deletion
nedtwigg
committed
d78e3a8
View commit details
Copy full SHA for d78e3a8
Browse repository at this point
give enhancers a way to return an optional cleanup function
nedtwigg
committed
9e46e00
View commit details
Copy full SHA for 9e46e00
Browse repository at this point
Abandon the `registerSubmitHandler`, and rely only on reliable cleanup.
nedtwigg
committed
f0df91f
View commit details
Copy full SHA for f0df91f
Browse repository at this point
So for PR append, a single text box persists for each comment, it just gets cleared after submission. Listeningfor `input` doesn't work, but listening with `MutationObserver` does.
nedtwigg
committed
603b314
View commit details
Copy full SHA for 603b314
Browse repository at this point
So for PR enhancer, we can get by with only a MutationObserver.
nedtwigg
committed
c20cbd3
View commit details
Copy full SHA for c20cbd3
Browse repository at this point
Same result with less code by doing cleanup on *every* overtype that gets destroyed (probably a good idea).
nedtwigg
committed
2655e2b
View commit details
Copy full SHA for 2655e2b
Browse repository at this point
You can’t perform that action at this time.