search

googlearchive / core-toolbar

Basic toolbar
7 stars 10 forks source link

fix small bug in new justify attributes #12

Closed albeec13 closed 9 years ago

albeec13 commented 9 years ago

When the justify, middleJustify, and bottomJustify attributes were added, all three changed watchers were using this.justify instead of their respective attributes, so only one justify style would be applied to all three divs. This has been corrected. Also, some new verbiage was added to the comments at the top of the file to indicate a difference between tall and medium-tall class behavior.

ebidel commented 9 years ago

Before this PR can be merged, please sign the CLA. See the CONTRIBUTING guide.

albeec13 commented 9 years ago

Thanks. I signed the electronic version. Let me know if I need to do anything else.

addyosmani commented 9 years ago

Thanks for signing the CLA!

frankiefu commented 9 years ago

LGTM. Thanks for the fix!