Skip to content

feat: add latest work log editing#10

Merged
okaryo merged 4 commits into
mainfrom
feature/ammend-log
May 22, 2026
Merged

feat: add latest work log editing#10
okaryo merged 4 commits into
mainfrom
feature/ammend-log

Conversation

@okaryo
Copy link
Copy Markdown
Owner

@okaryo okaryo commented May 22, 2026

Summary

  • Add a shortcut to edit the latest Work Log entry
  • Keep the original Work Log timestamp when editing
  • Show Work Log date headings in English regardless of system locale
  • Centralize app date formatting for consistent display

Checks

  • pnpm run format:check
  • pnpm run lint
  • pnpm run check
  • pnpm test
  • pnpm run build

@okaryo okaryo self-assigned this May 22, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2e24d17e22

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 158 to +161
event.preventDefault();
event.stopPropagation();
textareaElement?.blur();
cancelEditingWorkLog();
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Clear edit state when closing the entry panel

This cancellation logic only runs on Escape, so if the panel closes via focus loss (e.g., clicking another section or switching sections with shortcuts), editingWorkLog remains set. Reopening the input then stays in locked edit mode and Cmd+Enter updates the old log instead of creating a new entry, which can cause accidental overwrites; the edit state should be cleared on all close paths, not just the Escape handler.

Useful? React with 👍 / 👎.

@okaryo okaryo merged commit c8f5f34 into main May 22, 2026
2 checks passed
@okaryo okaryo deleted the feature/ammend-log branch May 22, 2026 14:49
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