Account created on 14 August 2017, over 7 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States mferanda

Also, feel free to reach out through profile email or slack. I definitely dont mind collaborating on anything or discussing for federal use.

🇺🇸United States mferanda

Hi, @timwood, there is nothing stopping you from upgrading to the latest minor version of 3.x. As it says on project home.

This theme is meant to be basic for that reason outside of major versions. As far as I know, there are no issues. I do have a change for adding some new header tags for a view table *I think* but thats it. Thats honestly a catch up from an item i believe they added for 3.8

When I release a new version, all that is truly modify are example related CDNs that no one should be using for production. In fact, it only updates those for a fresh install and doesn’t change your version.

Give it a try and let me know if there are any issues with the core theme. Ive been running the latest myself with no issues.

Ill push out to sync up the page out there soon… but again. Unless you’re waiting for sticky header uswds class in a table view theme… not a lot going on with minor updates versus major.

🇺🇸United States mferanda

This is the core patch I'm referring to, btw.

https://www.drupal.org/files/issues/2024-12-04/3456176-bootstrap_barrio-fix-status-message-core-10.3.patch

It removes the newer js bigpipe barrier to then allow warnings to flow to the normal twig template path.

🇺🇸United States mferanda

Temporarily remove your twig template for the warning messages. It worked for me when I tested prior to me asking, but there is a theming issue with this fix. When I did try to custom theme the warnings with a twig, broke.

So far, the best route in my testing was to use a patch that removes the bad code in Drupal Core itself. I'm hoping that goes in place. Theming inside of js isn't good. However, I'd be inclined to add the js fix in documentation. As well as that Drupal Core patch that could be added to composer.

🇺🇸United States mferanda

Looks like this should just be a simple change to the views table template. Take a look and see if this works for you.

🇺🇸United States mferanda

I may have found a simple fix. Please try and let me know.

🇺🇸United States mferanda

Always best to keep the core classes. Have you tested out the sticky headers with USWDS and Drupal?

🇺🇸United States mferanda

@jonmcl How would you change this theme to reflect the preprocess function? Can you provide a change and demonstrate that it works?

🇺🇸United States mferanda

Sorry, completely missed this. I'll review soon.

Production build 0.71.5 2024