Skip to content

Commit 347ed18

Browse files
committed
Clean up settings UI and move it to a collapsible element at the top.
1 parent 3c84e0f commit 347ed18

File tree

1 file changed

+61
-25
lines changed

1 file changed

+61
-25
lines changed

src/app.ts

Lines changed: 61 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,44 @@ export class App extends LitElement {
240240
font-size: 0.8rem;
241241
color: var(--text-secondary);
242242
}
243+
244+
/* Settings section */
245+
#settings-content {
246+
padding: 1.5rem 2rem;
247+
background: var(--bg-secondary, #232323);
248+
border-radius: 10px;
249+
box-shadow: 0 2px 8px rgba(0, 0, 0, 0.04);
250+
display: grid;
251+
grid-template-columns: max-content 1fr;
252+
gap: 1rem 1.5rem;
253+
align-items: center;
254+
max-width: 480px;
255+
}
256+
#settings-content label {
257+
justify-self: end;
258+
margin-right: 0.5rem;
259+
font-weight: 500;
260+
}
261+
#settings-content select,
262+
#settings-content input[type="checkbox"] {
263+
margin-left: 0.5rem;
264+
font-size: 1rem;
265+
}
266+
#settings-content input[type="checkbox"] {
267+
transform: scale(1.2);
268+
margin-right: 0.5rem;
269+
}
270+
@media (max-width: 600px) {
271+
#settings-content {
272+
grid-template-columns: 1fr;
273+
padding: 1rem;
274+
max-width: 100%;
275+
}
276+
#settings-content label {
277+
justify-self: start;
278+
margin-right: 0;
279+
}
280+
}
243281
`;
244282

245283
/**
@@ -561,38 +599,32 @@ export class App extends LitElement {
561599
>
562600
for more information.
563601
</p>
564-
565-
${this.renderMainContent()}
566-
567-
<footer>
568-
<div id="settings">
602+
<details>
603+
<summary>UI Settings</summary>
604+
<div id="settings-content">
569605
<label for="rs-env">Remote Settings Environment:</label>
570606
<select id="rs-env" @change=${this.handleRSEnvChange}>
571607
<option value="prod" ?selected=${this.rsEnv === "prod"}>Prod</option>
572608
<option value="stage" ?selected=${this.rsEnv === "stage"}>Stage</option>
573609
<option value="dev" ?selected=${this.rsEnv === "dev"}>Dev</option>
574610
</select>
575-
<br />
576-
<div id="firefox-version-picker">
577-
<label for="fx-version">Firefox Version:</label>
578-
<select
579-
id="fx-version"
580-
@change=${this.handleFirefoxVersionFilterChange}
581-
?disabled=${!this.firefoxVersions}
611+
<label for="fx-version">Firefox Version:</label>
612+
<select
613+
id="fx-version"
614+
@change=${this.handleFirefoxVersionFilterChange}
615+
?disabled=${!this.firefoxVersions}
616+
>
617+
<option value="" ?selected=${this.filterFirefoxChannel === null}>All</option>
618+
<option value="nightly" ?selected=${this.filterFirefoxChannel === "nightly"}
619+
>Nightly ${this.firefoxVersions?.nightly}</option
620+
>
621+
<option value="beta" ?selected=${this.filterFirefoxChannel === "beta"}
622+
>Beta ${this.firefoxVersions?.beta}</option
623+
>
624+
<option value="release" ?selected=${this.filterFirefoxChannel === "release"}
625+
>Release ${this.firefoxVersions?.release}</option
582626
>
583-
<option value="" ?selected=${this.filterFirefoxChannel === null}>All</option>
584-
<option value="nightly" ?selected=${this.filterFirefoxChannel === "nightly"}
585-
>Nightly ${this.firefoxVersions?.nightly}</option
586-
>
587-
<option value="beta" ?selected=${this.filterFirefoxChannel === "beta"}
588-
>Beta ${this.firefoxVersions?.beta}</option
589-
>
590-
<option value="release" ?selected=${this.filterFirefoxChannel === "release"}
591-
>Release ${this.firefoxVersions?.release}</option
592-
>
593-
</select>
594-
</div>
595-
<br />
627+
</select>
596628
<label for="rs-env-preview">Include changes pending review:</label>
597629
<input
598630
id="rs-env-preview"
@@ -601,7 +633,11 @@ export class App extends LitElement {
601633
@change=${this.handleRSEnvChange}
602634
/>
603635
</div>
636+
</details>
637+
638+
${this.renderMainContent()}
604639
640+
<footer>
605641
<p>
606642
Data source:
607643
${(() => {

0 commit comments

Comments
 (0)