Skip to content

Commit

Permalink
Fix careers listings filtering (Fixes mozilla#12490)
Browse files Browse the repository at this point in the history
  • Loading branch information
alexgibson committed Dec 16, 2022
1 parent d45bf0e commit 4ed8b18
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions media/js/careers/listings/filters.es6.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ PositionFilters.prototype = {
for (let i = 0; i < positions.length; i++) {
const data = positions.item(i).dataset[field];

if (data.indexOf(value + ',') === -1) {
if (data.indexOf(value) === -1) {
positions.item(i).classList.add('hidden');
}
}
Expand All @@ -139,7 +139,7 @@ PositionFilters.prototype = {
const positions = this.positionTable.getElementsByClassName('position');

for (let i = 0; i < positions.length; i++) {
const data = positions.item(i).dataset.location;
const data = positions.item(i).dataset.location + ',';

// When user selects 'Remote' only list jobs explicitly marked
// Remote otherwise list jobs matching value (which is a mozilla
Expand Down

0 comments on commit 4ed8b18

Please sign in to comment.