Skip to content

Improve recent dialog path readability - #2567

Merged
bajrangCoder merged 2 commits into
mainfrom
feat/improve-recent-dialog-paths
Aug 2, 2026
Merged

Improve recent dialog path readability#2567
bajrangCoder merged 2 commits into
mainfrom
feat/improve-recent-dialog-paths

Conversation

@bajrangCoder

Copy link
Copy Markdown
Member

Summary

Improves the recent files and folders dialog so entries remain identifiable on mobile

Changes

  • Display the file or folder name as the primary text
  • Display its location as secondary text
  • Show complete paths when space is available
  • Truncate overflowing paths from the left, preserving the useful ending
  • Decode SAF document paths into readable locations
  • Preserve absolute Termux paths and their leading /
  • Use configured storage aliases for concise primary labels
  • Increase the dialog width responsively
  • Improve spacing and remove-button touch targets
  • Expose complete paths through accessibility labels and hover titles
  • Extend select-dialog items with subtitles and custom styling hooks
  • Add regression coverage for primary storage and Termux SAF URIs

@greptile-apps

greptile-apps Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Improves recent-path readability and selection-dialog presentation.

  • Separates recent entry names from their containing locations.
  • Adds readable SAF paths with storage aliases and preserved absolute Termux paths.
  • Adds responsive dialog styling, subtitles, accessibility labels, and larger removal targets.
  • Adds URI regression coverage for storage roots, prefix-sharing roots, and Termux paths.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
src/lib/recents.js Recent entries now derive distinct primary names, parent locations, and complete accessible paths; the previously reported repeated-filename issue is resolved.
src/utils/Uri.js Adds readable SAF path formatting with exact root-or-descendant alias matching, resolving the previously reported partial-root match.
src/dialogs/select.js Extends select items with subtitles, titles, accessibility labels, and custom classes.
src/dialogs/style.scss Adds responsive recent-dialog layout, left-truncated location text, and larger entry controls.
tests/unit/uri.test.js Covers storage aliases, prefix-sharing roots, volume roots, fallback paths, and absolute Termux URIs.

Reviews (2): Last reviewed commit: "fix" | Re-trigger Greptile

Comment thread src/lib/recents.js Outdated
Comment thread src/utils/Uri.js
@bajrangCoder

This comment was marked as outdated.

@bajrangCoder
bajrangCoder added this pull request to the merge queue Aug 2, 2026
Merged via the queue into main with commit 0dd1076 Aug 2, 2026
13 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Aug 2, 2026
@bajrangCoder
bajrangCoder deleted the feat/improve-recent-dialog-paths branch August 2, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant