Add feedback button to sidebar bottom #32
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a "Give Feedback" button to the task sidebar that opens a GitHub issues page for users to provide feedback.
- Imports the
MessageSquareicon from lucide-react - Adds a
handleFeedbackcallback that opens GitHub issues in a new tab - Renders a "Give Feedback" button at the bottom of the sidebar
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| window.open( | ||
| 'https://github.com/scaleapi/scale-agentex/issues/new', | ||
| '_blank', | ||
| 'noopener,noreferrer' | ||
| ); |
There was a problem hiding this comment.
The third parameter of window.open is for window features (like width, height), not security attributes. To properly apply noopener and noreferrer security attributes with window.open, you need to manually set the opener property to null on the returned window reference. Consider using: const newWindow = window.open('...', '_blank'); if (newWindow) newWindow.opener = null; Alternatively, consider using an anchor tag with rel=\"noopener noreferrer\" which is more straightforward and consistent with other usage in the codebase (see investigate-traces-button.tsx and json-viewer.tsx).
| window.open( | |
| 'https://github.com/scaleapi/scale-agentex/issues/new', | |
| '_blank', | |
| 'noopener,noreferrer' | |
| ); | |
| const newWindow = window.open( | |
| 'https://github.com/scaleapi/scale-agentex/issues/new', | |
| '_blank' | |
| ); | |
| if (newWindow) newWindow.opener = null; |
Adds a stickyt feedback button to the bottom of the task sidebar:
redirects to the new issue page for this repository