From a3cb05334a4bfe7f33e6770202ce3d29fbdaf21f Mon Sep 17 00:00:00 2001 From: Brad Simpson Date: Thu, 7 Sep 2023 12:48:30 -0600 Subject: [PATCH] onButtonClick: Lookup model props instead of checking for CSS classes --- js/PageNavView.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/js/PageNavView.js b/js/PageNavView.js index da5ed9e..b01d49c 100644 --- a/js/PageNavView.js +++ b/js/PageNavView.js @@ -29,13 +29,14 @@ class PageNavView extends ComponentView { onButtonClick(event) { const $target = $(event.currentTarget); - const isLocked = $target.hasClass('is-locked'); - const isSelected = $target.attr('aria-current'); + const index = $target.data('item-index'); + const item = this.model.get('_items')[index]; + const isLocked = item._isHidden || item._isLocked; + const isSelected = item._isCurrent; if (isLocked || isSelected) return; - const id = $target.data('id'); - this.navigateTo(id); + this.navigateTo(item._id); }; setupTooltips() {