Skip to content

Improve tagger preview image formatting - #7195

Draft
smith113-p wants to merge 3 commits into
stashapp:developfrom
smith113-p:codex/tagger-image-formatting
Draft

smith113-p wants to merge 3 commits into
stashapp:developfrom
smith113-p:codex/tagger-image-formatting

Conversation

@smith113-p

@smith113-p smith113-p commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Description

The primary change here is to render portrait cover images as portrait in tagger. Because Stash already renders the scraped image in portrait, this change makes it easier/possible to tell whether the image is the same or not.

Also makes minor formatting changes to improve how this looks particularly with the duration overlay:

  • Standardizes preview image/video container dimensions (14rem x 10rem) with a subtle dark background.
  • Formats and aligns the overlay duration badge cleanly on top of both preview cards and scraped scene images.

Includes the changes from #7193; I will rebase once that PR lands.

Related Issue

Fixes #7202

Testing

Manually checked how vertical and horizontal cover images look in tagger.

Screenshots

After:
Screenshot 2026-09-02 174933

Checklist

  • I have read and understood the Contributing document.
  • I have read and understood the AI Usage Policy document.
  • I have made corresponding changes to the documentation (if applicable).

AI Usage Disclosure

  • I have used AI tools to assist with this pull request, and I have disclosed the tools and how I used them below.

I used AI tools (Codex aka ChatGPT and Antigravity) to draft and refine the CSS formatting changes.

Additional Context

@smith113-p
smith113-p force-pushed the codex/tagger-image-formatting branch from d5db995 to 5a3ec14 Compare September 2, 2026 20:56
@smith113-p
smith113-p force-pushed the codex/tagger-image-formatting branch from 5a3ec14 to e4c09a2 Compare September 3, 2026 18:22
@DogmaDragon

Copy link
Copy Markdown
Collaborator

This PR used https://github.com/smith113-p/stash/tree/link branch as base, not develop.

@smith113-p

Copy link
Copy Markdown
Contributor Author

Yes -- I was going to keep this as a draft until that PR lands. Also the PR description/title is not final.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tagger should not cut off vertical cover images (for the local scene)

2 participants