Skip to content

PR: Initial file dialog definition to select a remote directory (Remote Client)#25972

Open
dalthviz wants to merge 4 commits intospyder-ide:masterfrom
dalthviz:fixes_issue_12828_p2
Open

PR: Initial file dialog definition to select a remote directory (Remote Client)#25972
dalthviz wants to merge 4 commits intospyder-ide:masterfrom
dalthviz:fixes_issue_12828_p2

Conversation

@dalthviz
Copy link
Copy Markdown
Member

@dalthviz dalthviz commented Apr 20, 2026

Description of Changes

  • Added a test
  • Wrote at least one-line docstrings (for any new functions)
  • Included a screenshot or animation (if affecting the UI, see Licecap)
remote_files_chooser
  • Add RemoteFileDialog class definition with a static method to retrieve a directory path
  • Add a kwarg to the RemoteExplorer to show only directories

Issue(s) Resolved

Part of #12828

Affirmation

By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: dalthviz

@ccordoba12
Copy link
Copy Markdown
Member

ccordoba12 commented Apr 22, 2026

@dalthviz, quick feedback according to the screenshot you posted:

  • Headers (i.e. Name and Size) shouldn't be visible.

  • The tree lines to the left shouldn't be visible either.

  • Folder and file icons should be a bit bigger (perhaps 22px) to display them differently from the regular remote file explorer

  • Ideally they should be arranged in a grid (but that's not a must), like this (from the KDE file dialog)

    image

@dalthviz dalthviz force-pushed the fixes_issue_12828_p2 branch from 1ba548b to 12b42f4 Compare April 22, 2026 21:00
@dalthviz dalthviz marked this pull request as ready for review April 22, 2026 22:33
@dalthviz dalthviz requested a review from ccordoba12 April 22, 2026 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants