Jump to content

Forum Upgrades


Pete
 Share

Recommended Posts

I've just upgraded the forums to the latest version 4.4.x as there was a critical security patch that warranted it.

As usual, there is some pain in upgrading from a 4.x to 4.x branch due to template changes and feature changes/additions so you may find things have moved around a bit.

There is still work to do on the template as this was a little rushed (security update forced my hand) but it should be functional at the very least. If you spot anything drastically broken that I may have missed, please let me know here. Minor template tweaks (fonts, alignment etc) will be updated in the next few days.

  • Like 19
Link to comment
Share on other sites

Looking nice even in its unfinished state :-).

One thing now that we have you here (I think I might have asked this before), you know when one is browsing list of unread posts/content, we used to be able to hover on a topic's title in the list and click a button/link to mark that AND ONLY that post as read. That was a time saver! There are times when you just want to mark A FEW/SOME posts as read without actually opening them (?). Are you able to look into this please?

Thanks!

  • Like 2
Link to comment
Share on other sites

Something looks wrong here. The avatars in the Latest Topics column are stretched out vertically. Also the whole Latest Topics column has moved upwards out of line with the Welcome to Processwire column.

  • Like 2
Link to comment
Share on other sites

Just now, pwired said:

Something looks wrong here. The avatars in the Latest Topics column are stretched out vertically. Also the whole Latest Topics column has moved upwards out of line with the Welcome to Processwire column.

Yep, spotted those so will look into them as I get time.

  • Like 2
Link to comment
Share on other sites

Most of the styling issues are where UIKit for the new PW header is conflicting with the forum software styling. I'm actually surprised it broke as little as it has to be honest but will get onto these theme issues in the next day or two.

  • Like 4
Link to comment
Share on other sites

This is less of a bug and more of a.. well, issue: the line length in the forum is kind of crazy. I'm getting 220-240 characters on a single line, while 100-120 would be good for readability. Would be great to get this fixed as well. I'm thinking that there probably should be a max width for the content area – or perhaps the font size should be double what it is now? ?

The font is also really tiny and way too light (at least in some places, such as the breadcrumbs and other "less important" texts), neither of which really helps with the readability issue.

Anyway, great job getting the update rolled out, and the forum looking more like the main site!

Edit: I'm officially getting old. Setting browser zoom level to 125% or 150% makes things way more comfortable ?

  • Like 6
Link to comment
Share on other sites

When you do an accessibility audit on the forum (and the main PW site) there are tons of color contrast failures (among many other issues). WCAG 2.1 states that color contrast has to be at least 4.5:1 for copy text, whereas here most text is below that.

And yes, font size and line-heights are too small. body base font size should be at 16px with line-height 1.6 (unitless, not in pixels). And ideally, all other font-sizes further down should be specified in em or rem.

The main container could be reduced: e.g. max-width 1280px instead of 1630px.

  • Like 2
Link to comment
Share on other sites

On 3/14/2019 at 3:17 PM, bernhard said:

wow!! new design for the forum, great!! ? 

Avatars are also squeezed in the latest topics links:

o0PbuSF.png

I also see this. Solved with:

.ipsDataItem_icon .ipsUserPhoto {
    width: 44px;
}

 

  • Like 1
Link to comment
Share on other sites

On 3/15/2019 at 10:38 PM, Pete said:

Most of the styling issues are where UIKit for the new PW header is conflicting with the forum software styling.

The padding around quoted text is a bit much, as can be seen in the quote immediately above. Seems to be coming from Uikit styles, which I guess are being included in their entirety just for the header. Could we strip it back a bit so only the needed styles are included? Or iframe the header part to avoid the styling conflicts?

Another issue I've noticed since the upgrade: in iOS Safari (older versions) links are triggered on release of long touch (to open the context menu). This makes it difficult to open links in a new tab - you have to slide your finger away from the link before you release in order to select an item in the context menu, or else you navigate away within the current tab. This only applies to Safari and only on older Safari/iOS versions so perhaps it doesn't affect too many people. It's a pretty big annoyance to me though because you cannot select an alternative default browser on iOS (thanks Apple) and I don't want to have to buy a new iPad.

  • Like 1
Link to comment
Share on other sites

Fixed a lot of these. I don't really want to change much else as it's a massive pain in the ass when they change the forum templates every other upgrade.

Things like UIKit styles adding big padding on quotes - I could spend a long time tweaking UIKit to just target the header element only somehow I guess but it's easier to just include Ryan's stylesheets and  override a single class (done, padding is now 20px) ?

Accessibility - if someone wants to give me the CSS changes I can just plug in I'd be happy to do so. I don't know enough about accessibility to tackle that myself.

Forum width is same as the site width which makes sense visually, though perhaps we're missing a trick nowadays and could have a helpful sidebar at larger screen widths? Maybe some sort of quick links for docs etc? I'm not sure - it's more work and it's taken me this long just to tweak a dozen styles - I don't get paid for this and have to prioritise client work to pay the bills ?

  • Like 8
  • Thanks 1
Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...