Skip to content

Conversation

analytically
Copy link
Contributor

@analytically analytically commented Mar 14, 2025

Add support for configurable filter setting for pipeline background images. This enhances the visual customization options for pipelines by allowing users to set:

  • Custom filter effects (defaults to grayscale(100%) opacity(30%) as before)

Release Note

  • Add background_filter option for pipeline background images which takes in string of CSS filters. Defaults to the current filters opacity(30%) grayscale(100%)

@analytically analytically requested a review from a team as a code owner March 14, 2025 10:14
@taylorsilva
Copy link
Member

Copy link
Member

@taylorsilva taylorsilva left a comment

Choose a reason for hiding this comment

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

Very nice! I think we can actually remove the background_opacity field because background_filter can be used to combine multiple filters like opacity(): https://developer.mozilla.org/en-US/docs/Web/CSS/filter

I made this filter:

background_filter: blur(5px)grayscale(100%)

and got:

image

Adding opacity()

background_filter: blur(5px)grayscale(100%)opacity(50%)
image

background_filter is all we really need :)

@analytically
Copy link
Contributor Author

Currently opacity is set to 30%. If we'd just allow the filter to be set, what do we do with the default value? Not set it? People might expect it to be kept at 30% when just setting the filter?

@analytically
Copy link
Contributor Author

Having both opacity and filter is more flexible IMO. You can just set the image and opacity without thinking about filter. Or just image and filter, then an opacity of 30% is applied by default.

@analytically
Copy link
Contributor Author

analytically commented Mar 17, 2025

Or, we move the default for opacity into the filter.

Add support for configurable filter setting for pipeline background images. This enhances the visual customization options for pipelines by allowing users to set:

- Custom filter effects (defaults to grayscale(100%) opacity(30%) as before)

Signed-off-by: Mathias Bogaert <mathias.bogaert@gmail.com>
@taylorsilva taylorsilva merged commit 9340fd3 into concourse:master Mar 17, 2025
12 checks passed
@analytically analytically deleted the bgopt branch March 17, 2025 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants