darosh / samples-piano-mp3

Salamander Grand Piano V3 MP3 samples

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

unable to navigate to URL with all 'D#' notes and 'F#' notes (due to the # symbol)

erichlof opened this issue · comments

Hello! First, thank you for this very helpful resource! I encountered an issue when trying to view/use/download any notes that are sharped - i.e. all D# and F# pitches. This is apparently due to the fact that if a # hashtag symbol is used in the url, it instructs browsers to do something not intentional to your file system.

After researching a bit (I am still learning about all the intricacies of web/html programming), I was finally able to navigate to the file and view it (listen to the .mp3) in my browser by replacing the hashtag symbol with the following characters: %23 (that's percent 2 3). This is apparently a character escape sequence and the url is correctly navigated to. So for instance, here is the original non-working link and here is the corrected working link

I don't know how much of a pain it would be to update all of your D# and F# notes via your package system. What the authors of Tone.js did to get around this browser url quirk is to simply rename the D#4 and F#4 for instance to Ds4 and Fs4 so there is no issue at all. Although I did at first think it was Ds as in multiple plural D's, ha ha. I don't know, maybe Dsharp4 would be less ambiguous but slightly more verbose.

Thanks again for this valuable resource!
-Erich

Hi @erichlof

The original sample files are named with #.

There is
image
for that.

I guess the file name and its URI are both valid.
Maybe just a notice about URI encoding in a readme could the job? (Instead of renaming files and explaining Ds is/are D# :-)

@darosh
Oh cool - I wasn't aware of the encodeURICompenent. Yes I agree, a notice about this in a readme would do the trick!
I'll go ahead and close out this issue. Thanks again!

@erichlof Note added to every readme. Thank you!