Skip to content

fix(autofocus): persist cursor positions in localStorage#8723

Merged
max-nextcloud merged 2 commits into
mainfrom
fix/autofocus_local_storage
Jun 24, 2026
Merged

fix(autofocus): persist cursor positions in localStorage#8723
max-nextcloud merged 2 commits into
mainfrom
fix/autofocus_local_storage

Conversation

@mejo-

@mejo- mejo- commented Jun 9, 2026

Copy link
Copy Markdown
Member

Fixes: #8699

This makes the autofocus feature much more usable, as it persists the cursor position per device, not only per browser tab.

Also migrates Autofocus extension to TypeScript.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

🤖 AI (if applicable)

  • The content of this PR was partly generated using AI tools
  • The AI-generated content was reviewed, comprehended and tested by a human

@mejo- mejo- self-assigned this Jun 9, 2026
@mejo- mejo- added bug Something isn't working 3. to review labels Jun 9, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Jun 9, 2026
@mejo- mejo- moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team Jun 9, 2026
@mejo-

mejo- commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

/backport to stable34

@mejo-

mejo- commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

/backport to stable33

@max-nextcloud max-nextcloud left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Code changes look good. I wonder if the playwright failures are related though.

@max-nextcloud

Copy link
Copy Markdown
Collaborator

Saw the same timeout failure elsewhere. So I suspect it's not related.

@max-nextcloud max-nextcloud force-pushed the fix/autofocus_local_storage branch from 7527472 to b5b11bc Compare June 24, 2026 08:04
@max-nextcloud max-nextcloud enabled auto-merge June 24, 2026 08:04
mejo- added 2 commits June 24, 2026 11:07
Signed-off-by: Jonas <jonas@freesources.org>
Fixes: #8699

Signed-off-by: Jonas <jonas@freesources.org>
@max-nextcloud max-nextcloud force-pushed the fix/autofocus_local_storage branch from b5b11bc to 3772580 Compare June 24, 2026 09:07
@max-nextcloud max-nextcloud disabled auto-merge June 24, 2026 12:04
@max-nextcloud max-nextcloud merged commit e886598 into main Jun 24, 2026
84 of 90 checks passed
@max-nextcloud max-nextcloud deleted the fix/autofocus_local_storage branch June 24, 2026 12:04
@github-project-automation github-project-automation Bot moved this from 👀 In review to ☑️ Done in 📝 Productivity team Jun 24, 2026
@max-nextcloud

Copy link
Copy Markdown
Collaborator

Playwright test is known to fail.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

Remember where the user left off

2 participants