Skip to content
This repository was archived by the owner on Jul 31, 2026. It is now read-only.

feat: improve org selector layout sizing - #326

Merged
danielkov merged 3 commits into
mainfrom
daniel/gen-2269-bug-fix-workspace-selection-screen-for-mobile
Dec 24, 2025
Merged

feat: improve org selector layout sizing#326
danielkov merged 3 commits into
mainfrom
daniel/gen-2269-bug-fix-workspace-selection-screen-for-mobile

Conversation

@danielkov

@danielkov danielkov commented Dec 23, 2025

Copy link
Copy Markdown
Contributor
Before After
Screen.Recording.2025-12-23.at.15.16.10.mov
Screen.Recording.2025-12-23.at.15.14.19.mov

@linear

linear Bot commented Dec 23, 2025

Copy link
Copy Markdown

const inputRef = useRef<HTMLInputElement>(null)
const [filteredWorkspaces, setFilteredWorkspaces] = useState<Workspace[]>(
selectedOrg?.workspaces || []
selectedOrg.workspaces

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just sense-checking this doesn't crash anything when someone goes to a non-workspace page. Always a bit cautious when we remove guarding logic

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's true, if there are no orgs at all, this check can fail. Since I'm there anyway, I'll just refactor this part as it's a bit spaghetti right now.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(We wouldn't be rendering this component if there are no orgs, which is why this is fine)

@danielkov
danielkov merged commit 189ca77 into main Dec 24, 2025
9 checks passed
@danielkov
danielkov deleted the daniel/gen-2269-bug-fix-workspace-selection-screen-for-mobile branch December 24, 2025 15:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants