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

Fix bugs with the context link text for localdocs #2218

Merged
merged 2 commits into from
Apr 15, 2024
Merged

Conversation

manyoso
Copy link
Collaborator

@manyoso manyoso commented Apr 15, 2024

This fixes two problems with current context links:

  1. When you have a long conversation and each response has context links, scrolling to the bottom of the conversation can cause the context links to no longer work for the responses at the top.

  2. The context links do not work across application loads.

This patch fixes both these issues by implementing the QTextObjectInterface which allows for insertion of custom objects in the QTextDocument.

...

The second patch makes the setting for 'showReferences' simply a cosmetic toggle for the view. This way we always capture the references across restarts.

… links

persistently work across application loads and fix scrolling bug with context
links.

Signed-off-by: Adam Treat <treat.adam@gmail.com>
@manyoso manyoso requested a review from cebtenzzre April 15, 2024 15:23
Signed-off-by: Adam Treat <treat.adam@gmail.com>
@manyoso manyoso merged commit 94a9943 into main Apr 15, 2024
6 of 10 checks passed
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.

2 participants