Skip to content

Refactor OS codebase and add plain text editor application#2

Open
celento wants to merge 1 commit into
masterfrom
claude/refactor-os-add-editor-W59BS
Open

Refactor OS codebase and add plain text editor application#2
celento wants to merge 1 commit into
masterfrom
claude/refactor-os-add-editor-W59BS

Conversation

@celento

@celento celento commented Mar 23, 2026

Copy link
Copy Markdown
Owner
  • Fix Makefile: remove hardcoded macOS homebrew paths, use generic tool
    names; remove macOS-only -display cocoa flag
  • Extract magic numbers into named constants (CHAR_WIDTH, TITLE_BAR_HEIGHT,
    WINDOW_PADDING, MENU_ITEM_SPACING)
  • Add 28 symbol font glyphs (punctuation, brackets, operators) to font.h
    using a struct-based lookup table
  • Replace manual pixel-drawn symbols (>, -, .) with proper bitmap font data
  • Add plain text editor app with full keyboard input via PS/2 scancode
    table, cursor navigation (arrow keys), character insertion, backspace
    with line joining, and enter for newlines
  • Editor supports 16 rows x 43 columns of editable text, persists content
    when returning to menu

https://claude.ai/code/session_014qqjnv6Pt778mmNLjvibe3

- Fix Makefile: remove hardcoded macOS homebrew paths, use generic tool
  names; remove macOS-only -display cocoa flag
- Extract magic numbers into named constants (CHAR_WIDTH, TITLE_BAR_HEIGHT,
  WINDOW_PADDING, MENU_ITEM_SPACING)
- Add 28 symbol font glyphs (punctuation, brackets, operators) to font.h
  using a struct-based lookup table
- Replace manual pixel-drawn symbols (>, -, .) with proper bitmap font data
- Add plain text editor app with full keyboard input via PS/2 scancode
  table, cursor navigation (arrow keys), character insertion, backspace
  with line joining, and enter for newlines
- Editor supports 16 rows x 43 columns of editable text, persists content
  when returning to menu

https://claude.ai/code/session_014qqjnv6Pt778mmNLjvibe3
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