Skip to content

Conversation

@valentinedwv
Copy link

Update documentation to include details on functionality (per claude). Add a query-analysis.md that can be the start of optimizing queries
Update documentation to include details on functionality (per claude). Add a query-analysis.md that can be the start of optimizing queries
Update documentation to include details on functionality (per claude). Add a query-analysis.md that can be the start of optimizing queries
@valentinedwv
Copy link
Author

Screen.Recording.2025-08-20.at.10.53.13.AM.mov

Copy link
Contributor

@hannahbast hannahbast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@valentinedwv Thanks a lot for this. Could you please remove db/qleverui.sqlite3 from this PR (you probably committed it accidentally by committing all your local changes). For example, you can do that as follows when in your branch:

git checkout origin/master -- db/qleverui.sqlite3
git add db/qleverui.sqlite3
git commit -m "Restore db/qleverui.sqlite3 to match master"
git push

@hannahbast
Copy link
Contributor

@valentinedwv I will take a closer look. For now, it's already live on https://qlever.cs.uni-freiburg.de to play around with

@hannahbast
Copy link
Contributor

@valentinedwv The pan&zoom feature is great and very useful. Two suggestions:

  1. Right now, when opening the "Analysis" window for a longer query, most of the window will be filled by the query and you first have to scroll down, for which you have to move your mouse to the scrollbar on the right. This takes some time to find out. I would suggest to have the window scrolled down when it is first opened. If one really wants to see the query, one can scroll up. But typically, one wants to see the tree. And it's also typical that one wants to see the bottom of the tree first.

  2. Right now there is a text at the top right (new) together with an x to close the dialog. But there is also still the Close button on the bottom right. I would remove the x and stick to the Close button that is already there. And the font (style and size) of the new text should be more in line with what's already on the page.

I will also have a look at the new descriptions (in the .md files)

remove the drag to pan zoom close button.
add scrollbar to the query text,
remove the drag to pan zoom close button.
add scrollbar to the query text,
remove the drag to pan zoom close button.
add scrollbar to the query text,
remove the drag to pan zoom close button.
add scrollbar to the query text,
@valentinedwv
Copy link
Author

Think this mostly covers the changes. Can't always get the graphic to center correctly, but better than the previous scrollbar version.

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