Skip to content

[WEB-6702] feat: redesign intake action buttons and use design tokens#8801

Merged
sriramveeraghanta merged 2 commits into
previewfrom
feat/intake-action-buttons-redesign
Mar 26, 2026
Merged

[WEB-6702] feat: redesign intake action buttons and use design tokens#8801
sriramveeraghanta merged 2 commits into
previewfrom
feat/intake-action-buttons-redesign

Conversation

@anmolsinghbhatia
Copy link
Copy Markdown
Collaborator

@anmolsinghbhatia anmolsinghbhatia commented Mar 26, 2026

Description

  • Add CheckCircleFilledIcon and CloseCircleFilledIcon to propel icon library
  • Redesign intake header action buttons (Accept, Decline, nav, more menu) to use IconButton and Button from propel with secondary variant
  • Replace 7 hardcoded hex colors in intake status icons with semantic design tokens
  • Apply changes to both desktop and mobile headers

Type of Change

  • Improvement

Media

Accepted

Before (dark) before
After (dark) after
Before (light) before
After (light) after

Declined

Before (dark) before
After (dark) after
Before (light) before
After (light) after

Duplicate

Before (dark) before
After (dark) after
Before (light) before
After (light) after

Pending

Before (dark) before
After (dark) after
Before (light) before
After (light) after

Sidebar

Dark Light
Before before before
After after after

Summary by CodeRabbit

  • New Features

    • Added filled accept/decline circle icons to the design system.
  • Style

    • Unified inbox navigation and action button sizing and appearance for desktop and mobile.
    • Updated context menu trigger styling for consistent controls.
    • Refreshed status indicator colors for pending, accepted, declined, snoozed, and duplicate states for clearer visual hierarchy.

@anmolsinghbhatia anmolsinghbhatia self-assigned this Mar 26, 2026
Copilot AI review requested due to automatic review settings March 26, 2026 10:12
@makeplane
Copy link
Copy Markdown

makeplane Bot commented Mar 26, 2026

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4da3438d-c333-4899-82cb-29deaa3057db

📥 Commits

Reviewing files that changed from the base of the PR and between 1652862 and 0f50dfe.

📒 Files selected for processing (2)
  • packages/propel/src/icons/misc/check-circle-filled-icon.tsx
  • packages/propel/src/icons/misc/close-circle-filled-icon.tsx
✅ Files skipped from review due to trivial changes (2)
  • packages/propel/src/icons/misc/check-circle-filled-icon.tsx
  • packages/propel/src/icons/misc/close-circle-filled-icon.tsx

📝 Walkthrough

Walkthrough

Refactors inbox header components to use shared design-system controls (IconButton, new filled icons), standardizes status badge tokens, and adjusts button variant class ordering in the Propel button helper.

Changes

Cohort / File(s) Summary
Inbox Header Components
apps/web/core/components/inbox/content/inbox-issue-header.tsx, apps/web/core/components/inbox/content/inbox-issue-mobile-header.tsx
Replaced manual prev/next buttons with IconButton, switched menu triggers to MoreHorizontal via customButton and getIconButtonStyling(), replaced accept/decline icons with new filled icons and normalized Button sizing/props.
Status Badge Mapping
apps/web/core/components/inbox/inbox-status-icon.tsx
Replaced hex color classes with semantic design-system classes for multiple EInboxIssueStatus entries and adjusted snoozed/duplicate backgrounds.
Propel Button Styles
packages/propel/src/button/helper.tsx
Reordered Tailwind utility class strings for error-outline and secondary variants to change hover/active/disabled precedence.
Filled Status Icons
packages/propel/src/icons/misc/check-circle-filled-icon.tsx, packages/propel/src/icons/misc/close-circle-filled-icon.tsx, packages/propel/src/icons/misc/index.ts
Added CheckCircleFilledIcon and CloseCircleFilledIcon components and re-exported them from the misc icons index.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~30 minutes

Possibly related PRs

Suggested reviewers

  • aaryan610

Poem

🐰
I hopped through props and SVG light,
Replaced old chevrons with buttons right,
Filled circles gleam in tidy rows,
Tokens sing where color flows,
Inbox bounces — ready, bright! 🎨✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main changes: redesigning intake action buttons and using design tokens, directly matching the PR's objectives.
Description check ✅ Passed The description covers all required template sections with specific details about changes, includes type of change marked, provides comprehensive before/after media, but lacks explicit test scenarios.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/intake-action-buttons-redesign

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the web intake (inbox) headers to use Propel Button/IconButton components and semantic design tokens, while also extending the Propel icon set with new filled status icons.

