alleyinteractive / alleyinteractive/wp-asset-manager

Critical CSS doesn't respect dependency order

Ouverte
#22 0 commentaires 0 réactions 1 personne assignée Réclamée par @renatonascalves Voir sur GitHub
enhancement php
Langage dominant
PHP
Étoiles
30
Forks
5
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

When more than one critical CSS file is registered (for example, one from a parent theme and one from a child theme) the dependencies are not respected. They get output in the order in which `add_asset()` is called, not according to their dependencies.

In the code below, `critical-first` should be output first because it's a dependency of `critical-second`, but `critical-second` gets output first because that's the order in which `add_asset()` is called.

```php
$styles = [
[
'condition' => 'global',
'deps' => [
'critical-first',
],
'handle' => 'critical-second',
'load_hook' => 'am_critical',
'load_method' => 'inline',
'src' => MY_PATH . '/critical-second.css',
'version' => 1,
],
[
'condition' => 'global',
'deps' => [],
'handle' => 'critical-first',
'load_hook' => 'am_critical',
'load_method' => 'inline',
'src' => MY_PATH . '/critical-first.css',
'version' => 1,
],

];

array_map( [ \Asset_Manager_Styles::instance(), 'add_asset' ], $styles );
```

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.