Skip to content

Commit 69ee39f

Browse files
authored
Fix settings: dark borders, update dropdown z-index, dead accessibility link (#6528)
1 parent 353b5c8 commit 69ee39f

4 files changed

Lines changed: 15 additions & 15 deletions

File tree

frontend/editor/src/core/components/shared/Footer.tsx

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -44,13 +44,10 @@ export default function Footer({
4444
// Default URLs
4545
const defaultTermsUrl = "https://www.stirling.com/terms";
4646
const defaultPrivacyUrl = "https://www.stirling.com/privacy";
47-
const defaultAccessibilityUrl = "https://www.stirling.com/accessibility";
4847

4948
// Use provided URLs or fall back to defaults
5049
const finalTermsUrl = finalTermsAndConditions || defaultTermsUrl;
5150
const finalPrivacyUrl = finalPrivacyPolicy || defaultPrivacyUrl;
52-
const finalAccessibilityUrl =
53-
finalAccessibilityStatement || defaultAccessibilityUrl;
5451

5552
// Helper to check if a value is valid (not null/undefined/empty string)
5653
const isValidLink = (link?: string) => link && link.trim().length > 0;
@@ -121,14 +118,16 @@ export default function Footer({
121118
>
122119
{t("footer.issues", "GitHub")}
123120
</a>
124-
<a
125-
className="footer-link px-3"
126-
target="_blank"
127-
rel="noopener noreferrer"
128-
href={finalAccessibilityUrl}
129-
>
130-
{t("legal.accessibility", "Accessibility")}
131-
</a>
121+
{isValidLink(finalAccessibilityStatement) && (
122+
<a
123+
className="footer-link px-3"
124+
target="_blank"
125+
rel="noopener noreferrer"
126+
href={finalAccessibilityStatement}
127+
>
128+
{t("legal.accessibility", "Accessibility")}
129+
</a>
130+
)}
132131
{isValidLink(finalCookiePolicy) && (
133132
<a
134133
className="footer-link px-3"

frontend/editor/src/core/components/shared/config/configSections/GeneralSection.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -451,6 +451,10 @@ const GeneralSection: React.FC<GeneralSectionProps> = ({
451451
},
452452
]}
453453
maw={360}
454+
comboboxProps={{
455+
withinPortal: true,
456+
zIndex: Z_INDEX_OVER_CONFIG_MODAL,
457+
}}
454458
/>
455459
</Stack>
456460
)}

frontend/editor/src/core/styles/theme.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -515,7 +515,7 @@
515515
/* Always-dark text (for use on light backgrounds like alerts) - does not change in dark mode */
516516
--text-always-dark: #1f2937;
517517
--text-always-dark-muted: #6b7280;
518-
--border-subtle: #2a2f36;
518+
--border-subtle: rgba(255, 255, 255, 0.08);
519519
--border-default: #3a4047;
520520
--border-strong: #4b5563;
521521
--hover-bg: #374151;

frontend/editor/src/core/tests/stubbed/main-dashboard.spec.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,9 +104,6 @@ test.describe("2. Main Dashboard / Home Page", () => {
104104
await expect(page.getByText("GitHub").first()).toBeVisible({
105105
timeout: 10000,
106106
});
107-
await expect(page.getByText("Accessibility").first()).toBeVisible({
108-
timeout: 10000,
109-
});
110107

111108
const githubLink = page
112109
.locator('a[href*="github.com/Stirling-Tools/Stirling-PDF"]')

0 commit comments

Comments
 (0)