Skip to content

✨(frontend) expose the full last-update date - #2587

Open
fch-aa wants to merge 1 commit into
suitenumerique:mainfrom
fch-aa:feat/last-edit-full-date
Open

✨(frontend) expose the full last-update date#2587
fch-aa wants to merge 1 commit into
suitenumerique:mainfrom
fch-aa:feat/last-edit-full-date

Conversation

@fch-aa

@fch-aa fch-aa commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Fixes #1215

Purpose

Documents currently show their last update as a relative value, which does not
provide the exact timestamp when users need it. This PR addresses #1215 by
exposing the localized full date on hover and keyboard focus while preserving
the existing compact relative-date display.

The copy action is intentionally omitted following the maintainer discussion
on the issue.

Proposal

  • Reuse the existing useDate().formatDate localization utility.
  • Display the full date with the existing Cunningham Tooltip component.
  • Use focusable semantic <time> markup with a machine-readable value.
  • Cover the relative value, localized full date, hover, and keyboard focus.

Local validation completed:

  • yarn test: 55 collaboration-server tests and 306 Impress tests passed.
  • yarn lint: TypeScript and all frontend ESLint workspaces passed.
  • yarn app:build: Prettier, Stylelint, TypeScript, and production build passed.
  • gitlint --commits HEAD^..HEAD: passed.

Video

2026-08-14.16-33-49.mp4

External contributions

General requirements

CI requirements

  • I made sure that all existing tests are passing
  • I have signed off my commits with git commit --signoff (DCO compliance)
  • I have signed my commits with my SSH or GPG key (git commit -S)
  • My commit messages follow the required format: <gitmoji>(type) title description
  • I have added a changelog entry under ## [Unreleased] section (if noticeable change)

AI requirements

  • I used AI assistance to produce part or all of this contribution
  • I have read, reviewed, understood and can explain the code I am submitting
  • I can jump in a call or a chat to explain my work to a maintainer

Keep the relative timestamp in the document header while exposing the
localized full date through the existing tooltip on hover and keyboard focus.

Signed-off-by: fch-aa <21101725+fch-aa@users.noreply.github.com>
@fch-aa
fch-aa marked this pull request as ready for review August 14, 2026 14:36
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a02e3f61-3a84-4ce0-b1ac-218916759b4b

📥 Commits

Reviewing files that changed from the base of the PR and between fb984ab and 127a427.

📒 Files selected for processing (3)
  • CHANGELOG.md
  • src/frontend/apps/impress/src/features/docs/doc-header/__tests__/DocHeaderInfo.spec.tsx
  • src/frontend/apps/impress/src/features/docs/doc-header/components/DocHeaderInfo.tsx

Walkthrough

DocHeaderInfo now shows the relative update date inside a tooltip with the full formatted date for active documents. The timestamp is focusable and has an accessible label. Deleted documents keep their remaining-days display. Tests cover English and French formatting, keyboard focus, hover behavior, and deterministic dates. The changelog records the frontend change.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to 127a4

This localized change adds an accessible full last-update date while preserving the existing relative display, with no actionable merge-blocking risk remaining after normal checks and review.

Suggested reviewers: antolc

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes exposing the full last-update date, which is the primary change in the pull request.
Description check ✅ Passed The description directly explains the full-date tooltip, keyboard accessibility, implementation choices, tests, and validation results.
Linked Issues check ✅ Passed The changes satisfy issue #1215 by exposing the precise last-edit date on hover; the suggested copy action is optional and intentionally omitted.
Out of Scope Changes check ✅ Passed The component change, focused tests, and changelog entry are directly related to the full last-update date objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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.

Make last edit full date available

1 participant