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

Open in a new tab RedirectorPage #971

Closed
wants to merge 2 commits into from
Closed

Conversation

mullerivan
Copy link

i try to include a option to open the links in a new windows

@tractorcow
Copy link
Contributor

This can be easily done with extensions, and it's not necessary to have this in core. The logic in your target method can, and probably should, be in your templates.

<a href="$Link" <% if $NewTab %>target="_blank"<% end_if %>>$MenuTitle</a>

@mullerivan
Copy link
Author

yes I use in this way on my project....
u Think just add the check box on the Admin area and not the target function???

I have though is useful and Clear to read (maybe because i think the engine template is real bad..)

@simonwelsh
Copy link
Contributor

This is trivial to do with a DataExtension. There is no need for it to be in core.

@simonwelsh simonwelsh closed this Mar 13, 2014
@mullerivan
Copy link
Author

But is a cms, some people use it and don't know how to add it...

I have seen this kind of functionality in many CMS like Drupal wordpress joomla etc...
But if this CMS is only for programmers............

i just wandering to do this CMS more useful for the final users/Humans

Cheers

@tractorcow
Copy link
Contributor

Developing a site with Silverstripe is for programmers, yes. It's their job to make a tool that their clients can use. The ideal is to keep the core framework as lean as possible, and make it available for developers to bolt on functionality with extensions.

I suggest you read up on this here http://docs.silverstripe.org/framework/en/reference/dataextension and learn how you can add custom functionality (such as your suggested feature) in your own projects. If you have questions feel free to ask on the forums, or email me if you're really stuck on something.

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

Successfully merging this pull request may close these issues.

3 participants