Sal Ferrarello

Results 75 comments of Sal Ferrarello

The Genesis WordPress theme (a.k.a. the Genesis Framework) is available for purchase at https://my.studiopress.com/themes/genesis/ My thoughts on why this purchase is worth it ([Why Use the Genesis Framework](https://salferrarello.com/why-use-genesis-framework/)). One other...

Here are some screenshots for reference ## HTML Widget ### With Title ![screenshot 2018-02-23 16 30 17](https://user-images.githubusercontent.com/5194588/36617874-22670152-18b7-11e8-9bb6-c017f43148cf.png) ### Without Title ![screenshot 2018-02-23 16 30 48](https://user-images.githubusercontent.com/5194588/36617876-2481e57e-18b7-11e8-8f77-ab177b046497.png) ## Text Widget ### With...

This WordPress StackExchange Question: [Adding a div to wrap widget content after the widget title](https://wordpress.stackexchange.com/questions/74732/adding-a-div-to-wrap-widget-content-after-the-widget-title) and the WordPress support issue that it links to [Add HTML wrapper around widget content](https://wordpress.org/support/topic/add-html-wrapper-around-widget-content/#post-2799774)...

Hmmm.... the hook in their solution, `widget_text`, only applies to the Text and HTML widgets. Here is the base code for [class-wp-widget.php](https://github.com/WordPress/WordPress/blob/aaf99e691391cfceb004d848450dbbf3344b1bee/wp-includes/class-wp-widget.php) I don't see any great place to work...

Okay, I've given this some more thought, how we might handle this. ## Add the classes via a filter Unfortunately, I don't see a great way to wrap the content...

Playing around with this, if I add the padding to `.panel`, it messes up the header. ![screenshot 2018-02-27 17 12 26](https://user-images.githubusercontent.com/5194588/36758130-7b7d1cfe-1be1-11e8-9af6-1f3f8ab7f134.png) but if we then applying a negative margin to...

@atcraigwatson I got a chance to look at this. Am I correct you are talking about your branch [cw/remove-panel-heading-wrapper](https://github.com/atcraigwatson/bootstrap-genesis/tree/cw/remove-panel-heading-wrapper)? I'm looking at this and visually comparing it to before we...

Thanks for doing the leg work on reading those docs. > This says to me, drop it in the .gitignore, like you say it's part of the build process of...

I've updated the title of this issue, now that we have a game plan.

Is it possible your text editor is recognizing the incorrect formatting and correcting it? In any case, it definitely makes sense to fix.