Remove redundant ARIA role="main" from BP legacy template files

[mercime]

WP 4.2.2 and BP trunk

We have ARIA role="main" markups in BP template files which should not there because all WordPress core themes from Twenty Fifteen down to Twenty Ten already have that ARIA role in respective page.php and relevant files. The same goes for Automattic's _s theme which continue to serve as a starter theme to many themes in the WP Theme Repository. There should only be one main landmark role per page. ​http://www.w3.org/TR/wai-aria/roles#main

This was a carry-over when template files were ported from the old BP Default theme to the legacy folder for the theme compatibility feature.

Attached is a screenshot of a BP page with 2 extra role="main" instances

Attached are patches for the respective files in the activity, blogs, forums, groups, and member folders.

[mercime]

3 in one page example

[johnjamesjacoby]

Good catch, mercime.

I am completely comfortable squeezing these into 2.3. Anyone opposed?

[hnla]

It is a good catch, I'm for squeezing this in, it's not a feature change, should not upset any theme styling (very unlikely anyone would ever style on this attr

In 2.4 we'll start to address more of these concerns hopefully with some new templates but anything we can catch and update now the better.

[johnjamesjacoby]

In 9896:

Templates: Remove redundant ARIA role="main" declarations from bp-legacy template parts.

These were carried over from bp-default and are no longer accurate in the context of theme compatibility, where the main parent/child theme is commonly responsible for this declaration.

Props mercime. Fixes #6456.