Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Съехал ползунок включения/выключения расширенения #230

Open
Narkoman4ek opened this issue Mar 29, 2024 · 13 comments

Comments

@Narkoman4ek
Copy link

Narkoman4ek commented Mar 29, 2024

Ползунок вышел за границы окна расширения, в прошлой версии такого не было.
image
image
image
{
"manifest_version": 2,

"name": "MSG_extName",
"default_locale": "ru",
"description": "MSG_extDesc",
"version": "0.0.1.63",
"icons": {
"128": "icons/default-128.png"
},
"author": "anticensority+owners@googlegroups.com",

"permissions": [
"proxy"
, "alarms"
, "storage"
, "unlimitedStorage"
, "<all_urls>"
, "tabs"
, "contextMenus"
, "notifications"
, "webRequest", "webRequestBlocking", "webNavigation"
],
"minimum_chrome_version": "55.0.0.0",
"browser_specific_settings": {
"gecko": {
"strict_min_version": "91.1.0"
}
},

"background": {

"scripts": [
    "00-init-apis.js"
  
  , "11-error-handlers-api.js"
  , "12-errors-lib.js"
  , "13-http-lib.js"
  , "15-firefox-proxy-settings.js"
  , "20-ip-to-host-api.js"
  , "35-pac-kitchen-api.js"
  , "37-sync-pac-script-with-pac-provider-api.js"
  , "80-error-menu.js", "83-last-errors.js", "85-block-informer.js"
  , "70-menu-items.js"
  , "75-context-menus.js"
]

},

"browser_action": {
"default_title": "Этот сайт благословлён | Версия 1.63",
"default_popup": "/pages/options/index.html"
},
"options_ui": {
"page": "/pages/options/index.html",
"chrome_style": false
}

}
image
image
Не знаю нужно ли добавлять код на 296 строк, лучше не буду.

@Verity-Freedom
Copy link

Браузер Opera gx если что у него.

@Verity-Freedom
Copy link

Verity-Freedom commented Mar 29, 2024

У меня тоже такое пишет.

API returned error: [object Object]
Контекст:
_generated_background_page.html
Трассировка стека
00-init-apis.js:54 (checkChromeError)
00-init-apis.js:72 (анонимные функции)
(дальше вырезал)

@Verity-Freedom
Copy link

Verity-Freedom commented Mar 29, 2024

Строки 46-58:

46checkChromeError() {
47
48 // Chrome API calls your cb in a context different from the point of API
49 // method invokation.
50 const err = chrome.runtime.lastError || chrome.extension.lastError || self.lastError;
51 if (!err) {
52 return;
53 }
54 console.warn('API returned error:', err);
55 delete self.lastError;
56 return new Error(err.message); // Add stack.
57
58 },

Жёлтым выделена строка 54.

@Verity-Freedom
Copy link

У меня несмотря на наличие тех же ошибок ползунок не выехал, вероятно вещь специфичная для Оперы
image

Ну а Манифест v2 это язык а не ошибка тут уже браузер лютует называя его ошибкой.

@Verity-Freedom
Copy link

1.61 тоже написан на manifest v2 так что эта ошибка вероятно "самоисправится" при исправлении других ошибок или при одобрении расширения гуглмагазином.

@Verity-Freedom
Copy link

@Narkoman4ek

@Verity-Freedom
Copy link

Ну а дальше ждать, разработчики постоянно онлайн, версия в формате бетатестирования, они увидят.

@Verity-Freedom
Copy link

@ilyaigpetrov в принципе это и есть по сути проблемы, что выявило бетатестирование, в остальном расширение отрабатывает.

@Verity-Freedom
Copy link

@Narkoman4ek я думаю что съехавший ползунок вызывает сама надпись "ошибки" в рамках оперы. Попробуй установить версию 1.62 (тоже с Гитхаба), полагаю надпись тоже будет съехавшей. Найди её в последних релизах.

@MrDrone-ru
Copy link

MrDrone-ru commented Mar 29, 2024

В обычной опере те же ошибки, за исключением последней с API. И да, интерфейс на буржуйском и ничё не съехало, особенности локализации))
Как я понял, пофиг на них, а первый фрагмент можно снести (если знать, в каком он файле).

@Verity-Freedom
Copy link

Verity-Freedom commented Mar 29, 2024

первый фрагмент можно снести

Ты шаришь видимо, подскажи разрабам.

У меня на 1.62 те же ошибки за исключением ошибки с api. Плюс ничего не съехало ни там, ни там.

@MrDrone-ru
Copy link

MrDrone-ru commented Mar 29, 2024

шаришь видимо

сарказм?
Я думаю, что разрабу пофиг на оперу, они для хрома аддон выкладывают. Они в мелочах отличаются, отсюда и ошибки, в версии 1.61 с вебстора ошибок было 0, она конвертирется при установке.
А если в ошибке написано unrecognised key, значит опера не знает такого ключа и если ошибочный код удалить по идее ничё сломаться не должно.
Протестил, код был в манифесте, осталось 2))

@Verity-Freedom
Copy link

Найден фикс:
#233

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants