microsoft/Terminal

Terminal should always wrap drag-dropped paths in double quotes

Open

#8.109 geöffnet am 30. Okt. 2020

Auf GitHub ansehen
 (23 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)C++ (3.212 Forks)batch import
Area-TerminalControlHelp WantedIssue-TaskPriority-3Product-Terminal

Repository-Metriken

Stars
 (35.764 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 27T 19h) (24 gemergte PRs in 30 T)

Beschreibung

Environment

Microsoft Windows [版本 10.0.18363.1139] Windows Terminal [ powershell] 版本: 1.3.2651.0 python 3.7

Steps to reproduce

1.write the followings code xx.py import sys with open(sys.argv[1]) as f: pass 2.create a txt as "xxx(1).txt", the name must include"()" 3:open windows terminal powershell, cd to the xx.py directory input “python xx.py” drop the txt into terminal run 4.if the filename contain "(xx)" can not resolve the path.

Expected behavior

it worked in cmd open the file success

Actual behavior

FileNotFoundError: [Errno 2] No such file or directory: 'C:\Users\xx\Desktop\\input'

Contributor Guide

Research-Richtung
Verfolgen Sie den Drag and Drop Ereignishandler im Terminal Codebasis, wahrscheinlich in `TerminalPage.cpp` oder `DropHandler.cpp`, um zu sehen, wo Pfade in den Eingabepuffer eingefügt werden, und stellen Sie sicher, dass sie in doppelte Anführungszeichen gesetzt werden, wenn sie Leerzeichen oder Klammern enthalten.
Tech Stack
cpp
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65