Skip to content

Commit

Permalink
Merge pull request #14666 from TaneliTuomola/issue-14608
Browse files Browse the repository at this point in the history
fix #14608 p-ColumnFilter aria-label
  • Loading branch information
cetincakiroglu authored Feb 1, 2024
2 parents c2a6730 + 618cf72 commit 1abcc8e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/app/components/api/translationkeys.ts
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ export class TranslationKeys {
public static readonly PASSWORD_PROMPT = 'passwordPrompt';
public static readonly EMPTY_MESSAGE = 'emptyMessage';
public static readonly EMPTY_FILTER_MESSAGE = 'emptyFilterMessage';
public static readonly SHOW_FILTER_MENU = 'Show Filter Menu';
public static readonly HIDE_FILTER_MENU = 'Hide Filter Menu';
public static readonly SHOW_FILTER_MENU = 'showFilterMenu';
public static readonly HIDE_FILTER_MENU = 'hideFilterMenu';
public static readonly SELECTION_MESSAGE = '{0} items selected';
}
4 changes: 2 additions & 2 deletions src/app/components/table/table.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4947,7 +4947,7 @@ export class ReorderableRow implements AfterViewInit {
type="button"
class="p-column-filter-menu-button p-link"
aria-haspopup="true"
[attr.aria-label]="showMenuButtonLabel"
[attr.aria-label]="filterMenuButtonAriaLabel"
[attr.aria-controls]="overlayId"
[attr.aria-expanded]="overlayVisible"
[ngClass]="{ 'p-column-filter-menu-button-open': overlayVisible, 'p-column-filter-menu-button-active': hasFilter() }"
Expand Down Expand Up @@ -5279,7 +5279,7 @@ export class ColumnFilter implements AfterContentInit {
}

get filterMenuButtonAriaLabel() {
return this.config.translation ? (this.overlayVisible ? this.config.translation.aria.showFilterMenu : this.config.translation.aria.hideFilterMenu) : undefined;
return this.config.translation ? (this.overlayVisible ? this.config.translation.aria.hideFilterMenu : this.config.translation.aria.showFilterMenu) : undefined;
}

get removeRuleButtonAriaLabel() {
Expand Down

0 comments on commit 1abcc8e

Please sign in to comment.