Skip to content

Commit f93e002

Browse files
committed
Remove console warnings for missing icons in ComboBox, TechReport, and TableLinked components
1 parent c84d0fa commit f93e002

3 files changed

Lines changed: 1 addition & 11 deletions

File tree

src/js/techreport/combobox.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ class ComboBox {
3535
logo.setAttribute('src', `https://cdn.httparchive.org/v1/static/icons/${icon}`);
3636
logo.setAttribute('loading', 'lazy');
3737
option.append(logo);
38-
} else {
39-
console.warn('No icon found for technology:', row.technology);
4038
}
4139
if(this.selected.includes(row.technology)) {
4240
option.setAttribute('aria-selected', true);
@@ -209,8 +207,6 @@ class ComboBox {
209207
appIcon.setAttribute('alt', '');
210208
appIcon.classList.add('logo');
211209
deleteSelection.append(appIcon);
212-
} else {
213-
console.warn('No icon found for technology:', name);
214210
}
215211

216212
/* Add the delete icon */

src/js/techreport/index.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@ class TechReport {
299299
// Get the information about the selected technology
300300
getTechInfo() {
301301
const technologies = this.filters.app;
302-
const technology = technologies.map(encodeURIComponent).join(':');
302+
const technology = technologies.map(encodeURIComponent).join(',');
303303
const url = `${Constants.apiBase}/technologies?technology=${technology}`;
304304

305305
fetch(url)
@@ -315,8 +315,6 @@ class TechReport {
315315
DrilldownHeader.setDescription(techInfo.description);
316316
if (techInfo.icon) {
317317
DrilldownHeader.setIcon(techInfo.icon);
318-
} else {
319-
console.warn('No icon found for technology:', technologies[0]);
320318
}
321319
});
322320
}
@@ -412,8 +410,6 @@ class TechReport {
412410
DrilldownHeader.update(this.filters);
413411
if (icon) {
414412
DrilldownHeader.setIcon(`${encodeURI(icon)}`);
415-
} else {
416-
console.warn('No icon found for technology:', app);
417413
}
418414

419415
if(data && data[app]) {

src/js/techreport/tableLinked.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,8 +113,6 @@ class TableLinked {
113113
img.setAttribute('style', `background-image: url(${imgUrl})`);
114114
img.classList.add('app-img');
115115
wrapper.append(img);
116-
} else {
117-
console.warn('No icon found for technology:', app);
118116
}
119117

120118
const formattedApp = DataUtils.formatAppName(app);

0 commit comments

Comments
 (0)