We could argue a profile is an article, especially so if/when profiles become proper entities (http://groups.drupal.org/node/155634 some related discussion).

http://api.drupal.org/api/drupal/modules--user--user-profile.tpl.php

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stijnbe’s picture

I changed the div tags into html5 article tags. Patch attached.

stijnbe’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, user-profile.tpl_.php-html5-1190214.patch, failed testing.

stijnbe’s picture

Status: Needs work » Needs review
FileSize
490 bytes

I added foaf:person xmlns:foaf="http://xmlns.com/foaf/0.1/". Is this the right way to add rdfa for a user profile?

stijnbe’s picture

It seems like I was a bit to fast. The correct rdfa tag was already set with the patch in comment #1. Why is the test failing?

Status: Needs review » Needs work

The last submitted patch, user-profile-html5-1190214-4.patch, failed testing.

stijnbe’s picture

Status: Needs work » Needs review
FileSize
1.16 KB

fixed rdf.test

scor’s picture

@stijnbe: re #4, as you found out, no need to add the namespaces manually, they are handled automatically by Drupal (and inserted in the html tag at the top of the HTML output).

cosmicdreams’s picture

As per http://www.drupal4hu.com/node/304 , Profile is no longer apart of D8. Can we (won't fix) this issue?

droplet’s picture

Oh. Profile module still in D7 and hidden, never known it.

the file in this issue used by User module :)

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
1.33 KB

This looks very RTBC for me...
Rerolled.

Adding sprint tag so it gets noticed this sunday. (if there is a sprint on sunday)

aspilicious’s picture

Status: Reviewed & tested by the community » Needs review

not rtbc yet ;)

aspilicious’s picture

Issue tags: +sprint

forgot the tag :(

Jacine’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -sprint

This is RTBC. Shouldn't have the sprint tag though. We define issues for the sprints every other Tuesday in our IRC meeting.

catch’s picture

Status: Reviewed & tested by the community » Needs review

hmm it feels like a stretch calling user account pages as core provides them an article. Out of the box it is just "Member for x weeks and y days" and not a lot else. On Drupal.org it includes all the stuff at http://drupal.org/user/35733 including commits etc. within what is currently that div.

I'm not going to argue against it, but it'd be good to have a better justification for why this is semantically a good idea.

mortendk’s picture

Status: Needs review » Reviewed & tested by the community

using the article tag here is 100% what its used for, its very specific, out of the box every little tiny node in drupal is probably article, its nothing to do with the size of it.

to quote from the good htnl5 doctor

An independent piece of content, one suitable for putting in an element, is content that makes sense on its own. This yardstick is up to your interpretation, but an easy smell test is would this make sense in an RSS feed? Of course weblog articles and static pages would make sense in a feed reader, and some sites have weblog comment feeds. On the other hand, a feed with each paragraph of this article as a separate post wouldn’t be very useful. The key point here is that the content has to make sense independent of its context, i.e. when all the surrounding content is stripped away.

http://html5doctor.com/the-article-element/

or take it from our good friends at MDN

The HTML Article Element () represents a self-contained composition in a document, page, application, or site, which is intended to be independently distributable or reusable, e.g., in syndication. This could be a forum post, a magazine or newspaper article, a blog entry, a user-submitted comment, an interactive widget or gadget, or any other independent item of content.

https://developer.mozilla.org/en/HTML/Element/article

not using article here would be a great mistake & its following the precise definition of the tag.
rtbc +1

catch’s picture

Status: Reviewed & tested by the community » Fixed

OK the html5doctor example is not really conclusive for me does "member for 5 weeks" make sense out of context?

However ", an interactive widget or gadget, or any other independent item of content." sounds completely OK for this, and it's not the template's fault that the account page is a bit bare out of the box.

Committed and pushed to 8.x, thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.