Link_to image_tag

Hi all,

Can anyone tell me what it is I’ve done wrong with the following?

I’m wanting to use images to link to the page in the site.

So I have this:

<%= link_to(image_tag “about.png”, :border =>“0”, :controller =>
“site”, :action => “about”) %>

Unfortunately, the image doesn’t link as I’d expected. The page
renders, but that’s all.

Can anyone help? I know it’s probably something REALLY obvious, but i
just can pin it down

Many Thanks.

oops…

Spotted the issue:

the correct line should be:

<%= link_to image_tag (“about.png”, :border =>“0”), :controller =>
“site”, :action => “about” %>

That works now.

On Nov 23, 2010, at 1:19 PM, RubyonRails_newbie wrote:

Unfortunately, the image doesn’t link as I’d expected. The page
renders, but that’s all.

Can anyone help? I know it’s probably something REALLY obvious, but i
just can pin it down

Many Thanks.

Try placing parentheses around the arguments to each function.

Think about where Ruby is presuming the argument lists are from the
punctuation that you’ve given.

-Rob

Rob B.
[email protected] http://AgileConsultingLLC.com/
[email protected] http://GaslightSoftware.com/

On Nov 23, 2010, at 1:29 PM, RubyonRails_newbie wrote:

oops…

Spotted the issue:

the correct line should be:

<%= link_to image_tag (“about.png”, :border =>“0”), :controller =>
“site”, :action => “about” %>

That works now.

Well, I’d suggest formatting like:

<%= link_to(image_tag(“about.png”, :border =>“0”),
:controller => “site”, :action => “about”) %>

I might leave the parentheses off the call to link_to, but only if it
didn’t break across lines in the actual source. In fact, since you’re
using the hash argument form for the url construction, you really
could have this and be exactly the same (from a Rails-semantics
perspective):

<%= link_to(image_tag(“about.png”, :border =>“0”),
{:controller => “site”, :action => “about”}, {}) %>

I mention this because going back and adding an option that you expect
to be given to link_to can easily be given to the underlying call to
url_for that happens inside the link_to helper.

You might want to take a look at an old article by James Edward G. II

http://blog.grayproductions.net/articles/do_i_need_these_parentheses

Enjoy your Ruby! (and Rails)

-Rob

“site”, :action => “about”) %>
You received this message because you are subscribed to the Google
Groups “Ruby on Rails: Talk” group.
To post to this group, send email to rubyonrails-
[email protected].
To unsubscribe from this group, send email to
[email protected]
.
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en
.

Rob B. http://agileconsultingllc.com
[email protected]
+1 513-295-4739
Skype: rob.biedenharn

Great - thanks for the feedback… :slight_smile:

On Nov 23, 7:05pm, Rob B. [email protected]