Convert Olivero's tags to use single directory components

Converted Oliveros tags to single directory component

CC failure in the MR.

Tried testing though and after applying the MR and clearing cache I get this error

Twig\Error\LoaderError: Template "olivero:tags" is not defined in "core/themes/olivero/templates/field/field--node--field-tags.html.twig" at line 42. in Twig\Loader\ChainLoader->getCacheKey() (line 99 of vendor/twig/twig/src/Loader/ChainLoader.php).

Robert Ngo → made their first commit to this issue’s fork.

After a quick review, this is looking really good!

The only quick nit is that you can simplify your include function.

instead of doing a

{{ include('olivero:tags', {
  attributes: attributes,
  label_hidden: label_hidden,

you can simplify to

{{ include('olivero:tags', {
  attributes,
  label_hidden,

Also, I'm guessing that the tests are failing because SDC is not being enabled by default. So tests won't pass at this point

This is looking just about perfect! The only major change is to assemble the classes and title_classes arrays within the field--node--field-tags.html.twig file.

Note that tests will not pass on this because SDC is not a part of standard profile. Once this gets to a good place we'll postpone it until SDC becomes stable.

Changes look good (with my limited knowledge of SDC). I'm assuming the failures are due to SDC not being in standard. Postponing issue until that happens.

Gonna work on it

I started a new approach because I believe that styles in components should not be tied to Drupal. This makes them more flexible.

So it shouldn't be
.field .node-- and other things

Just pure simple and clear css

Also fixed flexboxes and added support for label above

Please check!

The styles are looking as expected. Only thing i can figure out is the flex direction is column now. Attaching screenshot below

Yeah. I've added above/inline label display support.

@finnsky, This makes sense and the above/inline field display is working as expected.

Inline:

Above: