Carolina Nymark
Carolina Nymark
"We" is problematic when it is not known who "we" are, But using passive voice should also be avoided. I want to avoid it sounding like "The themes team couldn't...
The use of "test-theme" is confusing. Can the visual presentation of the name be replaced with the theme slug? `Notice: Undefined variable: something in wp-content/themes/test-theme/header.php on line 14`
Repeated results are also shown for the link test.
I think this test is too restrictive. It is an attempt to use an allowed list for links, but I think it will be too much work for us to...
This is why I tried to express that checking the menu is not that simple. That opening sub menus on hover is allowed -as long as the keyboard navigation also...
As suggested, I tried the same steps with Powershell as administrator, with the same results. _If it matters, I am not able to destroy wp-env in between attempts, as the...
I tried that again as well as delete the theme review action folder and git cloned it again, force cleaned the npm cache, turned off firewalls, triple checked folder permissions,...
This is the current log of the terminal on Windows: ``` PS D:\tests\theme-review-action> npm install audited 185 packages in 1.216s 20 packages are looking for funding run `npm fund` for...
This is only required for links inside large blocks of text, like post and page content, comment content, and large content in text widgets. The purpose is only make sure...
As I read the list of blockers and flags provided here https://make.wordpress.org/themes/2021/03/17/next-steps-on-themes-and-reviews/ non-core blocks are allowed. For example, you may want to build a web shop with WooCommerce blocks. The...