-
-
Notifications
You must be signed in to change notification settings - Fork 927
New Vertical Tab Bar Option #3059
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 1 commit
6cf17da
187c9c6
5e3ff22
64bf73f
1dac092
f6dc5cf
df17a38
ef0dd32
7a55a3b
4f269be
8b80a20
0350a48
91b2144
147f445
1fd0ca9
99011b1
c3fd5ea
2b81a88
f61531d
9b28cf7
ca8979b
f895fd9
9ef05fa
58c4b3d
aa38645
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -32,6 +32,7 @@ import { activeTabIdAtom } from "@/store/tab-model"; | |
| import * as WOS from "@/store/wos"; | ||
| import { loadFonts } from "@/util/fontutil"; | ||
| import { setKeyUtilPlatform } from "@/util/keyutil"; | ||
| import { isMacOS, setMacOSVersion } from "@/util/platformutil"; | ||
| import { createElement } from "react"; | ||
| import { createRoot } from "react-dom/client"; | ||
|
|
||
|
|
@@ -163,6 +164,10 @@ async function initWave(initOpts: WaveInitOpts) { | |
| await loadBadges(); | ||
| initGlobalWaveEventSubs(initOpts); | ||
| subscribeToConnEvents(); | ||
| if (isMacOS()) { | ||
| const macOSVersion = await RpcApi.MacOSVersionCommand(TabRpcClient); | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. WARNING: Missing error handling for MacOSVersionCommand RPC call. If this RPC fails (e.g., server not ready), it will throw an unhandled error and could crash the app initialization. Consider wrapping this in a try-catch to gracefully handle failures: if (isMacOS()) {
try {
const macOSVersion = await RpcApi.MacOSVersionCommand(TabRpcClient);
setMacOSVersion(macOSVersion);
} catch (e) {
console.warn("Failed to get macOS version:", e);
}
} |
||
| setMacOSVersion(macOSVersion); | ||
| } | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Don’t let macOS version probing block app startup. If 💡 Suggested fix if (isMacOS()) {
- const macOSVersion = await RpcApi.MacOSVersionCommand(TabRpcClient);
- setMacOSVersion(macOSVersion);
+ try {
+ const macOSVersion = await RpcApi.MacOSVersionCommand(TabRpcClient);
+ setMacOSVersion(macOSVersion);
+ } catch (e) {
+ getApi().sendLog("MacOSVersionCommand failed: " + (e?.message ?? e));
+ }
}🤖 Prompt for AI Agents |
||
|
|
||
| // ensures client/window/workspace are loaded into the cache before rendering | ||
| try { | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't size the hidden strip by tab count.
When
noTabsis true, this wrapper still expands totabIds.length * tabWidthRef.current, so left-tab mode can end up with an empty horizontal scroller/drag region on large workspaces. Making the wrapper fill the available space avoids that blank strip.Suggested change
<div className="tabs-wrapper" ref={tabsWrapperRef} style={{ - width: `${tabsWrapperWidth}px`, + width: noTabs ? "100%" : `${tabsWrapperWidth}px`, ...(noTabs ? ({ WebkitAppRegion: "drag" } as React.CSSProperties) : {}), }} >🤖 Prompt for AI Agents