Skip to content

(#771): Fix incorrect default for the flood-color property

Federico Mena Quintero requested to merge federico/librsvg:771-flood-color-default into master

Per the spec, the default is opaque black. However, librsvg's default value was all-zeros, or transparent black.

43499a14 refactored the feFlood primitive to pass an already resolved color to SharedImageSurface::flood(). Prior to that, the code in flood() was incorrectly using just the RGB values from the flood-color property, but an alpha value equal to the flood-opacity - instead of first resolving the whole RGBA flood-color with the flood-opacity.

Now that the flood-color is resolved properly, it turns out that the default was wrong.

Fixes #771 (closed)

Closes #771 (closed)

Merge request reports