Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Greptile Summary
This PR simplifies the UI in the CitedSourcesToggle component by removing the confusing "+N remaining" count that displayed how many additional sources existed beyond the first 2 unique icons shown. The component previously had complex logic to calculate and display this count based on the number of citations and document map size, with conditional logic that could be misleading to users.
The change removes lines 124-126 that rendered this count indicator while preserving the core functionality of displaying up to 2 unique source icons followed by the "Sources" label. This aligns with the existing logic that already limits the display to 2 unique icons (controlled by the break condition when uniqueIcons.length >= 2). The component integrates with the broader chat interface where users can click to expand and see all available sources, making the specific count less critical for the user experience.
Confidence score: 5/5
- This PR is safe to merge with minimal risk as it only removes UI elements without affecting core functionality
- Score reflects the simple nature of removing confusing display logic while maintaining all underlying source handling capabilities
- No files require special attention as this is a straightforward UI simplification
1 file reviewed, no comments
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
1 participant
Description
[Provide a brief description of the changes in this PR]
How Has This Been Tested?
[Describe the tests you ran to verify your changes]
Backporting (check the box to trigger backport action)
Note: You have to check that the action passes, otherwise resolve the conflicts manually and tag the patches.
Summary by cubic
Removed the "+N" remaining sources count from the cited sources toggle to avoid confusion and visual clutter. The toggle now only shows unique source icons alongside the "Sources" label.