Skip to content

Commit 8f65746

Browse files
committed
Fixed first popup row not showing the initial site active preset.
1 parent 54dd9fa commit 8f65746

1 file changed

Lines changed: 30 additions & 30 deletions

File tree

src/ui/ui.js

Lines changed: 30 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,9 @@ var UI = (() => {
214214

215215
}
216216

217+
if (!UI.mobile) {
218+
UI.Sites.correctSize(presets);
219+
}
217220

218221
}
219222

@@ -246,42 +249,40 @@ var UI = (() => {
246249
return row;
247250
}
248251

249-
correctSize() {
250-
if (!(UI.mobile || document.getElementById("presets-sizer"))) {
251-
// adapt button to label if needed
252-
let presets = document.querySelector(".presets");
253-
let sizer = document.createElement("div");
254-
sizer.id = "presets-sizer";
255-
sizer.appendChild(presets.cloneNode(true));
256-
document.body.appendChild(sizer);
257-
setTimeout(async () => {
258-
let presetWidth = sizer.querySelector("input.preset").offsetWidth;
259-
let labelWidth = 0;
260-
for (let l of sizer.querySelectorAll("label.preset")) {
261-
let lw = l.offsetWidth;
262-
debug("lw", l.textContent, lw);
263-
if (lw > labelWidth) labelWidth = lw;
264-
}
252+
static correctSize(presets) {
253+
// adapt button to label if needed
254+
let sizer = document.createElement("div");
255+
sizer.id = "presets-sizer";
256+
sizer.appendChild(presets.cloneNode(true));
257+
document.body.appendChild(sizer);
258+
setTimeout(async () => {
259+
let presetWidth = sizer.querySelector("input.preset").offsetWidth;
260+
let labelWidth = 0;
261+
for (let l of sizer.querySelectorAll("label.preset")) {
262+
let lw = l.offsetWidth;
263+
debug("lw", l.textContent, lw);
264+
if (lw > labelWidth) labelWidth = lw;
265+
}
265266

266-
debug(`Preset: %s Label: %s`, presetWidth, labelWidth);
267-
labelWidth += 16;
268-
if (presetWidth < labelWidth) {
269-
for (let ss of document.styleSheets) {
270-
if (ss.href.endsWith("/ui.css")) {
271-
for (let r of ss.cssRules) {
272-
if (/input\.preset:checked.*min-width:/.test(r.cssText)) {
273-
r.style.minWidth = (labelWidth) + "px";
274-
break;
275-
}
267+
debug(`Preset: %s Label: %s`, presetWidth, labelWidth);
268+
labelWidth += 16;
269+
if (presetWidth < labelWidth) {
270+
for (let ss of document.styleSheets) {
271+
if (ss.href.endsWith("/ui.css")) {
272+
for (let r of ss.cssRules) {
273+
if (/input\.preset:checked.*min-width:/.test(r.cssText)) {
274+
r.style.minWidth = (labelWidth) + "px";
275+
break;
276276
}
277277
}
278278
}
279279
}
280+
}
280281

281-
sizer.style.display = "none";
282+
sizer.remove();
282283

283-
}, 100);
284-
}
284+
}, 100);
285+
UI.Sites.correctSize = () => {}; // just once, please!
285286
}
286287

287288
allSiteRows() {
@@ -446,7 +447,6 @@ var UI = (() => {
446447
root.addEventListener("change", this, true);
447448
root.wiredBy = this;
448449
}
449-
this.correctSize();
450450
return root;
451451
}
452452

0 commit comments

Comments
 (0)