-
-
Notifications
You must be signed in to change notification settings - Fork 457
Preview Feature #757
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
Preview Feature #757
Conversation
- Add QuickView Area
- add binding preview area - remove test code in resultlistbox.xaml.cs
- add binding preview area - remove test code in resultlistbox.xaml.cs
|
Remove comment
|
Do we want more description in the Preview feature? |
I don't need it anymore. how about you? |
|
@onesounds is this ready to go? if so could we resolve conflict please |
Not yet. This pr is a rough version. Additional tasks such as preview and shortcut key modification are required. |
|
i have marked it as draft |
i like this idea, it would be nice if this space offers most used results, but i dont think we can achieve this in this pr because the current model does not persist results and loading results correctly with action context is also something that needs working out. But yeah great idea. |
|
hey @onesounds was there a discussion on to put the preview on top or to the side of the results? what's your opinion on the positioning of the preview window? |
There were no other opinions. CitizenDee said He wish he could always turn it on. It is recommended that the default position be placed on the right. (like spotlight on mac) However, in that case, various problems such as window size and vertical size arise. Since I can't think of a neat way, I'll think more about this pr. We will work again after the problem of reducing the lnk path path and other design prs are organized. |

WIP
Working on it. Pr for the test.