Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link Type: enable_target looks like required but it's not #689

Open
remdan opened this issue Nov 19, 2021 · 1 comment
Open

Link Type: enable_target looks like required but it's not #689

remdan opened this issue Nov 19, 2021 · 1 comment

Comments

@remdan
Copy link
Contributor

remdan commented Nov 19, 2021

Q A
Sulu Version 2.4.0-RC1
PHP Version 7.4.20
DB Version mariadb 10.3
Browser Version Firefox 94.0.1

Actual Behavior

enable_target looks like required but it is possible to save the form without to choice the first time.

Bildschirmfoto 2021-11-19 um 21 35 42

that give you a error in the frontend, then view.link.target is not available.

Bildschirmfoto 2021-11-19 um 21 51 59

Steps to Reproduce

<property name="link" type="link">
    <meta>
        <title lang="en">Link</title>
        <title lang="de">Link</title>
    </meta>
    <params>
        <param name="enable_target" value="true"/>
        <param name="enable_anchor" value="true"/>
        <param name="types" type="collection">
            <param name="page"/>
            <param name="external"/>
            <param name="media"/>
        </param>
    </params>
</property>

<a href="{{ content.link }}" target="{{ view.link.target }}">Click me</a>

@alexander-schranz alexander-schranz transferred this issue from sulu/sulu Nov 21, 2021
@alexander-schranz
Copy link
Member

Thx for the issue. Do you want to create a Pull Request for the docs. To show first a simple and then a complex example how we do it in other content types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants