Feodor2 / Mypal68

web browser
Other
630 stars 39 forks source link

68.14.4.b SSE - Popup screen about closing multiple tabs shows even if only one tab #531

Open Robbbert opened 1 month ago

Robbbert commented 1 month ago

Using version: 68.14.4.b SSE

On: XP3 SP3 with latest (afaik) updates.

If I have one tab open, when I close the browser, a box warns me about closing multiple tabs. This should not happen.

I had to untick the warning in the box to get rid of it.

This is a brand new installation, never used MyPal before on this computer.

Yggdrax commented 1 month ago

This is a known issue of 68.14.4 and there's already a pull request to fix it: https://github.com/Feodor2/Mypal68/pull/507

Feodor2 commented 1 month ago

So you want this as was before a quit warning but only not on one page, I had done this asked otherswise a quit warning no matter pages. I shall consider.

zanud commented 1 month ago

I shall consider.

As for me, it makes sense to use two separate preferences: an old one ("Warn me when I attempt to close multiple tabs", browser.tabs.warnOnClose) and a new one ("Warn me when I attempt to close last tab", browser.tabs.warnOnCloseLastTab). Thus everybody will be satisfied.

NS-Clone commented 1 month ago

как по мне это пральное поведение для браузера намного больше гемора когда браузер молча закрываетсо по поводу и без

а если езе включить force classic theme для браузера то когда браузер не отвечает венда натягивает тему с большими кнопками других размеров и в итоге на месте развернуть оказываетсо закрыть на месте свернуть оказываетсо развернуть :rofl: я так n раз прибивал браузер пытаясь его наверно свернуть? пушо умя вообще стоят широкие кнопки а не отвечающий браузер не выдает вообще подтверждения а закрывается не обработав события клика лично 2024-10-27_173323

zanud commented 1 month ago

И ещё вот такой прикол в некоторых ситуациях наблюдать можно: 0tabs - предупреждение о закрытии окна, в котором ноль(!) вкладок.

Это уже второй раз я такое вижу. Первый был пару дней назад. Я тогда попытался повторить, но ничего не вышло. Сегодняшняя ситуация была аналогична прошлой, поэтому думаю, что найти последовательность действий для воспроизведения удастся. (Если, конечно, это нужно.)

Robbbert commented 3 weeks ago

That's the thing. It says "multiple tabs", in other words 2 or more. 0 and 1 are not "multiple" tabs.

If it's correct for the warning to appear for one tab, then change the wording.

zanud commented 3 weeks ago

If it's correct for the warning to appear for one tab, then change the wording.

Wording is only a minor part of problem. Check mark is a bigger one. Currently it disables this warning regardless of the number of tabs, whereas there are users, who want see the warning for multiple tabs and don't want for one tab (IOW, they prefer old browser's behavior). As I wrote above, this requires introduction of new parameter in about:config (and handling this parameter).

Feodor2 commented 1 week ago

I think to change this one setting only for numeric 0 no warning, 1 - one tab, - 2 tabs and such. The defaults will be 1

IPeluchito commented 3 days ago

@Feodor2 Doesn't work... 0=shows warning, "n"=shows warning with more than "n" tabs open...Mypal 68.14.5

Personally, I have always found the warning irritating but its function is understandable....but in Mypal68 there is already the option "Warn you when quitting the browser"... wouldn't it be better to eliminate everything related to warnings when exiting with open tabs and leave only the option "Warn you when quitting the browser" making it independent of "Restore previous session"? I don't see any point in having both that literally do the same thing... :S