Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/x-logo-dark-mode.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
---
'@clerk/ui': patch
---

Fix the X (formerly Twitter) provider logo being nearly invisible in dark mode by recoloring it to match the foreground color, consistent with other monochrome provider icons.
1 change: 1 addition & 0 deletions packages/clerk-js/sandbox/scenarios/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -4,3 +4,4 @@ export { CheckoutSeats } from './checkout-seats';
export { OrgProfileSeatLimit } from './org-profile-seat-limit';
export { PricingTableSBB } from './pricing-table-sbb';
export { AnnualOnlyPlans } from './annual-only-plans';
export { XProviderEnabled } from './x-provider-enabled';
34 changes: 34 additions & 0 deletions packages/clerk-js/sandbox/scenarios/x-provider-enabled.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
import { clerkHandlers, EnvironmentService, type MockScenario, setClerkState } from '@clerk/msw';

/**
* Signed-out sign-in page with the X / Twitter social provider enabled.
*
* The shared sandbox instance does not have X enabled, so this scenario mocks
* the environment to add it — useful for verifying the X provider logo (e.g. how
* it renders in dark mode). Activate it from the console with:
*
* scenario.setScenario('XProviderEnabled')
*/
export function XProviderEnabled(): MockScenario {
const environment = structuredClone(EnvironmentService.SINGLE_SESSION);

environment.config.user_settings.social = {
...environment.config.user_settings.social,
oauth_x: {
authenticatable: true,
enabled: true,
logo_url: '',
name: 'X / Twitter',
required: false,
strategy: 'oauth_x',
},
};

setClerkState({ environment, session: null });

return {
description: 'Sign-in page with the X / Twitter social provider enabled',
handlers: clerkHandlers,
name: 'x-provider-enabled',
};
}
2 changes: 1 addition & 1 deletion packages/ui/src/common/ProviderIcon.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,7 +7,7 @@ import { ProviderInitialIcon } from './ProviderInitialIcon';

type ProviderId = OAuthProvider | Web3Provider | PhoneCodeChannel;

const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel'] as const;
const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel', 'x'] as const;

