The sidebar looks great! My only criticism would be on a short line length once the advertising column is included on story pages.
I'll survive :) +1

Luis Roca
luisroca@protonmail.com
https://keybase.io/luisroca

Sent with ProtonMail Secure Email.

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On March 26, 2018 12:38 PM, Geoffrey Marr <gmarr@redhat.com> wrote:

It looks great Ryan. I’m +1!

On Monday, March 26, 2018, Justin W. Flory <jflory7@gmail.com> wrote:
On 03/26/2018 03:18 AM, Ryan Lerch wrote:
> Hi all,
>
> I have been hacking lately on a new theme for the magazine. This theme
> kinda brings us more in line with the other Fedora websites / apps in terms
> of theme.
>
> Additonally, this adds a sidebar that is constant on both the posts and
> front pages, which will allow us to better run "ads" like we discussed with
> a link to the community blog, or elsewhere in the Fedora Project.
>
> anyhoo, here are some screenshots, and it there are no major issues with
> the layout, i can work towards getting this live soon!
>
> https://ryanlerch.fedorapeople.org/magazine-2018-post.png
> https://ryanlerch.fedorapeople.org/magazine-2018-frontpage.png
> https://ryanlerch.fedorapeople.org/magazine-2018-postseries.png
>

The new look for the Magazine looks awesome! If you're working on
WordPress themes, it would also be nice to pull the Community Blog
closer in sync with what is happening on the Magazine too.

    https://pagure.io/communityblog-theme

I'd love to see some of these style changes work their way into the
Community Blog theme too.

--
Cheers,
Justin W. Flory
jflory7@gmail.com



--
Geoff Marr

IRC: coremodule