Skip to content

[dev] [Marfuen] mariano/ui-improvements#2030

Merged
Marfuen merged 8 commits intomainfrom
mariano/ui-improvements
Jan 22, 2026
Merged

[dev] [Marfuen] mariano/ui-improvements#2030
Marfuen merged 8 commits intomainfrom
mariano/ui-improvements

Conversation

@github-actions
Copy link
Contributor

This is an automated pull request to merge mariano/ui-improvements into dev.
It was created by the [Auto Pull Request] action.

@vercel
Copy link

vercel bot commented Jan 22, 2026

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

Project Deployment Review Updated (UTC)
app Ready Ready Preview, Comment Jan 22, 2026 4:27pm
1 Skipped Deployment
Project Deployment Review Updated (UTC)
portal Skipped Skipped Jan 22, 2026 4:27pm

Request Review

@cursor
Copy link

cursor bot commented Jan 22, 2026

PR Summary

  • New Settings navigation: Adds SettingsSidebar and integrates it into AppShellWrapper; sidebar header now switches between Settings and Compliance.
  • Feature flags: Plumbs isWebAutomationsEnabled from layout to conditionally show the "Browser" settings item.
  • SettingsTabs refactor: Replaces tab UI with a page header whose title derives from the current route.
  • Policy page improvements: Passes mutate directly to children and adds a stable key to UpdatePolicyOverview to ensure refresh; minor cleanup.
  • Notifications/UI polish: Swaps bell icon to DS Notification, adjusts styling; removes extra wrapper/classes in root layout.
  • Deps: Bumps @trycompai/design-system to ^1.0.42.

Written by Cursor Bugbot for commit ff52d8a. This will update automatically on new commits. Configure here.

@vercel vercel bot temporarily deployed to Preview – portal January 22, 2026 16:25 Inactive
@Marfuen Marfuen merged commit d0511f4 into main Jan 22, 2026
8 of 9 checks passed
@Marfuen Marfuen deleted the mariano/ui-improvements branch January 22, 2026 16:26
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

import { PageHeader, PageLayout, Tabs, TabsList, TabsTrigger } from '@trycompai/design-system';
import { usePathname, useRouter } from 'next/navigation';
import { PageHeader, PageLayout } from '@trycompai/design-system';
import { usePathname } from 'next/navigation';
Copy link

Choose a reason for hiding this comment

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

Unused prop in SettingsTabs interface after refactoring

Low Severity

The showBrowserTab prop is defined in SettingsTabsProps interface and passed from layout.tsx, but it's no longer used by the SettingsTabs component after the tabs navigation was moved to SettingsSidebar. The component destructures only { orgId, children }, completely ignoring showBrowserTab. This creates dead code that adds confusion and maintenance burden.

Fix in Cursor Fix in Web

@claudfuen
Copy link
Contributor

🎉 This PR is included in version 1.79.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants