Skip to content

Conversation

@crhallberg
Copy link
Contributor

@crhallberg crhallberg commented May 14, 2025

Replaced by #1418.


Having the half-shaded and textured background behind the adjust image overlay makes those adjustments a guessing game. Moving the style adjustment down fixes the proper order of operations to remove the background.

Also:

  • add a black border to increase the visual contrast of the overlay.
  • move the overlay to the top left to hopefully hide less of the image.

@crhallberg crhallberg requested a review from demiankatz May 14, 2025 13:51
@vercel
Copy link

vercel bot commented May 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
universalviewer ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 1:16pm

demiankatz
demiankatz previously approved these changes May 14, 2025
Copy link
Contributor

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

Thanks, @crhallberg, I think this is a big improvement!

Should we rebase this against the release-4.2.0 branch so we can include it in the next release, or would people feel better keeping this against dev for inclusion in 4.3 instead?

@crhallberg crhallberg reopened this May 15, 2025
@crhallberg crhallberg closed this May 15, 2025
@crhallberg crhallberg changed the title Fix adjust image background (old) fix adjust image background May 15, 2025
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.

2 participants