Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add navigation to modified lines in prompt comparison #723

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

poroyo
Copy link
Contributor

@poroyo poroyo commented Jan 16, 2025

PR Checklist

  • Have you checked if it works normally in all models? Ignore this if it doesn't use models.
  • Have you checked if it works normally in all web, local, and node hosted versions? If it doesn't, have you blocked it in those versions?
  • Have you added type definitions?

Description

This PR adds a small improvement to the prompt comparison functionality.

When the two prompts are different, the tooltip overview now enables clicking on modified parts to navigate directly to the corresponding lines.

Previously, finding the modified parts required manual scrolling. With this update, it becomes much easier to locate and review the changes, even for very long prompts.

When navigating to a modified part, the target line is highlighted in yellow for 1 second to indicate the clicked location.

output.mov

For mobile devices, this functionality has been tested on iOS 16.7.

If there are any suggestions for improvement or concerns about the implementation, feedback is greatly appreciated. It’s completely understandable if this PR needs to be rejected or revised further.

Thank you for taking the time to review this!

@poroyo poroyo changed the title Add navigation from prompt comparison tooltip to modified lines feat: add navigation to modified lines in prompt comparison Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant