Skip to content

fix: Generate preview with proper aspect ratio for templates - #4386

Merged
juliusknorr merged 1 commit into
mainfrom
fix/template-preview-aspect-ratio
Jan 24, 2025
Merged

fix: Generate preview with proper aspect ratio for templates#4386
juliusknorr merged 1 commit into
mainfrom
fix/template-preview-aspect-ratio

Conversation

@juliusknorr

@juliusknorr juliusknorr commented Jan 14, 2025

Copy link
Copy Markdown
Member

Steps to reproduce:

  • Select a user template folder in richdocuments settings
  • Add a template
  • Open the template picker

Before this change we injected a square preview that looked odd in the tempalte picker with others coming from the server template directory having the aspect ratio of the document.

@juliusknorr

Copy link
Copy Markdown
Member Author

Not sure why psalm just fails now, but it is due to wrong phpdoc in server: nextcloud/server#50196

@juliusknorr
juliusknorr enabled auto-merge January 17, 2025 10:03
@elzody

elzody commented Jan 22, 2025

Copy link
Copy Markdown
Collaborator

looks like there is a merge conflict on composer.lock, but correct me if i am wrong we can just disregard it and push the new one instead, it's probably because of the dependency update or something?

@juliusknorr

Copy link
Copy Markdown
Member Author

Yes, I manually bumped the package, but was done separately on main meanwhile

Signed-off-by: Julius Knorr <jus@bitgrid.net>
@juliusknorr
juliusknorr force-pushed the fix/template-preview-aspect-ratio branch from cb1c8f0 to 920cf96 Compare January 24, 2025 13:58
@juliusknorr

Copy link
Copy Markdown
Member Author

Rebased

@juliusknorr
juliusknorr merged commit 174a203 into main Jan 24, 2025
@juliusknorr
juliusknorr deleted the fix/template-preview-aspect-ratio branch January 24, 2025 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants