Improving <button> elements in the HTML of Orchard Core

[Psichorex]

Hi!

We are using HtmlValidation in our UI Tests at Lombiq at we often have to filter the output logs because they contain the following error logs:

• Prefer to use the native element
• must have accessible text
• Redundant role "button" on

These warnings are all coming from OrchardCore's source html code. We could improve on these as these are kind of standard HTML validations.

Some examples:

OrchardCore/src/OrchardCore.Themes/TheTheme/Views/ToggleTheme.cshtml

Line 2 in 187b0e8

<a role="button" class="nav-link dropdown-toggle" id="bd-theme" aria-expanded="false" data-bs-toggle="dropdown" data-bs-display="static" aria-label="@T["Toggle theme"]">

OrchardCore/src/OrchardCore.Modules/OrchardCore.Users/Views/UserMenu.cshtml

Line 6 in 187b0e8

<a class="dropdown-toggle nav-link" id="navbarDropdown" data-bs-toggle="dropdown" aria-haspopup="true" aria-expanded="false" role="button">

Also here is the search result for <a role="button"> https://github.com/search?q=repo%3AOrchardCMS%2FOrchardCore%20role%3D%22button%22&type=code

So we should convert <a role="button"> to <button>
I found somewhere a <button> with no text that triggers must have accessible text
Also found <button role="button">

Locations I found these are mainly the BlogTheme and the TheTheme.