Results 333 comments of Kelly Dwan

You're talking about [the site title set here](https://github.com/ryelle/Foxhound/blob/master/functions.php#L167), right? I don't think `document_title_parts` really applies here– that's meant to be run over the entire title in parts, but this is...

My short answer is no, I don't plan to add server side rendering, the pre-loading that @iandunn mentioned was my compromise there — the theme doesn't have to wait for...

All attachments (including uploads) should be redirecting to the `/attachment/:id` page if loaded from the server ([code]( https://github.com/ryelle/Foxhound/blob/master/inc/permalinks.php#L128)), and attachment links in the JS-rendered content should also be caught by...

Thanks for the report. Can you send over a screenshot of what's not working for you? Are you seeing any errors in console?

Thanks for the report! External links in post and page content should be correctly treated as external, but the same isn't true for widgets — because a lot of widgets...

There was a reason I didn't go this route early on in development, but to be honest I don't remember why. I'll give it another go and we'll see 😄

Here are some ideas/suggestions after a little thinking (with the caveat that I don't know what the current landscape of theme javascript is like). - [eval()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/eval) Same as the PHP...

@hlashbrooke is that this section?

Sorry for un-approving, but I went back to the original locale PR. In #112, the wporg `locales` plugin was added to handle generating the locale names based on the standardized...

Well, my fix worked locally but isn't working in github actions, so [you can see the broken tests here](https://github.com/WordPress/pattern-directory/runs/7777236682?check_suite_focus=true).