Skip to content

[WIP] Add modal display for error verification#5

Merged
GGJdevelope merged 2 commits intomasterfrom
copilot/show-modal-on-error
Jan 11, 2026
Merged

[WIP] Add modal display for error verification#5
GGJdevelope merged 2 commits intomasterfrom
copilot/show-modal-on-error

Conversation

Copy link

Copilot AI commented Jan 10, 2026

Implementation Plan for Error Modal Display

  • Understand the existing error handling and display systems
  • Add platform API for showing error modals
    • Define error modal function in platform.h
    • Implement for macOS (src/platform/macos/)
    • Implement for Linux X11 (src/platform/linux/X/)
    • Implement for Linux Wayland (src/platform/linux/wayland/)
    • Implement for Windows (src/platform/windows/)
  • Create a global error handler function
    • Add show_error_modal() wrapper in warpd.h/warpd.c
    • Handle multi-line error messages
    • Add timeout or dismiss functionality
  • Replace critical fprintf(stderr) calls with modal errors
    • macOS accessibility permission errors
    • X11 keyboard grab errors
    • Additional runtime errors that users should see
  • Test the modal display functionality
  • Document the changes
Original prompt

직전 오류 관련해서 오류 발생시 확인 가능한 모달을 화면에 표시해줘

직전 pr의 오류는 아직 고쳐지지 않았어


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: GGJdevelope <57247934+GGJdevelope@users.noreply.github.com>
Copilot AI requested a review from GGJdevelope January 10, 2026 16:07
Copilot stopped work on behalf of GGJdevelope due to an error January 10, 2026 16:07
@GGJdevelope
Copy link
Owner

@copilot 지금 변경된 파일 그대로 pr해줘

@GGJdevelope GGJdevelope marked this pull request as ready for review January 11, 2026 01:32
@GGJdevelope GGJdevelope merged commit 93e56bb into master Jan 11, 2026
1 check failed
@GGJdevelope GGJdevelope deleted the copilot/show-modal-on-error branch January 11, 2026 01:32
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