From a6e29ab43c9b9b6b8ebb82e79bb0b6daf9e60216 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 16 Apr 2021 09:49:11 +0200 Subject: [PATCH] Remove the special handling of "errorFontLoadNative" and "errorFontMissing" in `PDFViewerApplication.fallback` (PR 12931 follow-up, PR 12995 follow-up) According to a decision by UX and PM, please see https://bugzilla.mozilla.org/show_bug.cgi?id=1705060#c2 (and implemented in https://bugzilla.mozilla.org/show_bug.cgi?id=1705327), we no longer show the notification-bar in Firefox. Hence there's (obviously) no need to ignore certain `featureId`s in `PDFViewerApplication.fallback` anymore. --- web/app.js | 7 ------- 1 file changed, 7 deletions(-) diff --git a/web/app.js b/web/app.js index f8e817dd4..8bbe331cb 100644 --- a/web/app.js +++ b/web/app.js @@ -1048,13 +1048,6 @@ const PDFViewerApplication = { featureId, }); - // Don't show the fallback bar for things that are *very* unlikely to cause - // user-visible errors, to avoid bothering the user unnecessarily. - switch (featureId) { - case UNSUPPORTED_FEATURES.errorFontLoadNative: - case UNSUPPORTED_FEATURES.errorFontMissing: - return; - } // Only trigger the fallback once so we don't spam the user with messages // for one PDF. if (this._fellback) {