Skip to content

Add setting to show timestamps on every message - #1942

Merged
7w1 merged 2 commits into
SableClient:devfrom
cwyptt:feat/show-timestamps-toggle
Aug 27, 2026
Merged

Add setting to show timestamps on every message#1942
7w1 merged 2 commits into
SableClient:devfrom
cwyptt:feat/show-timestamps-toggle

Conversation

@cwyptt

@cwyptt cwyptt commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Description

By default, Sable only shows a timestamp on the first message in a group of consecutive messages from the same sender. This adds a showAllTimestamps setting (default: off) that renders a timestamp on every message, even grouped/collapsed ones. The toggle is in Settings → General → Messages, below "Message Spacing".

When enabled, collapsed messages render a right-aligned timestamp in place of the empty header, without the username, avatar, or pronouns that appear on ungrouped messages. The layout direction respects the same Row/RowReverse logic as the full header (compact and right-aligned bubble layouts).

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

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 made corresponding changes to the documentation
  • My changes generate no new warnings

AI disclosure:

  • Partially AI assisted (clarify which code was AI assisted and briefly explain what it does).
  • Fully AI generated (explain what all the generated code does in moderate detail).

The implementation was AI-assisted: adding the showAllTimestamps boolean to the Settings interface and defaults, reading it via useSetting in Message.tsx and conditionally rendering a timestamp-only header for collapsed messages, and adding the SettingToggle in General.tsx. I reviewed each change, verified the collapsed header layout matches the existing direction logic, and ran typecheck, lint, format, and tests in a containerized environment.

@cwyptt
cwyptt force-pushed the feat/show-timestamps-toggle branch from a067352 to d5c1e5e Compare August 25, 2026 13:57
@cwyptt cwyptt changed the title Add setting to toggle message timestamps in the timeline Add setting to show timestamps on every message Aug 25, 2026
@Th-Underscore

Th-Underscore commented Aug 26, 2026

Copy link
Copy Markdown

A very similar feature is included in my local branch I'll PR in the near future once I manually clean and review everything; the timestamp appears on hover but has no opt-in config: dev...Th-Underscore:Sable:feat/message-edit-timestamps

Let me know if this PR #1942 gets merged soon so I can remove the timestamps bit from my feature set.

7w1

This comment was marked as outdated.

@7w1 7w1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

oops didn't check ci. The settings link file needs to be updated https://github.com/SableClient/Sable/blob/dev/src/app/features/settings/settingsLink.ts

@cwyptt
cwyptt force-pushed the feat/show-timestamps-toggle branch from 1a50afc to 610c139 Compare August 27, 2026 17:44
@cwyptt

cwyptt commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

ahhh, thank you @7w1; should be g2g now

@7w1
7w1 added this pull request to the merge queue Aug 27, 2026
Merged via the queue into SableClient:dev with commit 89043a4 Aug 27, 2026
13 checks passed
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.

3 participants