Skip to content

improvement(logging-ui): improve logging UI to be less of information dump #650

Merged
icecrasher321 merged 6 commits intostagingfrom
improvement/logging-ui
Jul 10, 2025
Merged

improvement(logging-ui): improve logging UI to be less of information dump #650
icecrasher321 merged 6 commits intostagingfrom
improvement/logging-ui

Conversation

@icecrasher321
Copy link
Collaborator

@icecrasher321 icecrasher321 commented Jul 10, 2025

Description

  • Make each block be droppable
Screenshot 2025-07-09 at 7 23 49 PM
  • Make Frozen Canvas have way to expand input/output windows
Screen.Recording.2025-07-09.at.7.25.12.PM.mov
  • Make table header be card like as well for better alignment
  • Remove useless information in Block Execution Stats
  • Make padding better for trace spans display
  • Fix frozen canvas blocks not being clickable for webhook events + erroring workflows

Type of change

  • Performance improvement

How Has This Been Tested?

See screenshots above.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • All tests pass locally and in CI (bun run test)
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have updated version numbers as needed (if needed)
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

  • My changes do not introduce any new security vulnerabilities
  • I have considered the security implications of my changes

Vikhyath Mondreti added 2 commits July 9, 2025 19:19
@vercel
Copy link

vercel bot commented Jul 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sim ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 3:42am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Skipped (Inspect) Jul 10, 2025 3:42am

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Enhanced the logging UI to provide a cleaner, more interactive interface with improved information hierarchy and expandable views.

  • Added collapsible blocks with drop-down functionality for better information organization
  • Implemented expandable input/output windows in Frozen Canvas for detailed view access
  • Redesigned table headers with card-like styling for improved alignment
  • Streamlined Block Execution Stats by removing non-essential information
  • Optimized trace spans display with better padding for improved readability

No files reviewed, no comments
Edit PR Review Bot Settings | Greptile

@delve-auditor
Copy link

delve-auditor bot commented Jul 10, 2025

No security or compliance issues detected. Reviewed everything up to 717b4dd.

Security Overview
  • 🔎 Scanned files: 9 changed file(s)
Detected Code Changes
Change Type Relevant files
Enhancement ► frozen-canvas/frozen-canvas.tsx
    Add expandable data sections with modal support
    Improve handling of non-executed blocks
► sidebar/sidebar.tsx
    Update UI labels and remove block stats
► trace-spans-display.tsx
    Streamline trace span display
    Remove expand all functionality
► logs.tsx
    Update log table styling
Refactor ► enhanced-execution-logger.ts
    Remove block stats tracking
► enhanced-logging-factory.ts
    Remove block stats calculation
► enhanced-logging-session.ts
    Simplify error handling and logging
► types.ts
    Remove block stats interfaces

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@vercel vercel bot temporarily deployed to Preview – docs July 10, 2025 03:36 Inactive
@icecrasher321 icecrasher321 merged commit 529fd44 into staging Jul 10, 2025
5 of 6 checks passed
@waleedlatif1 waleedlatif1 deleted the improvement/logging-ui branch July 10, 2025 22:25
arenadeveloper02 pushed a commit to arenadeveloper02/p2-sim that referenced this pull request Sep 19, 2025
…ng-ui

improvement(logging-ui): improve logging UI to be less of information dump
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.

1 participant