starred/radzen-blazor
1
0
Fork 0
mirror of https://github.com/radzenhq/radzen-blazor.git synced 2026-02-12 21:31:24 +00:00
Code Issues 1.3k Packages Projects Releases 27 Wiki Activity

Fixed redundant link WCAG warning

Browse Source 
Removed href="javascript:void(0)" and added @onclick:preventDefault="true" instead
This commit is contained in:
Vladimir Enchev
2023-12-07 16:04:56 +02:00
parent 4a50a3161d
commit a2144d21c2
14 changed files with 28 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Radzen.Blazor/RadzenAccordion.razor
View File

@@ -18,7 +18,7 @@
continue;
<div @ref="@item.Element" id="@item.GetItemId()" @attributes="item.Attributes" class="@item.GetItemCssClass()" style="@item.Style">
<a @onclick="@((args) => SelectItem(item))" aria-label="@ItemAriaLabel(i, item)" title="@ItemTitle(i, item)" href="javascript:void(0)" role="tab" tabindex="0"
<a @onclick="@((args) => SelectItem(item))" aria-label="@ItemAriaLabel(i, item)" title="@ItemTitle(i, item)" @onclick:preventDefault="true" role="tab" tabindex="0"
id="@($"rz-accordiontab-{items.IndexOf(item)}")" aria-controls="@($"rz-accordiontab-{items.IndexOf(item)}-content")" aria-expanded="true">
@if (IsSelected(i, item))
{

4
Radzen.Blazor/RadzenDataGrid.razor
View File

@@ -35,7 +35,7 @@
{
<div class="rz-group-header-item">
<span class="rz-group-header-item-title">@gd.GetTitle()</span>
<a aria-label="@RemoveGroupArialLabel" href="javascript:void(0)" @onclick=@(args => RemoveGroupAsync(gd)) role="button" class="rz-dialog-titlebar-icon rz-dialog-titlebar-close">
<a aria-label="@RemoveGroupArialLabel" @onclick:preventDefault="true" @onclick=@(args => RemoveGroupAsync(gd)) role="button" class="rz-dialog-titlebar-icon rz-dialog-titlebar-close">
<span class="rzi rzi-times"></span>
</a>
</div>
@@ -508,7 +508,7 @@
@if (this.LoadChildData.HasDelegate && this.ShowExpandColumn && this.allColumns.IndexOf(column) == 0)
{
<span class="rz-cell-toggle">
<a aria-label="@ExpandChildItemAriaLabel" class="@(getExpandIconCssClass(this, Item))" style="@(getExpandIconStyle(this, Item, rowArgs.Item1.Expandable))" href="javascript:void(0)" @onclick="_ => this.ExpandItem(Item)" @onclick:stopPropagation>
<a aria-label="@ExpandChildItemAriaLabel" class="@(getExpandIconCssClass(this, Item))" style="@(getExpandIconStyle(this, Item, rowArgs.Item1.Expandable))" @onclick:preventDefault="true" @onclick="_ => this.ExpandItem(Item)" @onclick:stopPropagation>
<span class="@(this.ExpandedItemStyle(Item))"></span>
</a>
<span class="rz-cell-data" @attributes="@spanAttributes">

2
Radzen.Blazor/RadzenDataGridGroupRow.razor
View File

@@ -17,7 +17,7 @@
}
<td class="rz-col-icon">
<span class="rz-column-title"></span>
<a aria-label=@Grid.ExpandGroupAriaLabel href="javascript:void(0)" @onclick="@(_ => Grid.ExpandGroupItem(this, rowArgs.Item1.Expanded))">
<a aria-label=@Grid.ExpandGroupAriaLabel @onclick:preventDefault="true" @onclick="@(_ => Grid.ExpandGroupItem(this, rowArgs.Item1.Expanded))">
<span class="@(Grid.ExpandedGroupItemStyle(this, Grid.allGroupsExpanded != null ? Grid.allGroupsExpanded : rowArgs.Item1.Expanded))"></span>
</a>
</td>

2
Radzen.Blazor/RadzenDataGridRow.razor
View File

@@ -24,7 +24,7 @@
<span class="rz-column-title"></span>
@if (rowArgs.Item1.Expandable)
{
<a aria-label="@Grid.ExpandChildItemAriaLabel" href="javascript:void(0)" @onclick="@(_ => Grid.ExpandItem(Item))" @onclick:stopPropagation>
<a aria-label="@Grid.ExpandChildItemAriaLabel" @onclick:preventDefault="true" @onclick="@(_ => Grid.ExpandItem(Item))" @onclick:stopPropagation>
<span class="@(Grid.ExpandedItemStyle(Item))"></span>
</a>
}

8
Radzen.Blazor/RadzenDatePicker.razor
View File

@@ -35,10 +35,10 @@
@if (!TimeOnly)
{
<div class="rz-datepicker-header">
<a aria-label="@PrevMonthAriaLabel" href="javascript:void(0)" class="rz-datepicker-prev" @onclick="@(async () => { if (!Disabled) { try { if(CurrentDate.AddMonths(-1).Year >= YearFrom) {CurrentDate = CurrentDate.AddMonths(-1);}} catch (ArgumentOutOfRangeException) {}} })">
<a aria-label="@PrevMonthAriaLabel" @onclick:preventDefault="true" class="rz-datepicker-prev" @onclick="@(async () => { if (!Disabled) { try { if(CurrentDate.AddMonths(-1).Year >= YearFrom) {CurrentDate = CurrentDate.AddMonths(-1);}} catch (ArgumentOutOfRangeException) {}} })">
<span class="rz-datepicker-prev-icon rzi rzi-chevron-left"></span>
</a>
<a aria-label="@NextMonthAriaLabel" href="javascript:void(0)" class="rz-datepicker-next" @onclick="@(async () => { if (!Disabled) { try { if(CurrentDate.AddMonths(1).Year <= YearTo) {CurrentDate = CurrentDate.AddMonths(1);}} catch (ArgumentOutOfRangeException) {} } })">
<a aria-label="@NextMonthAriaLabel" @onclick:preventDefault="true" class="rz-datepicker-next" @onclick="@(async () => { if (!Disabled) { try { if(CurrentDate.AddMonths(1).Year <= YearTo) {CurrentDate = CurrentDate.AddMonths(1);}} catch (ArgumentOutOfRangeException) {} } })">
<span class="rz-datepicker-next-icon rzi rzi-chevron-right"></span>
</a>
<div class="rz-datepicker-title">
@@ -122,11 +122,11 @@
@if (HourFormat == "12")
{
<div class="rz-ampm-picker">
<a aria-label="@ToggleAmPmAriaLabel" href="javascript:void(0)" @onclick="@ToggleAmPm">
<a aria-label="@ToggleAmPmAriaLabel" @onclick:preventDefault="true" @onclick="@ToggleAmPm">
<span class="rzi rzi-chevron-up"></span>
</a>
<span>@CurrentDate.ToString("tt")</span>
<a aria-label="@ToggleAmPmAriaLabel" href="javascript:void(0)" @onclick="@ToggleAmPm">
<a aria-label="@ToggleAmPmAriaLabel" @onclick:preventDefault="true" @onclick="@ToggleAmPm">
<span class="rzi rzi-chevron-down"></span>
</a>
</div>

2
Radzen.Blazor/RadzenDialog.razor
View File

@@ -22,7 +22,7 @@
<div class="rz-dialog-side-title" style="display: inline" id="rz-dialog-side-label">@((MarkupString)sideDialogOptions.Title)</div>
@if (sideDialogOptions.ShowClose)
{
<a aria-label="@CloseSideDialogAriaLabel" href="javascript:void(0)" class="rz-dialog-side-titlebar-close" @onclick="@(_ => Service.CloseSide(null))" role="button" tabindex="@sideDialogOptions.CloseTabIndex">
<a aria-label="@CloseSideDialogAriaLabel" @onclick:preventDefault="true" class="rz-dialog-side-titlebar-close" @onclick="@(_ => Service.CloseSide(null))" role="button" tabindex="@sideDialogOptions.CloseTabIndex">
<span class="rzi rzi-times"></span>
</a>
}

2
Radzen.Blazor/RadzenFieldset.razor
View File

@@ -9,7 +9,7 @@
@if (AllowCollapse)
{
<a title="@TitleAttribute()" aria-label="@AriaLabelAttribute()" href="javascript:void(0)"
<a title="@TitleAttribute()" aria-label="@AriaLabelAttribute()" @onclick:preventDefault="true"
aria-controls="rz-fieldset-0-content" aria-expanded="false" @onclick=@Toggle>
@if (collapsed)
{

2
Radzen.Blazor/RadzenGrid.razor
View File

@@ -293,7 +293,7 @@
<span class="rz-column-title"></span>
@if (rowArgs.Item1.Expandable)
{
<a href="javascript:void(0)" @onclick="@(_ => ExpandItem(item))">
<a @onclick:preventDefault="true" @onclick="@(_ => ExpandItem(item))">
<span class="@(ExpandedItemStyle(item))"></span>
</a>
}

10
Radzen.Blazor/RadzenPager.razor
View File

@@ -7,23 +7,23 @@
<span class="rz-paginator-summary">@((MarkupString)string.Format(PagingSummaryFormat, CurrentPage + 1, numberOfPages, Count))</span>
}
<a class="rz-paginator-first rz-paginator-element @((skip > 0) ? "": "rz-state-disabled")" href="javascript:void(0)" tabindex="-1" @onclick:preventDefault="true" @onclick="@(() => FirstPage())" aria-label="@FirstPageAriaLabel" title="@FirstPageTitle">
<a class="rz-paginator-first rz-paginator-element @((skip > 0) ? "": "rz-state-disabled")" tabindex="-1" @onclick:preventDefault="true" @onclick="@(() => FirstPage())" aria-label="@FirstPageAriaLabel" title="@FirstPageTitle">
<span class="rz-paginator-icon rzi rzi-step-backward"></span>
</a>
<a class="rz-paginator-prev rz-paginator-element @((skip > 0) ? "": "rz-state-disabled")" href="javascript:void(0)" tabindex="-1" @onclick:preventDefault="true" @onclick="@(() => PrevPage())" aria-label="@PrevPageAriaLabel" title="@PrevPageTitle">
<a class="rz-paginator-prev rz-paginator-element @((skip > 0) ? "": "rz-state-disabled")" tabindex="-1" @onclick:preventDefault="true" @onclick="@(() => PrevPage())" aria-label="@PrevPageAriaLabel" title="@PrevPageTitle">
<span class="rz-paginator-icon rzi rzi-caret-left"></span>
</a>
<span class="rz-paginator-pages">
@foreach (var i in Enumerable.Range(startPage, Math.Min(endPage + 1, PageNumbersCount)))
{
<a class="rz-paginator-page rz-paginator-element @((i == CurrentPage) ? "rz-state-active" : "")" href="javascript:void(0)" @onclick:preventDefault="true" @onclick="@(() => GoToPage(i))" aria-label="@string.Format(PageAriaLabelFormat, (i + 1).ToString())" title="@string.Format(PageTitleFormat, (i + 1).ToString())">@(i + 1)</a>
<a class="rz-paginator-page rz-paginator-element @((i == CurrentPage) ? "rz-state-active" : "")" @onclick:preventDefault="true" @onclick="@(() => GoToPage(i))" aria-label="@string.Format(PageAriaLabelFormat, (i + 1).ToString())" title="@string.Format(PageTitleFormat, (i + 1).ToString())">@(i + 1)</a>
}
</span>
<a class="rz-paginator-next rz-paginator-element @((CurrentPage != numberOfPages - 1) ? "": "rz-state-disabled")" href="javascript:void(0)" tabindex="0" @onclick:preventDefault="true" @onclick="@(() => NextPage())" aria-label="@NextPageAriaLabel" title="@NextPageTitle">
<a class="rz-paginator-next rz-paginator-element @((CurrentPage != numberOfPages - 1) ? "": "rz-state-disabled")" tabindex="0" @onclick:preventDefault="true" @onclick="@(() => NextPage())" aria-label="@NextPageAriaLabel" title="@NextPageTitle">
<span class="rz-paginator-icon rzi rzi-caret-right"></span>
</a>
<a class="rz-paginator-last rz-paginator-element @((CurrentPage != numberOfPages - 1) ? "": "rz-state-disabled")" href="javascript:void(0)" tabindex="0" @onclick:preventDefault="true" @onclick="@(() => LastPage())" aria-label="@LastPageAriaLabel" title="@LastPageTitle">
<a class="rz-paginator-last rz-paginator-element @((CurrentPage != numberOfPages - 1) ? "": "rz-state-disabled")" tabindex="0" @onclick:preventDefault="true" @onclick="@(() => LastPage())" aria-label="@LastPageAriaLabel" title="@LastPageTitle">
<span class="rz-paginator-icon rzi rzi-step-forward"></span>
</a>

4
Radzen.Blazor/RadzenPanel.razor
View File

@@ -22,7 +22,7 @@
@if (collapsed)
{
<a @onclick=@Toggle class="rz-panel-titlebar-icon rz-panel-titlebar-toggler"
href="javascript:void(0)" role="tab" id="rz-panel-0-label" aria-controls="rz-panel-0-content"
@onclick:preventDefault="true" role="tab" id="rz-panel-0-label" aria-controls="rz-panel-0-content"
aria-expanded="false" aria-label="@ExpandAriaLabel" title="@ExpandTitle">
<span class="rzi rzi-plus"></span>
</a>
@@ -30,7 +30,7 @@
else
{
<a @onclick=@Toggle class="rz-panel-titlebar-icon rz-panel-titlebar-toggler"
href="javascript:void(0)" role="tab" id="rz-panel-0-label" aria-controls="rz-panel-0-content"
@onclick:preventDefault="true" role="tab" id="rz-panel-0-label" aria-controls="rz-panel-0-content"
aria-expanded="true" aria-label="@CollapseAriaLabel" title="@CollapseTitle">
<span class="rzi rzi-minus"></span>
</a>

6
Radzen.Blazor/RadzenRating.razor
View File

@@ -6,7 +6,7 @@
<div @ref="@Element" @attributes="Attributes" class="@GetCssClass()" id="@GetId()">
@if (!ReadOnly)
{
<a aria-label="@ClearAriaLabel" href="javascript:void(0)" class="rz-rating-cancel" tabindex="@(Disabled ? "-1" : $"{TabIndex}")" @onclick="@(args => SetValue(0))" @onkeypress="@(async args => { if (args.Code == "Space") { await SetValue(0); } })">
<a aria-label="@ClearAriaLabel" @onclick:preventDefault="true" class="rz-rating-cancel" tabindex="@(Disabled ? "-1" : $"{TabIndex}")" @onclick="@(args => SetValue(0))" @onkeypress="@(async args => { if (args.Code == "Space") { await SetValue(0); } })">
<span class="rz-rating-icon rzi rzi-ban"></span>
</a>
}
@@ -15,13 +15,13 @@
@if (index <= Value)
{
<a aria-label="@RateAriaLabel" href="javascript:void(0)" tabindex="@(Disabled ? "-1" : $"{TabIndex}")" @onclick="@(args => SetValue(index))" @onkeypress="@(async args => { if (args.Code == "Space") { await SetValue(index); } })">
<a aria-label="@RateAriaLabel" @onclick:preventDefault="true" tabindex="@(Disabled ? "-1" : $"{TabIndex}")" @onclick="@(args => SetValue(index))" @onkeypress="@(async args => { if (args.Code == "Space") { await SetValue(index); } })">
<span class="rz-rating-icon rzi rzi-star"></span>
</a>
}
else
{
<a aria-label="@RateAriaLabel" href="javascript:void(0)" tabindex="@(Disabled ? "-1" : $"{TabIndex}")" @onclick="@(args => SetValue(index))" @onkeypress="@(async args => { if (args.Code == "Space") { await SetValue(index); } })">
<a aria-label="@RateAriaLabel" @onclick:preventDefault="true" tabindex="@(Disabled ? "-1" : $"{TabIndex}")" @onclick="@(args => SetValue(index))" @onkeypress="@(async args => { if (args.Code == "Space") { await SetValue(index); } })">
<span class="rz-rating-icon rzi rzi-star-o"></span>
</a>
}

6
Radzen.Blazor/RadzenSteps.razor
View File

@@ -24,7 +24,7 @@
<li class="@step.GetItemCssClass()" @attributes="step.Attributes" style="@step.Style">
<a title="@step.Title" aria-label="@step.AriaLabel"
@onclick="@(async (args) => { if (!step.Disabled) { await SelectStep(step, true); } })"
href="javascript:void(0)" class="rz-menuitem-link">
@onclick:preventDefault="true" class="rz-menuitem-link">
<span class="rz-steps-number">@(steps.Where(s => s.Visible).ToList().IndexOf(step) + 1)</span>
@if (step.Template != null)
{
@@ -61,10 +61,10 @@
<div class="rz-steps-buttons">
<a title="@PreviousTitle" arial-label="@PreviousAriaLabel"
class='@($"rz-steps-prev {(IsFirstVisibleStep() ? "rz-state-disabled" : "")}")' @onclick="@PrevStep"
href="javascript:void(0)"><i class="rzi">navigate_before</i>@PreviousText</a>
@onclick:preventDefault="true"><i class="rzi">navigate_before</i>@PreviousText</a>
<a title="@NextTitle" arial-label="@NextAriaLabel"
class='@($"rz-steps-next {(IsLastVisibleStep() ? "rz-state-disabled" : "")}")' @onclick="@NextStep"
href="javascript:void(0)">@NextText<i class="rzi">navigate_next</i></a>
@onclick:preventDefault="true">@NextText<i class="rzi">navigate_next</i></a>
</div>
}
</div>

2
Radzen.Blazor/RadzenTabsItem.razor
View File

@@ -3,7 +3,7 @@
@if (Tabs.RenderMode == TabRenderMode.Server ? Visible : true)
{
<li role="presentation" @attributes=@Attributes style=@getStyle() class=@ClassList>
<a @onclick=@OnClick role="tab" href="javascript: void(0)" tabindex=@(Disabled? "-1" : "0") id="@($"{Tabs.Id}-tabpanel-{Index}-label")"
<a @onclick=@OnClick role="tab" @onclick:preventDefault="true" tabindex=@(Disabled? "-1" : "0") id="@($"{Tabs.Id}-tabpanel-{Index}-label")"
aria-selected=@(IsSelected? "true" : "false") aria-controls="@($"{Tabs.Id}-tabpanel-{Index}")">
@if (!string.IsNullOrEmpty(Icon))
{

4
Radzen.Blazor/Rendering/DialogContainer.razor
View File

@@ -13,7 +13,7 @@
<div class="rz-dialog-title" style="display: inline" id="rz-dialog-0-label">@((MarkupString)Dialog.Title)</div>
@if (Dialog.Options.ShowClose)
{
<a href="javascript:void(0)" @onclick=@Close role="button" class="rz-dialog-titlebar-icon rz-dialog-titlebar-close" tabindex=@Dialog.Options.CloseTabIndex>
<a @onclick:preventDefault="true" @onclick=@Close role="button" class="rz-dialog-titlebar-icon rz-dialog-titlebar-close" tabindex=@Dialog.Options.CloseTabIndex>
<span class="rzi rzi-times"></span>
</a>
}
@@ -26,7 +26,7 @@
<div class="rz-dialog-title" style="display: inline" id="rz-dialog-0-label">@((MarkupString)Dialog.Title)</div>
@if (Dialog.Options.ShowClose)
{
<a href="javascript:void(0)" @onclick=@Close role="button" class="rz-dialog-titlebar-icon rz-dialog-titlebar-close" tabindex=@Dialog.Options.CloseTabIndex>
<a @onclick:preventDefault="true" @onclick=@Close role="button" class="rz-dialog-titlebar-icon rz-dialog-titlebar-close" tabindex=@Dialog.Options.CloseTabIndex>
<span class="rzi rzi-times"></span>
</a>
}