blender
/
blender
137
404
Fork 586
Code Issues 6.2k Pull Requests 761 Projects 19 Wiki Activity

macOS/QuickLook: support rich thumbnail #107072

Open
Ankit Meel wants to merge 10 commits from ankitm/blender:ankitm/2ql into main
pull from: ankitm/blender:ankitm/2ql
merge into: blender:main

When changing the target branch, be careful to rebase the branch in your fork to match. See documentation.
Conversation 17 Commits 10 Files Changed 6 -3
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9ecdf8a3f7 - Show all commits

3
release/darwin/Blender.app/Contents/Plugins/blender-thumbnailer.appex/Contents/Info.plist
View File

@ -45,9 +45,6 @@
<key>NSExtensionPrincipalClass</key>
<!-- Must be the same as the class implementing the reply method. -->
<string>ThumbnailProvider</string>
<!-- Shows checkbox in System Preferences. -->
<key>com.apple.showsInExtensionsManager</key>
<true/>
</dict>
Brecht Van Lommel commented 2023-04-19 12:48:47 +02:00
Outdated
Review
Copy Link

Is this required, and common for these kinds of thumbnails generators? I don't see it for any other apps, but maybe I just happen to not have any apps that do this.

Is this required, and common for these kinds of thumbnails generators? I don't see it for any other apps, but maybe I just happen to not have any apps that do this.
Ankit Meel commented 2023-04-19 13:25:26 +02:00
Outdated
Review
Copy Link

Not required and not sure about commonness. But since there's no other way to disable it, I added it.

I could remove the key from plist and let the system decide.

Not required and not sure about commonness. But since there's no other way to disable it, I added it. I could remove the key from plist and let the system decide.
Brecht Van Lommel commented 2023-04-20 17:39:10 +02:00
Outdated
Review
Copy Link

I'd leave it out and let the system decide.

I'd leave it out and let the system decide.
</dict>
</plist>