Skip to content

[stable34] fix(FilePreview): sanitize file name - #19229

Merged
nickvergessen merged 4 commits into
stable34from
backport/19228/stable34
Sep 3, 2026
Merged

[stable34] fix(FilePreview): sanitize file name#19229
nickvergessen merged 4 commits into
stable34from
backport/19228/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Sep 2, 2026

Copy link
Copy Markdown

Backport of #19228

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot Bot added bug feature: frontend πŸ–ŒοΈ "Web UI" client feature: upload & shares & voice πŸ“€πŸŽ™οΈ Sharing files into a chat and audio recordings AI assisted This PR contains AI-assisted commits 3. to review labels Sep 2, 2026
@backportbot backportbot Bot added this to the πŸ–οΈ Next Patch (34) milestone Sep 2, 2026
- only used in SharedItems.vue together with rowLayout, so redundant

Assisted-by: ClaudeCode:claude-sonnet-4-6
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- bidi-control unicode characters are not visible in browser, but causing confusion with inline rendering under certain conditions
- for rendered DOM: align with Files app, render file extension in separate tag, isolate unicode-bidi
- for titles and aria-labels: replace with underscore (same as downloaded file name)

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy force-pushed the backport/19228/stable34 branch from a522d8e to 7d066d4 Compare September 2, 2026 16:04
@Antreesy

Antreesy commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

/backport! to stable33

@Antreesy
Antreesy marked this pull request as ready for review September 2, 2026 16:05
@backportbot

This comment was marked as resolved.

@nickvergessen
nickvergessen merged commit 5ab5152 into stable34 Sep 3, 2026
55 of 58 checks passed
@nickvergessen
nickvergessen deleted the backport/19228/stable34 branch September 3, 2026 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review AI assisted This PR contains AI-assisted commits backport-request bug feature: frontend πŸ–ŒοΈ "Web UI" client feature: upload & shares & voice πŸ“€πŸŽ™οΈ Sharing files into a chat and audio recordings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants