It should be probably resolved by #9804? cc @pamil

@Zales0123 not really, #9804 solves templates resolving, their location is another issue to solve.

I just made a PR #9814 to change this.

The default path : %kernel.project_dir%/app/themes has to be changed in Sylius\Bundle\ThemeBundle\Configuration\Filesystem\FilesystemConfigurationSourceFactory, in all a test file, and documentation.

I use this conf now to get ride of this bug :

# config/packages/sylius_theme.yaml
sylius_theme:
    sources:
        filesystem:
            directories:
                - "%kernel.project_dir%/templates/themes"

I agree that templates/themes should be the new place for themes. We also need a better doc on how to build a theme with a real and updated theme skeleton.

After giving it some thought, I would vote for top-level themes/ directory. templates/themes/ does not look like the best place for themes, because they also contain translations and assets, which might be misleading when under templates/ directory.

My first thought was the same as yours but the templates dir is already used by SF users as the default place to override things.
If there is a root themes dir it means that you want to composer require themes outside the vendor dir when some customs themes will be available in a Bundle.

Is there any custom themes already available somewhere ?

Themes are mostly proprietary and made-on-demand because most of the stores need their own, unique theme.

Symfony 4 structure contains assets/, templates/ and translations/ root directories. Themes contain all of these resources (also using the old nomenclature, assets -> public and templates -> views).

With templates/themes/ there are the following issues:

• translations and assets are under templates/ instead of their dedicated directories which is counter-intuitive
• the structure for templates is also different than templates/bundles/:
  • overriding a bundle template templates/bundles/FooBundle/template.html.twig in a theme would be templates/themes/ThemeName/FooBundle/views/template.html.twig - so there's views directory out of nowhere

If your themes are installed via Composer, it's needed to specify that path in your sylius_theme configuration.

@pamil I change my PR #9814 to get this fact as default.

So app/themes/ default value will become themes/.

will theme directory reflect new flex directory?