Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A folder with the name '&' shows blank in the 'Recent locations' dropdown list #3955

Closed
v-limu opened this issue Jan 5, 2021 · 4 comments
Closed
Assignees
Labels
⚙️ blobs Related to blob storage ⚙️ files Related to file storage ❔ external Root cause of this issue is in another component, product, or service 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Milestone

Comments

@v-limu
Copy link

v-limu commented Jan 5, 2021

Storage Explorer Version: 1.17.0
Build: 20201222.4
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu16.04 / MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression

Steps to Reproduce

  1. Expand one storage account -> File Shares.
  2. Create a file share -> Click 'New Folder' -> Input '&' as the new folder name -> Click 'OK'.
  3. Open the folder -> Click 'Recent locations'.
  4. Check whether the folder name '&' shows in the 'Recent locations' dropdown list.

Expected Experience

The folder name '&' shows well in the 'Recent locations' dropdown list.

Actual Experience

The folder name shows blank in the 'Recent locations' dropdown list.
image

More Info:
This issue also reproduces for one blob container.

@v-limu v-limu added 🧪 testing Found through regular testing ⚙️ files Related to file storage ⚙️ blobs Related to blob storage labels Jan 5, 2021
@MRayermannMSFT MRayermannMSFT added the ❔ external Root cause of this issue is in another component, product, or service label Jan 11, 2021
@craxal
Copy link
Contributor

craxal commented Jan 26, 2021

This is an Electron issue, which can be tracked here: electron/electron#25028.

@craxal craxal added this to the Near Future milestone Jan 26, 2021
@MRayermannMSFT MRayermannMSFT modified the milestones: Near Future, future Feb 11, 2021
@craxal
Copy link
Contributor

craxal commented Feb 19, 2021

Electron folks are updating their documentation. We can escape the & with another &.

@craxal craxal modified the milestones: future, 1.18.0 Feb 19, 2021
@craxal craxal added the ✅ merged A fix for this issue has been merged label Feb 22, 2021
@craxal craxal closed this as completed Feb 22, 2021
@v-xianya
Copy link
Member

Hi @craxal
Verified this issue on build 20210223.2. There are some problems on this scenario.
We are going to re-open it. Could you help to take a look?

The folder named '&&' show as '&' under the recent locations;
image

The folder named '&&&&' show as '&&' under the recent locations;
image

@v-xianya v-xianya reopened this Feb 23, 2021
@craxal
Copy link
Contributor

craxal commented Feb 23, 2021

Argh! Apparently String.replace rather unintuitively replaces the first occurrence of the pattern unless the pattern is a global regular expression. In other words:

"&&".replace("&", "&&"); // Returns "&&&", not what we want
"&&".replace(/&/g, "&&"); // Returns "&&&&", which is what we want

Change has been merged.

@craxal craxal closed this as completed Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ blobs Related to blob storage ⚙️ files Related to file storage ❔ external Root cause of this issue is in another component, product, or service 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Projects
None yet
Development

No branches or pull requests

4 participants