Skip to content

Persist actions log time display settings in localStorage#36623

Merged
techknowlogick merged 3 commits into
go-gitea:mainfrom
silverwind:persistlog
Feb 15, 2026
Merged

Persist actions log time display settings in localStorage#36623
techknowlogick merged 3 commits into
go-gitea:mainfrom
silverwind:persistlog

Conversation

@silverwind
Copy link
Copy Markdown
Member

@silverwind silverwind commented Feb 14, 2026

Persist the two boolean settings in the actions log into localStorage so that they are remembered across page reloads.

image

The "Show Log Seconds" and "Show Timestamps" toggles in the actions log
viewer were not persisted across page reloads. Add them to the existing
localStorage-backed settings so they survive navigation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 14, 2026
@silverwind silverwind added the type/enhancement An improvement of existing functionality label Feb 14, 2026
@silverwind silverwind requested a review from Copilot February 14, 2026 12:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds persistence for the actions log time display settings using localStorage. Previously, the two boolean settings for showing seconds and timestamps in action logs were reset on every page reload. Now they are saved to localStorage and restored when the user returns to the page.

Changes:

  • Extended LocaleStorageOptions type to include actionsLogShowSeconds and actionsLogShowTimestamps
  • Modified initialization to load these settings from localStorage with appropriate defaults (both false)
  • Updated saveLocaleStorageOptions() to persist the time visibility settings alongside existing auto-scroll and expand-running options

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@silverwind silverwind added the topic/gitea-actions related to the actions of Gitea label Feb 14, 2026
Comment thread web_src/js/components/RepoActionView.vue Outdated
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Feb 14, 2026
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Feb 15, 2026
@techknowlogick techknowlogick enabled auto-merge (squash) February 15, 2026 19:39
@techknowlogick techknowlogick added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Feb 15, 2026
@techknowlogick techknowlogick merged commit 26bb175 into go-gitea:main Feb 15, 2026
24 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Feb 15, 2026
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Feb 15, 2026
@silverwind silverwind deleted the persistlog branch February 15, 2026 19:47
silverwind added a commit to silverwind/gitea that referenced this pull request Feb 16, 2026
…#36623)

Persist the two boolean settings in the actions log into `localStorage`
so that they are remembered across page reloads.

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/gitea-actions related to the actions of Gitea type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants