To <p> or not to <p>

[ldoodle]

I have this code:

<div class="module">

<p>

<span class="float_right">

<a href="" onclick="toggle('<%= strCategoryName %>'); return false;">

<img id="<%= strCategoryName %>Image" src="/images/collapse_module.png" alt="Collapse " title="Collapse this module " />

</a>

</span>

<img src="/images/web_links.png" alt="Web Link " /><%= strCategoryName %>

</p>

</div>

Is the use of <p> correct in this instance?

Thanks

[Fang]

<div>
<img /><img />
</div>

[Declan1991]

Neither the span nor the p is needed, and even the <a> can be removed by putting the onclick on the img tag instead of the <a>.

[WebJoel]

What Fang said. :)
Wondering, -if the "<a>" is removed and an 'onClick' on the "<img>" is used instead, -does this deprive user-agents for the handicapped from realizing this is a 'link' because the element would not identify itself as being an "anchor" tag? Just wondering.
The hyperlink would work, but for a blind person, the 'reader' might not identify the element with 'onClick' as being 'clickable.' Anyone? (I seriously need to re-install JAWS and check some of these issues out..) :o

[dmboyd]

As mentioned, only the "<img/>" tags are needed.

In addition, user agents that don't support scripting should not have an issue with the onclick, though I can't remember whether the page would simply reload the page or not when a link with href="" is used; such behaviour would definitely make a user's browsing experience painful.

As for JAWS, I can't really test, but I would assume that it wouldn't recognise it as being clickable. After all, just because there is an onclick attribute, that doesn't mean it is a link, and that's all a screen reader should really be searching for IMHO.

[Fang]

Wondering, -if the "<a>" is removed and an 'onClick' on the "<img>" is used instead, -does this deprive user-agents for the handicapped from realizing this is a 'link' because the element would not identify itself as being an "anchor" tag? Just wondering.
The hyperlink would work, but for a blind person, the 'reader' might not identify the element with 'onClick' as being 'clickable.' Anyone? (I seriously need to re-install JAWS and check some of these issues out..) :oEasy enough to change the cursor for the mouseover. Jaws will recognize the event.
The only problem is that JavaScript is required for the document to be functional.

[ldoodle]

Thanks.

The reason the span is there is as a container for the possibility of multiple items.

[Fang]

If the full markup is not given, how can we comment on the use of <p>?

[felgall]

If the whole thing is a part of a paragraph then why wrap the paragraph in a div rather than just applying the div styles directly to the paragraph. If it isn't a paragraph then the <p> tags shouldn't be there. Wrapping a paragraph in a div shouldn't be required (except in the rare instance where the required appearance can't be achieved by styling the paragraph and styling at two levels is required).