Skip to content

OUT-2852 | OUT-2840 | OUT-2865 | Add topbar and IU layout skeleton - #3

Merged
rrojan merged 9 commits into
mainfrom
OUT-2840
Dec 29, 2025
Merged

rrojan merged 9 commits into
mainfrom
OUT-2840

Conversation

@rrojan

@rrojan rrojan commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

Changes:

  • Add topbar (formatting toolbar is not present since the Assembly team mentioned it as an improvement change later down the line)
  • Add IU layout scaffold

Testing Criteria

  • Screenshot
Screen.Recording.2025-12-29.at.12.53.13.mov

@rrojan rrojan self-assigned this Dec 29, 2025
@linear

linear Bot commented Dec 29, 2025

Copy link
Copy Markdown

@vercel

vercel Bot commented Dec 29, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
client-home-v3 Ready Ready Preview, Comment Dec 29, 2025 11:55am

@rrojan rrojan changed the title OUT-2852 | OUT-2840 | Add topbar and IU layout skeleton OUT-2852 | OUT-2840 | OUT-2865 | Add topbar and IU layout skeleton Dec 29, 2025

@SandipBajracharya SandipBajracharya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🙌

<button
type="button"
onClick={handleClick}
className={cn('flex px-[6px] py-[2px] text-custom-xs', active && 'bg-background-secondary')}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
className={cn('flex px-[6px] py-[2px] text-custom-xs', active && 'bg-background-secondary')}
className={cn('flex px-1.5 py-0.5 text-custom-xs', active && 'bg-background-secondary')}

@rrojan
rrojan merged commit 12ee18d into main Dec 29, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants