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

Minor bug when typing "cd \" #1206

Closed
Skaruts opened this issue Oct 3, 2020 · 5 comments
Closed

Minor bug when typing "cd \" #1206

Skaruts opened this issue Oct 3, 2020 · 5 comments

Comments

@Skaruts
Copy link

Skaruts commented Oct 3, 2020

If you use the cd \ command, the console will think \ is a folder and enter it. Using dir in there will still show the contents of the root. And since TIC thinks you're in an actual folder, you can go back with cd ...

cd_bug

EDIT: I thought this bug was specific to the root, but it isn't. Seems to happen anywhere, as cd \ isn't even a valid command (I was assuming it was).


Also, when I noticed cd \ isn't a valid command, I realized there's another problem here (maybe I should open another issue about this?): the valid command is cd /, and in some keyboards that's a bit unwieldy to use. In my case, I have one / on shift+7 and the other / on the number pad, which TIC doesn't support.

Maybe TIC could support cd \ to workaround this problem? \ in my case is next to 1, which is quite accessible.

This is how my keyboard looks like. This is the most common layout I've seen in portugal, where I'm from.

I guess I might add as a suggestion that TIC could possibly even support cd\ and cd/, for the same reasons as #1150.

@Skaruts Skaruts changed the title Minor bug in "cd \" command when used at the root Minor bug in "cd \" command Oct 3, 2020
@Skaruts Skaruts changed the title Minor bug in "cd \" command Minor bug when typing "cd \" Oct 4, 2020
@txgruppi
Copy link
Contributor

I can't reproduce this on TIC-80 0.80.1345 on Linux.

When running cd \ I get dir doesn't exist.

Is this a windows only issue?

@nesbox
Copy link
Owner

nesbox commented Apr 15, 2021

Reproduced on 0.80 and seems it's already fixed on 0.90
Closing.

@nesbox nesbox closed this as completed Apr 15, 2021
@Skaruts
Copy link
Author

Skaruts commented Apr 15, 2021

@nesbox was it fixed in 0.90 to throw an error, or to be a usable command?

@nesbox
Copy link
Owner

nesbox commented Apr 16, 2021

Just fixed the /\/\/\/\/\> fence and TIC supports cd / command only, don't want to complicate it.

@Skaruts
Copy link
Author

Skaruts commented Aug 16, 2023

It's amazing that meaningless features of the just-because-linux-does-it kind, like #1742, seem to be necessary and don't complicate things, but a cd \, which would actually add a tiny bit of comfort navigating the console with different keyboards, would've been an unnecessary complication...
It's only by sheer coincidence that #1742 also adds the same comfort, almost two years later.

By the way, the cd \ bug returned in the latest releases, using the winxp version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants