themovation / th-widget-pack

Theme Widget Pack
19 stars 6 forks source link

Package widget - Markup ready #52

Closed teaganm closed 8 years ago

teaganm commented 8 years ago

Included in the template file are two examples - the first one is with the default settings, and the second one shows it with background colors specified and light text selected on both areas.

For the price section - “th-pkg-info” - there is a class that changes depending which text contrast is selected:

Also in the price section I was thinking the background color would be output as rgba. So let’s change ‘Transparency’ to ‘Opacity’ and output that as the alpha value in rgba. The Opacity field would then default to 100 instead of 0 instead. Does that work well?

Similarly in the content section - th-pkg-content - there is also a class that changes depending which text contrast is selected:

The image gets output into a div with a class.

For the link, there’s just an a tag just inside the article with a class added.

Let me know if there are any issues with that.

adiraoco commented 8 years ago

Implemented in d5b02cf7acc9d7036d528a263897e5fbd8fde94e.

Please have a look. Thanks.

teaganm commented 8 years ago

Looks great. Just one change. We'd like to have it so that if no link is input, then the a tag does not get output either.

adiraoco commented 8 years ago

Implemented in d2e8fc3744108dea3c60c199e25ab7bb1b34049f

teaganm commented 8 years ago

Looks great, thanks!