const supportsMaskImage = (id: ProviderId): boolean => {
return (SUPPORTS_MASK_IMAGE as readonly string[]).includes(id);
Expand Down
22 changes: 22 additions & 0 deletions packages/ui/src/common/__tests__/ProviderIcon.test.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -96,6 +96,28 @@ describe('ProviderIcon', () => {
expect(icon).toBeInTheDocument();
});

it('applies mask-image styles for supported providers (x)', async () => {
const { wrapper } = await createFixtures();

render(
<ProviderIcon
id='x'
iconUrl='https://example.com/x-icon.svg'
name='X / Twitter'
/>,
{ wrapper },
);

const icon = screen.getByLabelText('X / Twitter icon');
expect(icon).toBeInTheDocument();

// The mask-image path tints the icon with the foreground color so it stays
// visible in dark mode, instead of painting the raw (black) SVG as a background.
const styles = window.getComputedStyle(icon);
expect(styles.maskImage).toContain('https://example.com/x-icon.svg');
expect(styles.backgroundImage).not.toContain('https://example.com/x-icon.svg');
});

it('applies mask-image styles for supported providers (vercel)', async () => {
const { wrapper } = await createFixtures();

Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
feat(ui): Change X logo to foreground color in dark mode by jordan-bott · Pull Request #8912 · clerk/javascript · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/x-logo-dark-mode.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
---
'@clerk/ui': patch
---

Fix the X (formerly Twitter) provider logo being nearly invisible in dark mode by recoloring it to match the foreground color, consistent with other monochrome provider icons.
1 change: 1 addition & 0 deletions packages/clerk-js/sandbox/scenarios/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -4,3 +4,4 @@ export { CheckoutSeats } from './checkout-seats';
export { OrgProfileSeatLimit } from './org-profile-seat-limit';
export { PricingTableSBB } from './pricing-table-sbb';
export { AnnualOnlyPlans } from './annual-only-plans';
export { XProviderEnabled } from './x-provider-enabled';
34 changes: 34 additions & 0 deletions packages/clerk-js/sandbox/scenarios/x-provider-enabled.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
import { clerkHandlers, EnvironmentService, type MockScenario, setClerkState } from '@clerk/msw';

/**
* Signed-out sign-in page with the X / Twitter social provider enabled.
*
* The shared sandbox instance does not have X enabled, so this scenario mocks
* the environment to add it — useful for verifying the X provider logo (e.g. how
* it renders in dark mode). Activate it from the console with:
*
* scenario.setScenario('XProviderEnabled')
*/
export function XProviderEnabled(): MockScenario {
const environment = structuredClone(EnvironmentService.SINGLE_SESSION);

environment.config.user_settings.social = {
...environment.config.user_settings.social,
oauth_x: {
authenticatable: true,
enabled: true,
logo_url: '',
name: 'X / Twitter',
required: false,
strategy: 'oauth_x',
},
};

setClerkState({ environment, session: null });

return {
description: 'Sign-in page with the X / Twitter social provider enabled',
handlers: clerkHandlers,
name: 'x-provider-enabled',
};
}
2 changes: 1 addition & 1 deletion packages/ui/src/common/ProviderIcon.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,7 +7,7 @@ import { ProviderInitialIcon } from './ProviderInitialIcon';

type ProviderId = OAuthProvider | Web3Provider | PhoneCodeChannel;

const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel'] as const;
const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel', 'x'] as const;

const supportsMaskImage = (id: ProviderId): boolean => {
return (SUPPORTS_MASK_IMAGE as readonly string[]).includes(id);
Expand Down
22 changes: 22 additions & 0 deletions packages/ui/src/common/__tests__/ProviderIcon.test.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -96,6 +96,28 @@ describe('ProviderIcon', () => {
expect(icon).toBeInTheDocument();
});

it('applies mask-image styles for supported providers (x)', async () => {
const { wrapper } = await createFixtures();

render(
<ProviderIcon
id='x'
iconUrl='https://example.com/x-icon.svg'
name='X / Twitter'
/>,
{ wrapper },
);

const icon = screen.getByLabelText('X / Twitter icon');
expect(icon).toBeInTheDocument();

// The mask-image path tints the icon with the foreground color so it stays
// visible in dark mode, instead of painting the raw (black) SVG as a background.
const styles = window.getComputedStyle(icon);
expect(styles.maskImage).toContain('https://example.com/x-icon.svg');
expect(styles.backgroundImage).not.toContain('https://example.com/x-icon.svg');
});

it('applies mask-image styles for supported providers (vercel)', async () => {
const { wrapper } = await createFixtures();

Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' feat(ui): Change X logo to foreground color in dark mode by jordan-bott · Pull Request #8912 · clerk/javascript · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/x-logo-dark-mode.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
---
'@clerk/ui': patch
---

Fix the X (formerly Twitter) provider logo being nearly invisible in dark mode by recoloring it to match the foreground color, consistent with other monochrome provider icons.
1 change: 1 addition & 0 deletions packages/clerk-js/sandbox/scenarios/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -4,3 +4,4 @@ export { CheckoutSeats } from './checkout-seats';
export { OrgProfileSeatLimit } from './org-profile-seat-limit';
export { PricingTableSBB } from './pricing-table-sbb';
export { AnnualOnlyPlans } from './annual-only-plans';
export { XProviderEnabled } from './x-provider-enabled';
34 changes: 34 additions & 0 deletions packages/clerk-js/sandbox/scenarios/x-provider-enabled.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
import { clerkHandlers, EnvironmentService, type MockScenario, setClerkState } from '@clerk/msw';

/**
* Signed-out sign-in page with the X / Twitter social provider enabled.
*
* The shared sandbox instance does not have X enabled, so this scenario mocks
* the environment to add it — useful for verifying the X provider logo (e.g. how
* it renders in dark mode). Activate it from the console with:
*
* scenario.setScenario('XProviderEnabled')
*/
export function XProviderEnabled(): MockScenario {
const environment = structuredClone(EnvironmentService.SINGLE_SESSION);

environment.config.user_settings.social = {
...environment.config.user_settings.social,
oauth_x: {
authenticatable: true,
enabled: true,
logo_url: '',
name: 'X / Twitter',
required: false,
strategy: 'oauth_x',
},
};

setClerkState({ environment, session: null });

return {
description: 'Sign-in page with the X / Twitter social provider enabled',
handlers: clerkHandlers,
name: 'x-provider-enabled',
};
}
2 changes: 1 addition & 1 deletion packages/ui/src/common/ProviderIcon.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,7 +7,7 @@ import { ProviderInitialIcon } from './ProviderInitialIcon';

type ProviderId = OAuthProvider | Web3Provider | PhoneCodeChannel;

const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel'] as const;
const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel', 'x'] as const;

const supportsMaskImage = (id: ProviderId): boolean => {
return (SUPPORTS_MASK_IMAGE as readonly string[]).includes(id);
Expand Down
22 changes: 22 additions & 0 deletions packages/ui/src/common/__tests__/ProviderIcon.test.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -96,6 +96,28 @@ describe('ProviderIcon', () => {
expect(icon).toBeInTheDocument();
});

it('applies mask-image styles for supported providers (x)', async () => {
const { wrapper } = await createFixtures();

render(
<ProviderIcon
id='x'
iconUrl='https://example.com/x-icon.svg'
name='X / Twitter'
/>,
{ wrapper },
);

const icon = screen.getByLabelText('X / Twitter icon');
expect(icon).toBeInTheDocument();

// The mask-image path tints the icon with the foreground color so it stays
// visible in dark mode, instead of painting the raw (black) SVG as a background.
const styles = window.getComputedStyle(icon);
expect(styles.maskImage).toContain('https://example.com/x-icon.svg');
expect(styles.backgroundImage).not.toContain('https://example.com/x-icon.svg');
});

it('applies mask-image styles for supported providers (vercel)', async () => {
const { wrapper } = await createFixtures();

Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' feat(ui): Change X logo to foreground color in dark mode by jordan-bott · Pull Request #8912 · clerk/javascript · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/x-logo-dark-mode.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
---
'@clerk/ui': patch
---

Fix the X (formerly Twitter) provider logo being nearly invisible in dark mode by recoloring it to match the foreground color, consistent with other monochrome provider icons.
1 change: 1 addition & 0 deletions packages/clerk-js/sandbox/scenarios/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -4,3 +4,4 @@ export { CheckoutSeats } from './checkout-seats';
export { OrgProfileSeatLimit } from './org-profile-seat-limit';
export { PricingTableSBB } from './pricing-table-sbb';
export { AnnualOnlyPlans } from './annual-only-plans';
export { XProviderEnabled } from './x-provider-enabled';
34 changes: 34 additions & 0 deletions packages/clerk-js/sandbox/scenarios/x-provider-enabled.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
import { clerkHandlers, EnvironmentService, type MockScenario, setClerkState } from '@clerk/msw';

/**
* Signed-out sign-in page with the X / Twitter social provider enabled.
*
* The shared sandbox instance does not have X enabled, so this scenario mocks
* the environment to add it — useful for verifying the X provider logo (e.g. how
* it renders in dark mode). Activate it from the console with:
*
* scenario.setScenario('XProviderEnabled')
*/
export function XProviderEnabled(): MockScenario {
const environment = structuredClone(EnvironmentService.SINGLE_SESSION);

environment.config.user_settings.social = {
...environment.config.user_settings.social,
oauth_x: {
authenticatable: true,
enabled: true,
logo_url: '',
name: 'X / Twitter',
required: false,
strategy: 'oauth_x',
},
};

setClerkState({ environment, session: null });

return {
description: 'Sign-in page with the X / Twitter social provider enabled',
handlers: clerkHandlers,
name: 'x-provider-enabled',
};
}
2 changes: 1 addition & 1 deletion packages/ui/src/common/ProviderIcon.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,7 +7,7 @@ import { ProviderInitialIcon } from './ProviderInitialIcon';

type ProviderId = OAuthProvider | Web3Provider | PhoneCodeChannel;

const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel'] as const;
const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel', 'x'] as const;

const supportsMaskImage = (id: ProviderId): boolean => {
return (SUPPORTS_MASK_IMAGE as readonly string[]).includes(id);
Expand Down
22 changes: 22 additions & 0 deletions packages/ui/src/common/__tests__/ProviderIcon.test.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -96,6 +96,28 @@ describe('ProviderIcon', () => {
expect(icon).toBeInTheDocument();
});

it('applies mask-image styles for supported providers (x)', async () => {
const { wrapper } = await createFixtures();

render(
<ProviderIcon
id='x'
iconUrl='https://example.com/x-icon.svg'
name='X / Twitter'
/>,
{ wrapper },
);

const icon = screen.getByLabelText('X / Twitter icon');
expect(icon).toBeInTheDocument();

// The mask-image path tints the icon with the foreground color so it stays
// visible in dark mode, instead of painting the raw (black) SVG as a background.
const styles = window.getComputedStyle(icon);
expect(styles.maskImage).toContain('https://example.com/x-icon.svg');
expect(styles.backgroundImage).not.toContain('https://example.com/x-icon.svg');
});

it('applies mask-image styles for supported providers (vercel)', async () => {
const { wrapper } = await createFixtures();

Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ' feat(ui): Change X logo to foreground color in dark mode by jordan-bott · Pull Request #8912 · clerk/javascript · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/x-logo-dark-mode.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
---
'@clerk/ui': patch
---

Fix the X (formerly Twitter) provider logo being nearly invisible in dark mode by recoloring it to match the foreground color, consistent with other monochrome provider icons.
1 change: 1 addition & 0 deletions packages/clerk-js/sandbox/scenarios/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -4,3 +4,4 @@ export { CheckoutSeats } from './checkout-seats';
export { OrgProfileSeatLimit } from './org-profile-seat-limit';
export { PricingTableSBB } from './pricing-table-sbb';
export { AnnualOnlyPlans } from './annual-only-plans';
export { XProviderEnabled } from './x-provider-enabled';
34 changes: 34 additions & 0 deletions packages/clerk-js/sandbox/scenarios/x-provider-enabled.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
import { clerkHandlers, EnvironmentService, type MockScenario, setClerkState } from '@clerk/msw';

/**
* Signed-out sign-in page with the X / Twitter social provider enabled.
*
* The shared sandbox instance does not have X enabled, so this scenario mocks
* the environment to add it — useful for verifying the X provider logo (e.g. how
* it renders in dark mode). Activate it from the console with:
*
* scenario.setScenario('XProviderEnabled')
*/
export function XProviderEnabled(): MockScenario {
const environment = structuredClone(EnvironmentService.SINGLE_SESSION);

environment.config.user_settings.social = {
...environment.config.user_settings.social,
oauth_x: {
authenticatable: true,
enabled: true,
logo_url: '',
name: 'X / Twitter',
required: false,
strategy: 'oauth_x',
},
};

setClerkState({ environment, session: null });

return {
description: 'Sign-in page with the X / Twitter social provider enabled',
handlers: clerkHandlers,
name: 'x-provider-enabled',
};
}
2 changes: 1 addition & 1 deletion packages/ui/src/common/ProviderIcon.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,7 +7,7 @@ import { ProviderInitialIcon } from './ProviderInitialIcon';

type ProviderId = OAuthProvider | Web3Provider | PhoneCodeChannel;

const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel'] as const;
const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel', 'x'] as const;

const supportsMaskImage = (id: ProviderId): boolean => {
return (SUPPORTS_MASK_IMAGE as readonly string[]).includes(id);
Expand Down
22 changes: 22 additions & 0 deletions packages/ui/src/common/__tests__/ProviderIcon.test.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -96,6 +96,28 @@ describe('ProviderIcon', () => {
expect(icon).toBeInTheDocument();
});

it('applies mask-image styles for supported providers (x)', async () => {
const { wrapper } = await createFixtures();

render(
<ProviderIcon
id='x'
iconUrl='https://example.com/x-icon.svg'
name='X / Twitter'
/>,
{ wrapper },
);

const icon = screen.getByLabelText('X / Twitter icon');
expect(icon).toBeInTheDocument();

// The mask-image path tints the icon with the foreground color so it stays
// visible in dark mode, instead of painting the raw (black) SVG as a background.
const styles = window.getComputedStyle(icon);
expect(styles.maskImage).toContain('https://example.com/x-icon.svg');
expect(styles.backgroundImage).not.toContain('https://example.com/x-icon.svg');
});

it('applies mask-image styles for supported providers (vercel)', async () => {
const { wrapper } = await createFixtures();

Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' feat(ui): Change X logo to foreground color in dark mode by jordan-bott · Pull Request #8912 · clerk/javascript · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/x-logo-dark-mode.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
---
'@clerk/ui': patch
---

Fix the X (formerly Twitter) provider logo being nearly invisible in dark mode by recoloring it to match the foreground color, consistent with other monochrome provider icons.
1 change: 1 addition & 0 deletions packages/clerk-js/sandbox/scenarios/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -4,3 +4,4 @@ export { CheckoutSeats } from './checkout-seats';
export { OrgProfileSeatLimit } from './org-profile-seat-limit';
export { PricingTableSBB } from './pricing-table-sbb';
export { AnnualOnlyPlans } from './annual-only-plans';
export { XProviderEnabled } from './x-provider-enabled';
34 changes: 34 additions & 0 deletions packages/clerk-js/sandbox/scenarios/x-provider-enabled.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
import { clerkHandlers, EnvironmentService, type MockScenario, setClerkState } from '@clerk/msw';

/**
* Signed-out sign-in page with the X / Twitter social provider enabled.
*
* The shared sandbox instance does not have X enabled, so this scenario mocks
* the environment to add it — useful for verifying the X provider logo (e.g. how
* it renders in dark mode). Activate it from the console with:
*
* scenario.setScenario('XProviderEnabled')
*/
export function XProviderEnabled(): MockScenario {
const environment = structuredClone(EnvironmentService.SINGLE_SESSION);

environment.config.user_settings.social = {
...environment.config.user_settings.social,
oauth_x: {
authenticatable: true,
enabled: true,
logo_url: '',
name: 'X / Twitter',
required: false,
strategy: 'oauth_x',
},
};

setClerkState({ environment, session: null });

return {
description: 'Sign-in page with the X / Twitter social provider enabled',
handlers: clerkHandlers,
name: 'x-provider-enabled',
};
}
2 changes: 1 addition & 1 deletion packages/ui/src/common/ProviderIcon.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,7 +7,7 @@ import { ProviderInitialIcon } from './ProviderInitialIcon';

type ProviderId = OAuthProvider | Web3Provider | PhoneCodeChannel;

const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel'] as const;
const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel', 'x'] as const;

const supportsMaskImage = (id: ProviderId): boolean => {
return (SUPPORTS_MASK_IMAGE as readonly string[]).includes(id);
Expand Down
22 changes: 22 additions & 0 deletions packages/ui/src/common/__tests__/ProviderIcon.test.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -96,6 +96,28 @@ describe('ProviderIcon', () => {
expect(icon).toBeInTheDocument();
});

it('applies mask-image styles for supported providers (x)', async () => {
const { wrapper } = await createFixtures();

render(
<ProviderIcon
id='x'
iconUrl='https://example.com/x-icon.svg'
name='X / Twitter'
/>,
{ wrapper },
);

const icon = screen.getByLabelText('X / Twitter icon');
expect(icon).toBeInTheDocument();

// The mask-image path tints the icon with the foreground color so it stays
// visible in dark mode, instead of painting the raw (black) SVG as a background.
const styles = window.getComputedStyle(icon);
expect(styles.maskImage).toContain('https://example.com/x-icon.svg');
expect(styles.backgroundImage).not.toContain('https://example.com/x-icon.svg');
});

it('applies mask-image styles for supported providers (vercel)', async () => {
const { wrapper } = await createFixtures();

Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); })(); feat(ui): Change X logo to foreground color in dark mode by jordan-bott · Pull Request #8912 · clerk/javascript · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/x-logo-dark-mode.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
---
'@clerk/ui': patch
---

Fix the X (formerly Twitter) provider logo being nearly invisible in dark mode by recoloring it to match the foreground color, consistent with other monochrome provider icons.
1 change: 1 addition & 0 deletions packages/clerk-js/sandbox/scenarios/index.ts
Original file line numberDiff line numberDiff line change
Expand Up@@ -4,3 +4,4 @@ export { CheckoutSeats } from './checkout-seats';
export { OrgProfileSeatLimit } from './org-profile-seat-limit';
export { PricingTableSBB } from './pricing-table-sbb';
export { AnnualOnlyPlans } from './annual-only-plans';
export { XProviderEnabled } from './x-provider-enabled';
34 changes: 34 additions & 0 deletions packages/clerk-js/sandbox/scenarios/x-provider-enabled.ts
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
import { clerkHandlers, EnvironmentService, type MockScenario, setClerkState } from '@clerk/msw';

/**
* Signed-out sign-in page with the X / Twitter social provider enabled.
*
* The shared sandbox instance does not have X enabled, so this scenario mocks
* the environment to add it — useful for verifying the X provider logo (e.g. how
* it renders in dark mode). Activate it from the console with:
*
* scenario.setScenario('XProviderEnabled')
*/
export function XProviderEnabled(): MockScenario {
const environment = structuredClone(EnvironmentService.SINGLE_SESSION);

environment.config.user_settings.social = {
...environment.config.user_settings.social,
oauth_x: {
authenticatable: true,
enabled: true,
logo_url: '',
name: 'X / Twitter',
required: false,
strategy: 'oauth_x',
},
};

setClerkState({ environment, session: null });

return {
description: 'Sign-in page with the X / Twitter social provider enabled',
handlers: clerkHandlers,
name: 'x-provider-enabled',
};
}
2 changes: 1 addition & 1 deletion packages/ui/src/common/ProviderIcon.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,7 +7,7 @@ import { ProviderInitialIcon } from './ProviderInitialIcon';

type ProviderId = OAuthProvider | Web3Provider | PhoneCodeChannel;

const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel'] as const;
const SUPPORTS_MASK_IMAGE = ['apple', 'github', 'okx_wallet', 'vercel', 'x'] as const;

const supportsMaskImage = (id: ProviderId): boolean => {
return (SUPPORTS_MASK_IMAGE as readonly string[]).includes(id);
Expand Down
22 changes: 22 additions & 0 deletions packages/ui/src/common/__tests__/ProviderIcon.test.tsx
Original file line numberDiff line numberDiff line change
Expand Up@@ -96,6 +96,28 @@ describe('ProviderIcon', () => {
expect(icon).toBeInTheDocument();
});

it('applies mask-image styles for supported providers (x)', async () => {
const { wrapper } = await createFixtures();

render(
<ProviderIcon
id='x'
iconUrl='https://example.com/x-icon.svg'
name='X / Twitter'
/>,
{ wrapper },
);

const icon = screen.getByLabelText('X / Twitter icon');
expect(icon).toBeInTheDocument();

// The mask-image path tints the icon with the foreground color so it stays
// visible in dark mode, instead of painting the raw (black) SVG as a background.
const styles = window.getComputedStyle(icon);
expect(styles.maskImage).toContain('https://example.com/x-icon.svg');
expect(styles.backgroundImage).not.toContain('https://example.com/x-icon.svg');
});

it('applies mask-image styles for supported providers (vercel)', async () => {
const { wrapper } = await createFixtures();

Expand Down
Loading