INN / link-roundups

A WordPress plugin to make it easy to collect links from around the web, turn them into roundup posts and streamline the production of daily/weekly roundup newsletters using MailChimp. Built and maintained by INN Labs.
https://wordpress.org/plugins/link-roundups/
GNU General Public License v2.0
20 stars 4 forks source link

Saved links in the Saved Links List Widget are output as adjacent div.post-lead.clearfix, not as list items #182

Open benlk opened 5 years ago

benlk commented 5 years ago

The widget is listing items, so they should be output as a list of items, and then styled to hide the default li styles. Additionally, the post-lead and clearfix classes are Largo-specific, and should be removed. (Largo can get updated to match these styles, or this plugin can ship a Largo shim to add these styles back in.)

https://github.com/INN/link-roundups/blob/be0f2a93a892904bf1d55a9dfd8ccf9311822d87/inc/saved-links/class-saved-links-widget.php#L56

Having these items as a bunch of list items within a list element will improve semantic parseability, and also make it easier to style the list items to match https://github.com/INN/umbrella-workdayminn/issues/16