Extending the text block with text color select field

Kirby 3 Solved ✅
1

Hi,
I would like to extend the text block with a select field with which you can determine the text color.
With the Blueprint this works very well - but with the Template / Snippet I am not sure if this is a good solution.
Maybe an experienced Kirby user can take a look at it?

content.yml

extends: blocks/text
   fields:
      text_color:
      extends: fields/colors
      label: Text color

text.php

<?php
if ($block->text_color()->isNotEmpty()):
  $text = $block->text();
  $search = ["<p>"];
  $replace = ["<p class='text-color--" . $block->text_color() . "'>"];
  echo str_replace($search, $replace, $text);
else:
  echo $block->text();
endif
?>

Does this look strange to you or can I do so?
Best
Patrick :wink:

edit: something was replaced twice…

2 
<?php 
$text = $block->text();
if($block->text_color()->isNotEmpty()) {
  $text = preg_replace('/<p>/', '<p class="text-color--' . $block->text_color() . '">', $text);
}
echo $text;
3

Thanks :wink:

I am not sure why you are using preg_replace instead of str_replace and why it doesn’t throw out an exception because of the missing else statement…

4

An if statement doesn’t (necessarily) need an else statement. In my example above, we first define $text. If the condition is true, $text gets redefined (whereas you set your condition first and inside that condition define $text.

You can also use str_replace instead of preg_replace:

  $text = str_replace('<p>', '<p class="text-color--' . $block->text_color() . '">', $text);
5

Ah, understood :wink:
Best, Patrick