Allow empty background colors to denote transparency about simple-social-icons HOT 4 CLOSED

👍 from me since it was me that opened the original thread at the WP forums.

Here's my modification to the update() method:

/** Accept empty colors */
elseif ( strpos( $key, '_color' ) && '' == trim( $value ) ) {
    $newinstance[$key] = '';
}

/** Validate hex code colors */
elseif ( strpos( $key, '_color' ) && 0 == preg_match( '/^#(([a-fA-F0-9]{3}$)|([a-fA-F0-9]{6}$))/', $value ) ) {
    $newinstance[$key] = $oldinstance[$key];
}

This "works" as is, though as mentioned it is probably a good idea to adjust the CSS output.

from simple-social-icons.

@nickcernis This works as described.

It is possible to now have transparent icons. I'm not entirely sure there's a use case for that, but if icons aren't showing for someone, they may be transparent. Do we want that here?

from simple-social-icons.

@dreamwhisper Thanks for testing, and I agree — I opened #82 and will address this for 2.1.

from simple-social-icons.

@nickcernis Thanks! I'll mark this confirmed for the background.

from simple-social-icons.