RomanNum3ral
/
gstack
mirror of https://github.com/garrytan/gstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: remove duplicate gstack pill overlay, hide crash restore bubble 

The addInitScript indicator and the extension's content script were both
injecting bottom-right pills, causing duplicates. Remove the pill from
addInitScript (extension handles it). Replace --restore-last-session with
--hide-crash-restore-bubble to suppress the "Chromium didn't shut down
correctly" dialog.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Garry Tan 2026-03-21 16:58:33 -07:00
parent 459faade89
commit 2b02e9612d
No known key found for this signature in database
GPG Key ID: C1F69E85C74EFE1D
1 changed files with 3 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
browse/src/browser-manager.ts
View File

@ -160,7 +160,7 @@ export class BrowserManager {
// Find the gstack extension directory for auto-loading // Find the gstack extension directory for auto-loading
const extensionPath = this.findExtensionPath(); const extensionPath = this.findExtensionPath();
const launchArgs = ['--restore-last-session']; const launchArgs = ['--hide-crash-restore-bubble'];
if (extensionPath) { if (extensionPath) {
launchArgs.push(`--disable-extensions-except=${extensionPath}`); launchArgs.push(`--disable-extensions-except=${extensionPath}`);
launchArgs.push(`--load-extension=${extensionPath}`); launchArgs.push(`--load-extension=${extensionPath}`);
@ -189,13 +189,12 @@ export class BrowserManager {
this.connectionMode = 'cdp'; this.connectionMode = 'cdp';
this.intentionalDisconnect = false; this.intentionalDisconnect = false;
// Inject visual indicator — subtle top-edge gradient + floating pill // Inject visual indicator — subtle top-edge amber gradient
// so the user always knows which Chrome window gstack controls // Extension's content script handles the floating pill
const indicatorScript = () => { const indicatorScript = () => {
const injectIndicator = () => { const injectIndicator = () => {
if (document.getElementById('gstack-ctrl')) return; if (document.getElementById('gstack-ctrl')) return;
// Thin gradient line at the very top of the viewport
const topLine = document.createElement('div'); const topLine = document.createElement('div');
topLine.id = 'gstack-ctrl'; topLine.id = 'gstack-ctrl';
topLine.style.cssText = ` topLine.style.cssText = `
@ -207,27 +206,6 @@ export class BrowserManager {
opacity: 0.8; opacity: 0.8;
`; `;
// Floating pill — bottom-right, fades to subtle
const pill = document.createElement('div');
pill.id = 'gstack-pill';
pill.style.cssText = `
position: fixed; bottom: 12px; right: 12px;
z-index: 2147483647; pointer-events: none;
display: flex; align-items: center; gap: 5px;
padding: 4px 10px;
background: rgba(12, 12, 12, 0.7);
backdrop-filter: blur(8px); -webkit-backdrop-filter: blur(8px);
border: 1px solid rgba(245, 158, 11, 0.25);
border-radius: 9999px;
font: 500 10px -apple-system, BlinkMacSystemFont, 'Segoe UI', system-ui, sans-serif;
color: rgba(255, 255, 255, 0.7);
letter-spacing: 0.03em;
transition: opacity 0.5s ease;
opacity: 1;
`;
pill.innerHTML = '<span style="width:5px;height:5px;border-radius:50%;background:#F59E0B;box-shadow:0 0 4px rgba(245,158,11,0.5);flex-shrink:0;"></span>gstack';
// Keyframe for shimmer animation
const style = document.createElement('style'); const style = document.createElement('style');
style.textContent = ` style.textContent = `
@keyframes gstack-shimmer { @keyframes gstack-shimmer {
@ -241,10 +219,6 @@ export class BrowserManager {
document.documentElement.appendChild(style); document.documentElement.appendChild(style);
document.documentElement.appendChild(topLine); document.documentElement.appendChild(topLine);
document.documentElement.appendChild(pill);
// Fade pill to subtle after 4s
setTimeout(() => { pill.style.opacity = '0.25'; }, 4000);
}; };
if (document.readyState === 'loading') { if (document.readyState === 'loading') {
document.addEventListener('DOMContentLoaded', injectIndicator); document.addEventListener('DOMContentLoaded', injectIndicator);