Skip to content

Conversation

mashed5894
Copy link
Contributor

Summary

Implemented krita (.kra/.krz) and open raster (.ora) thumbnails per #977 and #979.

Tasks Completed

  • Platforms Tested:
    • Windows x86
    • Windows ARM
    • macOS x86
    • macOS ARM
    • Linux x86
    • Linux ARM
  • Tested For:
    • Basic functionality
    • PyInstaller executable

@CyanVoxel CyanVoxel added Type: Enhancement New feature or request Type: UI/UX User interface and/or user experience labels Jul 9, 2025
@CyanVoxel CyanVoxel moved this to 👀 In review in TagStudio Development Jul 9, 2025
Copy link
Member

@CyanVoxel CyanVoxel left a comment

Choose a reason for hiding this comment

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

Thank you for your work on this! From the example files given in #977 I believe everything is working as intended here

@CyanVoxel CyanVoxel added the Status: Mergeable The code is ready to be merged label Jul 9, 2025
@CyanVoxel CyanVoxel moved this from 👀 In review to 🍃 Pending Merge in TagStudio Development Jul 9, 2025
@Pheubel
Copy link
Contributor

Pheubel commented Jul 11, 2025

I tried this PR on Windows, The krita files seem to work as expected, but i don't see a thumbnail for the open raster file that i exported with krita.

image

i wonder if the export is different in a way, i am personally not familiar with the format.

@mashed5894
Copy link
Contributor Author

mashed5894 commented Jul 12, 2025

i wonder if the export is different in a way, i am personally not familiar with the format.

It follows the OpenDocument standard according to the documentation (https://www.openraster.org/baseline/file-layout-spec.html), so I simply added the extension to the list. You should be able to unzip your .ora file like any normal zip file and take a look at what the issue could be.

@CyanVoxel CyanVoxel added this to the Alpha v9.5.3 milestone Aug 1, 2025
@CyanVoxel CyanVoxel merged commit fd7d1f1 into TagStudioDev:main Aug 1, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from 🍃 Pending Merge to ✅ Done in TagStudio Development Aug 1, 2025
@CyanVoxel CyanVoxel removed the Status: Mergeable The code is ready to be merged label Aug 1, 2025
@mashed5894 mashed5894 deleted the kra_thumbs branch September 14, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement New feature or request Type: UI/UX User interface and/or user experience
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants