fix: Prevent text copying when clicking replay settings tabs

[naaa760]

I made a PR!

#35564

Changes

• Removed clipboard copy logic from settings tab navigation.
• Now, clicking a settings tab only navigates—no more "Copied to clipboard" toast or unwanted clipboard content.

How did you test this code?

• Manually tested by clicking all settings sidebar tabs and confirming no clipboard or toast appears.
• Verified navigation still works as expected.

[greptile-apps]

Greptile Summary

This PR addresses a clipboard copying bug in replay settings tabs while also including unrelated activity log refactoring. The main fix removes clipboard copy logic from settingsSceneLogic.ts that was unintentionally triggering when users clicked on settings tabs, causing unwanted "Copied to clipboard" toasts and clipboard pollution. The navigation functionality is preserved through existing URL routing.

Additionally, the PR includes several activity log-related changes that appear unrelated to the main fix:

• Adds a new utility function is_activity_unread() in posthog/api/utils.py for comparing activity timestamps with bookmark dates
• Refactors ActivityLogSerializer and MyNotificationsSerializer to use this shared utility instead of duplicated inline logic
• Adds comprehensive unit tests for the new utility function
• Updates SessionRecordingsPlaylist with FloatingContainerContext infrastructure

The activity log changes eliminate code duplication by centralizing timestamp comparison logic that determines if activities are unread based on user bookmark dates. The microsecond precision handling that was previously duplicated across serializers is now consolidated in the utility function.

PR Description Notes:

• The issue reference format should use "Fixes #35564" to automatically close the linked issue

Confidence score: 3/5

• This PR mixes unrelated changes which reduces clarity and makes review more complex
• The main clipboard fix appears solid, but the activity log changes seem disconnected from the stated purpose
• Most changes are low-risk refactoring, but mixing concerns makes it harder to assess the complete impact
• Files posthog/api/utils.py, posthog/api/activity_log.py, posthog/api/my_notifications.py, and posthog/api/test/test_utils.py need more attention as they contain the unrelated activity log functionality

_{6 files reviewed, 1 comment}

_{Edit Code Review Bot Settings | Greptile}

[greptile-apps]

style: Consider moving datetime imports to the top with other standard library imports for better organization