richard pushed to branch tor-browser-115.2.0esr-13.0-1 at The Tor Project / Applications / Tor Browser
Commits:
-
e35aaaa8
by Henry Wilkes at 2023-08-24T20:21:48+00:00
13 changed files:
- browser/components/torpreferences/content/bridgeQrDialog.jsm
- browser/components/torpreferences/content/bridgeQrDialog.xhtml
- browser/components/torpreferences/content/builtinBridgeDialog.jsm
- browser/components/torpreferences/content/builtinBridgeDialog.xhtml
- browser/components/torpreferences/content/connectionSettingsDialog.jsm
- browser/components/torpreferences/content/connectionSettingsDialog.xhtml
- browser/components/torpreferences/content/provideBridgeDialog.jsm
- browser/components/torpreferences/content/provideBridgeDialog.xhtml
- browser/components/torpreferences/content/requestBridgeDialog.jsm
- browser/components/torpreferences/content/requestBridgeDialog.xhtml
- browser/components/torpreferences/content/torLogDialog.jsm
- browser/components/torpreferences/content/torLogDialog.xhtml
- browser/components/torpreferences/content/torPreferences.css
Changes:
| ... | ... | @@ -34,10 +34,7 @@ class BridgeQrDialog { |
| 34 | 34 | }
|
| 35 | 35 | |
| 36 | 36 | init(window, dialog) {
|
| 37 | - // Defer to later until Firefox has populated the dialog with all our elements
|
|
| 38 | - window.setTimeout(() => {
|
|
| 39 | - this._populateXUL(window, dialog);
|
|
| 40 | - }, 0);
|
|
| 37 | + this._populateXUL(window, dialog);
|
|
| 41 | 38 | }
|
| 42 | 39 | |
| 43 | 40 | openDialog(gSubDialog, bridgeString) {
|
| ... | ... | @@ -21,8 +21,10 @@ |
| 21 | 21 | "use strict";
|
| 22 | 22 | |
| 23 | 23 | let dialogObject = window.arguments[0];
|
| 24 | - let dialogElement = document.getElementById("bridgeQr-dialog");
|
|
| 25 | - dialogObject.init(window, dialogElement);
|
|
| 24 | + document.addEventListener("DOMContentLoaded", () => {
|
|
| 25 | + let dialogElement = document.getElementById("bridgeQr-dialog");
|
|
| 26 | + dialogObject.init(window, dialogElement);
|
|
| 27 | + });
|
|
| 26 | 28 | ]]>
|
| 27 | 29 | </script>
|
| 28 | 30 | </dialog>
|
| ... | ... | @@ -95,10 +95,6 @@ class BuiltinBridgeDialog { |
| 95 | 95 | );
|
| 96 | 96 | });
|
| 97 | 97 | |
| 98 | - // Hack: see the CSS
|
|
| 99 | - dialog.style.minWidth = "0";
|
|
| 100 | - dialog.style.minHeight = "0";
|
|
| 101 | - |
|
| 102 | 98 | this._acceptButton = dialog.getButton("accept");
|
| 103 | 99 | |
| 104 | 100 | Services.obs.addObserver(this, TorConnectTopics.StateChange);
|
| ... | ... | @@ -129,10 +125,7 @@ class BuiltinBridgeDialog { |
| 129 | 125 | }
|
| 130 | 126 | |
| 131 | 127 | init(window, aDialog) {
|
| 132 | - // defer to later until firefox has populated the dialog with all our elements
|
|
| 133 | - window.setTimeout(() => {
|
|
| 134 | - this._populateXUL(window, aDialog);
|
|
| 135 | - }, 0);
|
|
| 128 | + this._populateXUL(window, aDialog);
|
|
| 136 | 129 | }
|
| 137 | 130 | |
| 138 | 131 | close() {
|
| ... | ... | @@ -83,8 +83,10 @@ |
| 83 | 83 | "use strict";
|
| 84 | 84 | |
| 85 | 85 | let builtinBridgeDialog = window.arguments[0];
|
| 86 | - let dialog = document.getElementById("torPreferences-builtinBridge-dialog");
|
|
| 87 | - builtinBridgeDialog.init(window, dialog);
|
|
| 86 | + document.addEventListener("DOMContentLoaded", () => {
|
|
| 87 | + let dialog = document.getElementById("torPreferences-builtinBridge-dialog");
|
|
| 88 | + builtinBridgeDialog.init(window, dialog);
|
|
| 89 | + });
|
|
| 88 | 90 | ]]>
|
| 89 | 91 | </script>
|
| 90 | 92 | </dialog>
|
| ... | ... | @@ -213,10 +213,6 @@ class ConnectionSettingsDialog { |
| 213 | 213 | this._dialog.addEventListener("dialogaccept", e => {
|
| 214 | 214 | this._applySettings();
|
| 215 | 215 | });
|
| 216 | - |
|
| 217 | - // Hack: see the CSS
|
|
| 218 | - this._dialog.style.minWidth = "0";
|
|
| 219 | - this._dialog.style.minHeight = "0";
|
|
| 220 | 216 | }
|
| 221 | 217 | |
| 222 | 218 | // callback when proxy is toggled
|
| ... | ... | @@ -380,10 +376,7 @@ class ConnectionSettingsDialog { |
| 380 | 376 | }
|
| 381 | 377 | |
| 382 | 378 | init(window, aDialog) {
|
| 383 | - // defer to later until firefox has populated the dialog with all our elements
|
|
| 384 | - window.setTimeout(() => {
|
|
| 385 | - this._populateXUL(window, aDialog);
|
|
| 386 | - }, 0);
|
|
| 379 | + this._populateXUL(window, aDialog);
|
|
| 387 | 380 | }
|
| 388 | 381 | |
| 389 | 382 | openDialog(gSubDialog) {
|
| ... | ... | @@ -83,8 +83,10 @@ |
| 83 | 83 | "use strict";
|
| 84 | 84 | |
| 85 | 85 | let connectionSettingsDialog = window.arguments[0];
|
| 86 | - let dialog = document.getElementById("torPreferences-connection-dialog");
|
|
| 87 | - connectionSettingsDialog.init(window, dialog);
|
|
| 86 | + document.addEventListener("DOMContentLoaded", () => {
|
|
| 87 | + let dialog = document.getElementById("torPreferences-connection-dialog");
|
|
| 88 | + connectionSettingsDialog.init(window, dialog);
|
|
| 89 | + });
|
|
| 88 | 90 | ]]>
|
| 89 | 91 | </script>
|
| 90 | 92 | </dialog>
|
| ... | ... | @@ -99,10 +99,7 @@ class ProvideBridgeDialog { |
| 99 | 99 | }
|
| 100 | 100 | |
| 101 | 101 | init(window, aDialog) {
|
| 102 | - // defer to later until firefox has populated the dialog with all our elements
|
|
| 103 | - window.setTimeout(() => {
|
|
| 104 | - this._populateXUL(window, aDialog);
|
|
| 105 | - }, 0);
|
|
| 102 | + this._populateXUL(window, aDialog);
|
|
| 106 | 103 | }
|
| 107 | 104 | |
| 108 | 105 | close() {
|
| ... | ... | @@ -24,8 +24,10 @@ |
| 24 | 24 | "use strict";
|
| 25 | 25 | |
| 26 | 26 | let provideBridgeDialog = window.arguments[0];
|
| 27 | - let dialog = document.getElementById("torPreferences-provideBridge-dialog");
|
|
| 28 | - provideBridgeDialog.init(window, dialog);
|
|
| 27 | + document.addEventListener("DOMContentLoaded", () => {
|
|
| 28 | + let dialog = document.getElementById("torPreferences-provideBridge-dialog");
|
|
| 29 | + provideBridgeDialog.init(window, dialog);
|
|
| 30 | + });
|
|
| 29 | 31 | ]]>
|
| 30 | 32 | </script>
|
| 31 | 33 | </dialog>
|
| ... | ... | @@ -157,10 +157,7 @@ class RequestBridgeDialog { |
| 157 | 157 | }
|
| 158 | 158 | |
| 159 | 159 | init(window, dialog) {
|
| 160 | - // defer to later until firefox has populated the dialog with all our elements
|
|
| 161 | - window.setTimeout(() => {
|
|
| 162 | - this._populateXUL(window, dialog);
|
|
| 163 | - }, 0);
|
|
| 160 | + this._populateXUL(window, dialog);
|
|
| 164 | 161 | }
|
| 165 | 162 | |
| 166 | 163 | close() {
|
| ... | ... | @@ -35,8 +35,10 @@ |
| 35 | 35 | "use strict";
|
| 36 | 36 | |
| 37 | 37 | let requestBridgeDialog = window.arguments[0];
|
| 38 | - let dialog = document.getElementById("torPreferences-requestBridge-dialog");
|
|
| 39 | - requestBridgeDialog.init(window, dialog);
|
|
| 38 | + document.addEventListener("DOMContentLoaded", () => {
|
|
| 39 | + let dialog = document.getElementById("torPreferences-requestBridge-dialog");
|
|
| 40 | + requestBridgeDialog.init(window, dialog);
|
|
| 41 | + });
|
|
| 40 | 42 | ]]>
|
| 41 | 43 | </script>
|
| 42 | 44 | </dialog>
|
| ... | ... | @@ -60,10 +60,7 @@ class TorLogDialog { |
| 60 | 60 | }
|
| 61 | 61 | |
| 62 | 62 | init(window, aDialog) {
|
| 63 | - // defer to later until firefox has populated the dialog with all our elements
|
|
| 64 | - window.setTimeout(() => {
|
|
| 65 | - this._populateXUL(aDialog);
|
|
| 66 | - }, 0);
|
|
| 63 | + this._populateXUL(aDialog);
|
|
| 67 | 64 | }
|
| 68 | 65 | |
| 69 | 66 | copyTorLog() {
|
| ... | ... | @@ -19,8 +19,10 @@ |
| 19 | 19 | "use strict";
|
| 20 | 20 | |
| 21 | 21 | let torLogDialog = window.arguments[0];
|
| 22 | - let dialog = document.getElementById("torPreferences-torLog-dialog");
|
|
| 23 | - torLogDialog.init(window, dialog);
|
|
| 22 | + document.addEventListener("DOMContentLoaded", () => {
|
|
| 23 | + let dialog = document.getElementById("torPreferences-torLog-dialog");
|
|
| 24 | + torLogDialog.init(window, dialog);
|
|
| 25 | + });
|
|
| 24 | 26 | ]]>
|
| 25 | 27 | </script>
|
| 26 | 28 | </dialog>
|
| ... | ... | @@ -441,22 +441,6 @@ dialog#torPreferences-requestBridge-dialog > hbox { |
| 441 | 441 | }
|
| 442 | 442 | |
| 443 | 443 | /* Builtin bridge dialog */
|
| 444 | -#torPreferences-builtinBridge-dialog {
|
|
| 445 | - /* Hack: ask XUL to load with a lot of space, then we will remove the minimum
|
|
| 446 | - sizes after we populate the string. Users with high resolutions should see a
|
|
| 447 | - big dialog, but with all the needed buttons, users with small resolutions
|
|
| 448 | - should see scrollbars.
|
|
| 449 | - Known problems:
|
|
| 450 | - 1) the hack works only if a window is *already* small: if it is big, and then
|
|
| 451 | - it is resized, the dialog will not shrink;
|
|
| 452 | - 2) horizontal scrollbars are never added.
|
|
| 453 | - The proper solution for this will be loading the localized string with
|
|
| 454 | - Mozilla's way.
|
|
| 455 | - See also https://gitlab.torproject.org/tpo/applications/tor-browser/-/issues/41044 */
|
|
| 456 | - min-width: 700px;
|
|
| 457 | - min-height: 550px;
|
|
| 458 | -}
|
|
| 459 | - |
|
| 460 | 444 | #torPreferences-builtinBridge-header {
|
| 461 | 445 | margin: 8px 0 10px 0;
|
| 462 | 446 | }
|
| ... | ... | @@ -504,12 +488,6 @@ groupbox#torPreferences-bridges-group textarea { |
| 504 | 488 | }
|
| 505 | 489 | |
| 506 | 490 | /* Connection settings dialog */
|
| 507 | -#torPreferences-connection-dialog {
|
|
| 508 | - /* Hack: like the built-in dialog */
|
|
| 509 | - min-width: 700px;
|
|
| 510 | - min-height: 550px;
|
|
| 511 | -}
|
|
| 512 | - |
|
| 513 | 491 | #torPreferences-connection-header {
|
| 514 | 492 | margin: 4px 0 14px 0;
|
| 515 | 493 | }
|