-
Notifications
You must be signed in to change notification settings - Fork 592
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(mac): ✨ use quick look for image popups #151
base: master
Are you sure you want to change the base?
Conversation
ab0ed44
to
cfcb991
Compare
Huge win. A major next step would be making the images focusable, so users can Tab between them and press Space to trigger Quick Look views. |
Thanks for the PR. I have plans to build it for windows and other OS as well. Can we only use quick look when its macOS |
This is exactly what I did, I check the platform and use quick look only on mac, all others use the old popup window.
|
@@ -3,7 +3,7 @@ | |||
<div class="animatable_content_box "> | |||
|
|||
<div v-if="Object.values(app_state.history).length > 0"> | |||
<div v-for="history_box in Object.values(app_state.history).reverse()" :key="history_box.key" style="clear: both;"> | |||
<div v-for="history_box in Object.values(app_state.history)" :key="history_box.key" style="clear: both;"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why remove the .reverse ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I must have made a mistake while rebasing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW why history is stored as an object with random keys instead of array?
cfcb991
to
27a667e
Compare
Fixes #173 |
Use system quick look to preview images instead of custom popup