fix: graph controls gap on compact viewports + model selector z-index#982
fix: graph controls gap on compact viewports + model selector z-index#982vorflux[bot] wants to merge 1 commit into
Conversation
- memory-graph: stack NavigationControls and Legend in a single flex column at bottom-left instead of using separate absolute-positioned elements with a hardcoded bottom offset, eliminating the empty gap on compact/mobile viewports - memory-graph: bump version 0.2.1 -> 0.2.2 - web: add z-10 to ChatModelSelector wrapper so its absolute dropdown renders above the SpaceSelector button that follows it in DOM order
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 4ddbd78 | May 21 2026, 12:33 AM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 4ddbd78 | Commit Preview URL Branch Preview URL |
May 21 2026, 12:34 AM |
Summary
Two UI bug fixes:
1. Graph page — controls gap on compact/mobile viewports
Before:
NavigationControlswas absolutely positioned withbottom: 148on compact viewports, whileLegendwas absolutely positioned atbottom: 16. This left ~120px of empty gap between them with no content filling it.Fix: Both
NavigationControlsandLegendare now rendered inside a singleflex-columncontainer absolutely positioned atbottom: 16, left: 16. They stack flush against each other with an 8px gap — no more empty space regardless of viewport size. Also removed the now-redundantposition: absolute/bottom/left/zIndexfromLegend's ownouterStylesince positioning is handled by the shared container.2. Model selector dialog z-index
Before: The
ChatModelSelectorwrapper had no explicit z-index. TheSpaceSelectorbutton rendered after it in the DOM would paint on top of the model selector's absolute-positioned dropdown (even though the dropdown hadz-50) because later DOM siblings at the same stacking level naturally paint over earlier ones.Fix: Added
z-10to theChatModelSelectorwrapper div, establishing a stacking context that places the entire model selector (including its dropdown) above theSpaceSelectorthat follows it.Version bump
@supermemory/memory-graph:0.2.1→0.2.2Testing
bottom: 148offset that caused the empty spacez-10on the wrapper creates a stacking context aboveSpaceSelector(z-auto) since both share the parentrelative z-20!container from the chat inputSession Details
(aside)to your comment to have me ignore it.