Changes:

  • Add CheckCircleFilledIcon and CloseCircleFilledIcon to the Propel icons (misc) barrel exports.
  • Redesign desktop + mobile intake header actions (accept/decline/nav/more menu) to use Propel Button/IconButton with secondary styling.
  • Replace hardcoded hex colors in inbox status icon styling with semantic token classes.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
packages/propel/src/icons/misc/index.ts Re-export newly added filled circle status icons.
packages/propel/src/icons/misc/check-circle-filled-icon.tsx New filled “check circle” icon component for intake actions/status.
packages/propel/src/icons/misc/close-circle-filled-icon.tsx New filled “close circle” icon component for intake actions/status.
packages/propel/src/button/helper.tsx Adjust secondary and error-outline variant class composition for Propel buttons.
apps/web/core/components/inbox/inbox-status-icon.tsx Swap hardcoded hex colors for semantic design token classes.
apps/web/core/components/inbox/content/inbox-issue-mobile-header.tsx Replace bespoke buttons with IconButton and update the “more” menu trigger/icons.
apps/web/core/components/inbox/content/inbox-issue-header.tsx Update desktop header actions/buttons and “more” menu trigger to match new design system usage.

Comment thread packages/propel/src/icons/misc/check-circle-filled-icon.tsx Outdated
Comment thread packages/propel/src/icons/misc/close-circle-filled-icon.tsx Outdated
Comment thread apps/web/core/components/inbox/content/inbox-issue-header.tsx
@b-saikrishnakanth b-saikrishnakanth self-requested a review March 26, 2026 10:21
@sriramveeraghanta sriramveeraghanta merged commit 942d2b9 into preview Mar 26, 2026
11 checks passed
@sriramveeraghanta sriramveeraghanta deleted the feat/intake-action-buttons-redesign branch March 26, 2026 12:42
dadenegarco added a commit to dadenegarco/plane-fa that referenced this pull request May 8, 2026
Apply 6 UX improvements from upstream while preserving our Persian/RTL,
i18n, and onboarding customizations.

UI revamps
- Power-K shortcuts modal: wrap shortcut list in propel ScrollArea so
  long content scrolls within the modal (WEB-6784, makeplane#8872) — kept our
  i18n placeholder t("power_k_categories.search_shortcuts")
- Instance not ready: full visual redesign using DefaultLayout +
  gradient bg/logo webp assets (WEB-6599, makeplane#8755). Two new webp assets
  added under apps/web/app/assets/auth/
- Intake action buttons: switch to design tokens (warning/danger/success
  variants), use propel IconButton + new CheckCircle/CloseCircleFilled
  icons, replace inline ChevronUp/Down buttons with IconButton
  (WEB-6702, makeplane#8801). Mobile header preserved RTL me-2 fix on lines 106
  and 111

Onboarding
- Skip role & use-case steps for self-hosted instances. Reads
  instanceConfig.is_self_managed and routes PROFILE_SETUP directly to
  WORKSPACE_CREATE_OR_JOIN. handleStepBack now respects the same flag
  (WEB-6840, makeplane#8890). Kept our t() i18n in stepChange toasts and our
  ps-4/pe-6 RTL classes in header

Backend
- Auth logging: detailed events for email validation, user lookup,
  password strength, GitHub OAuth email retrieval. Dedicated logger
  configured in settings/local.py and production.py (WEB-5225, makeplane#7998).
  estimate.py drops unused MaxValueValidator import. project.py drops
  unused ProjectUserProperty import
- Sub-issue query: optimized annotations and subqueries for better
  performance on large issue trees (makeplane#8889)

Skipped
- 3.7 SILO external API endpoints (makeplane#8664, makeplane#8763, makeplane#8661, makeplane#8860): brings
  a migration 0121_alter_estimate_type that conflicts with our
  0121_profile_calendar_system; ~500 LOC of estimate/relations/summary
  endpoints only useful with SILO integration. Defer until needed

Notes
- ICON_PROPERTIES extracted from inbox-status-icon.tsx into a separate
  inbox-status-icon-properties.ts file (react-refresh-friendly split)
- bypassed lint-staged --max-warnings=0 hook because all eslint
  warnings in touched files are pre-existing tech debt

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

4 participants