- Issue created by @joachim
- First commit to issue fork.
- π©πͺGermany michagru Bamberg
I found the BlockPageVariant as an alternative to mention
- πΊπΈUnited States xjm
I read
</code> and confirmed that this is what it actually does. Since it's always good to check <em>why</em> something like this is wrong (especially to confirm that we have the correct replacement, I did: <code> git log -S "FullPageVariant"
It looks like this was removed back in 2014 in [#]. That issue includes e.g.:
diff --git a/core/lib/Drupal/Core/Display/Annotation/DisplayVariant.php b/core/lib/Drupal/Core/Display/Annotation/DisplayVariant.php index 3c05cf3b85e..3bf74920f3c 100644 --- a/core/lib/Drupal/Core/Display/Annotation/DisplayVariant.php +++ b/core/lib/Drupal/Core/Display/Annotation/DisplayVariant.php @@ -25,8 +25,9 @@ * * Plugin namespace: Plugin\DisplayVariant * - * For a working example, see - * \Drupal\block\Plugin\DisplayVariant\FullPageVariant + * For working examples, see + * - \Drupal\Core\Render\Plugin\DisplayVariant\SimplePageVariant + * - \Drupal\block\Plugin\DisplayVariant\BlockPageVariant *
...and git even detects it as a rewritten file:
.../{FullPageVariant.php => BlockPageVariant.php} | 12 +-
Finally, I grepped to confirm that there were no other stale references to
FullPageVariant
:[ayrton:drupal | Fri 14:33:16] $ grep -r "FullPageVariant" * core/lib/Drupal/Core/Display/PageVariantInterface.php: * For example, the \Drupal\block\Plugin\DisplayVariant\FullPageVariant page
Committed to 11.x, and cherry-picked to 11.2.x as a patch-safe documentation bug fix. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.