Fix settings: dark borders, update dropdown z-index, dead accessibility link (#6528)

This commit is contained in:
Anthony Stirling
2026-06-04 17:59:22 +01:00
committed by GitHub
parent 353b5c807c
commit 69ee39fa6e
4 changed files with 15 additions and 15 deletions
@@ -44,13 +44,10 @@ export default function Footer({
// Default URLs // Default URLs
const defaultTermsUrl = "https://www.stirling.com/terms"; const defaultTermsUrl = "https://www.stirling.com/terms";
const defaultPrivacyUrl = "https://www.stirling.com/privacy"; const defaultPrivacyUrl = "https://www.stirling.com/privacy";
const defaultAccessibilityUrl = "https://www.stirling.com/accessibility";
// Use provided URLs or fall back to defaults // Use provided URLs or fall back to defaults
const finalTermsUrl = finalTermsAndConditions || defaultTermsUrl; const finalTermsUrl = finalTermsAndConditions || defaultTermsUrl;
const finalPrivacyUrl = finalPrivacyPolicy || defaultPrivacyUrl; const finalPrivacyUrl = finalPrivacyPolicy || defaultPrivacyUrl;
const finalAccessibilityUrl =
finalAccessibilityStatement || defaultAccessibilityUrl;
// Helper to check if a value is valid (not null/undefined/empty string) // Helper to check if a value is valid (not null/undefined/empty string)
const isValidLink = (link?: string) => link && link.trim().length > 0; const isValidLink = (link?: string) => link && link.trim().length > 0;
@@ -121,14 +118,16 @@ export default function Footer({
> >
{t("footer.issues", "GitHub")} {t("footer.issues", "GitHub")}
</a> </a>
<a {isValidLink(finalAccessibilityStatement) && (
className="footer-link px-3" <a
target="_blank" className="footer-link px-3"
rel="noopener noreferrer" target="_blank"
href={finalAccessibilityUrl} rel="noopener noreferrer"
> href={finalAccessibilityStatement}
{t("legal.accessibility", "Accessibility")} >
</a> {t("legal.accessibility", "Accessibility")}
</a>
)}
{isValidLink(finalCookiePolicy) && ( {isValidLink(finalCookiePolicy) && (
<a <a
className="footer-link px-3" className="footer-link px-3"
@@ -451,6 +451,10 @@ const GeneralSection: React.FC<GeneralSectionProps> = ({
}, },
]} ]}
maw={360} maw={360}
comboboxProps={{
withinPortal: true,
zIndex: Z_INDEX_OVER_CONFIG_MODAL,
}}
/> />
</Stack> </Stack>
)} )}
+1 -1
View File
@@ -515,7 +515,7 @@
/* Always-dark text (for use on light backgrounds like alerts) - does not change in dark mode */ /* Always-dark text (for use on light backgrounds like alerts) - does not change in dark mode */
--text-always-dark: #1f2937; --text-always-dark: #1f2937;
--text-always-dark-muted: #6b7280; --text-always-dark-muted: #6b7280;
--border-subtle: #2a2f36; --border-subtle: rgba(255, 255, 255, 0.08);
--border-default: #3a4047; --border-default: #3a4047;
--border-strong: #4b5563; --border-strong: #4b5563;
--hover-bg: #374151; --hover-bg: #374151;
@@ -104,9 +104,6 @@ test.describe("2. Main Dashboard / Home Page", () => {
await expect(page.getByText("GitHub").first()).toBeVisible({ await expect(page.getByText("GitHub").first()).toBeVisible({
timeout: 10000, timeout: 10000,
}); });
await expect(page.getByText("Accessibility").first()).toBeVisible({
timeout: 10000,
});
const githubLink = page const githubLink = page
.locator('a[href*="github.com/Stirling-Tools/Stirling-PDF"]') .locator('a[href*="github.com/Stirling-Tools/Stirling-PDF"]')