Theme customizer: Background attachment selector has no effect in the customizer

[johnbillion]

Changing between fixed/scroll has no effect in the customizer, but it does change the background attachment value when you save it and view your site for real.

[bainternet]

customize-preview.js and customize-preview.min.js patch

[bainternet]

I took a crack at it and in the customize-preview.js line 132 it's

background-position: top

instead of:

background-attachment:

So i created a quick patch ,tested it a bit and it looks good.

[duck_]

In [22131]:

Theme Customizer: Correctly update background-attachment property in live preview

Props bainternet. Fixes #21732.

[duck_]

Thanks for the patch bainternet! Just a couple of notes for the future:

1. Don't worry about updating minified JS/CSS
2. Patching from the root of the trunk checkout is preferred, so svn diff in /path/to/trunk/ instead of /path/to/trunk/wp-includes/js/

[bainternet]

Replying to duck_:

Thanks for the patch bainternet! Just a couple of notes for the future:

1. Don't worry about updating minified JS/CSS
2. Patching from the root of the trunk checkout is preferred, so svn diff in /path/to/trunk/ instead of /path/to/trunk/wp-includes/js/

Gotcha :)