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

Incorrect context menu #20

Closed
GibMeMyPacket opened this issue Jan 22, 2023 · 16 comments
Closed

Incorrect context menu #20

GibMeMyPacket opened this issue Jan 22, 2023 · 16 comments
Labels
bug Something isn't working Done

Comments

@GibMeMyPacket
Copy link

Hello,
using [srt,mkv,mp4] folder to add context menu for srt, mkv and mp4 files.
But it is showing the context menu everywhere!
image

image

@oleg-shilo oleg-shilo added the bug Something isn't working label Jan 24, 2023
@oleg-shilo
Copy link
Owner

I am investigating it. Thank you

@oleg-shilo
Copy link
Owner

Fixed.
I will also address #22 and will release it. (most likely in a day or two)

oleg-shilo added a commit that referenced this issue Feb 4, 2023
- Some improvements to minimize the frequency of #22
- Issue #20: Incorrect context menu
@oleg-shilo
Copy link
Owner

Published on Chocolatey: https://community.chocolatey.org/packages/shell-x/1.5.3.0

Will be available as soon as Chocolatey moderation is done

@masterofobzene
Copy link

masterofobzene commented Mar 14, 2023

Found a bug on the new release, if you set the folder as for example [mp4,mov,mpg,mpeg,avi,mkv,webm,webp,bmp,jpeg,jpg,tiff,gif] and select many files (like all jpg's) the menu doesn't show.

It does work with [any]

oleg-shilo added a commit that referenced this issue Mar 19, 2023
- Issue #20: Incorrect context menu
@oleg-shilo
Copy link
Owner

Please try now

@masterofobzene
Copy link

Please try now

Same behaviour, doesn't work when selecting more than one item.

@oleg-shilo
Copy link
Owner

image

image


Is it possible that your explorer did not restart after the upgrade?

@masterofobzene
Copy link

masterofobzene commented Mar 22, 2023

Try selecting diferent extension files and see if it works for you please I found that the problem happens when you select different extension files

here is my config folder:
[mp4,mov,mpg,mpeg,avi,mkv,webm,webp,bmp,jpeg,jpg,tiff,gif]

Selecting two different (but allowed) extensions causes the menu to not show up.

@oleg-shilo
Copy link
Owner

Strange...
image

@masterofobzene
Copy link

masterofobzene commented Mar 23, 2023

In the video you can see that when selecting different extensions the menu doesn't show up. I'm on Windows 10, don't know you (using the legacy menu?).

Also I noted you still have enabled the [any] folder, could that be causing it to work? do you have anything in that folder?

2023-03-23.10-04-36.mp4

2023-03-23 10-08-36

@masterofobzene
Copy link

EDIT: BINGO! it was because I disabled the [any] folder. What needs to be done is to leave an empty [any] folder to make it work with different extensions, otherwise when you select many filetypes the menu will not be shown.

@oleg-shilo
Copy link
Owner

oleg-shilo commented Mar 24, 2023

Also I noted you still have enabled the [any] folder

Correct. [any] is required to give the user convenient access to the shell-x configuration folder:

image

Saying that, the dependency on [any] is not intentional. I will address it but until then (as a workaround) please keep [any] enabled.

@masterofobzene
Copy link

masterofobzene commented Mar 26, 2023

Just found another bug: on long filenames (230 characters), the menu is not being shown.
(I'm not naming the files like that, they come named like that from internet)

@oleg-shilo
Copy link
Owner

oleg-shilo commented Oct 16, 2023

It is a limitation of the operating system.
IE you cannot even create a file with such a long name (248 chars) if you are using .NET for that.

image

@masterofobzene
Copy link

Weird cause I've patched the system to allow long names... in fact they are being shown and can rename them in the explorer. Anyway I take it as a limitation and not a bug.

@oleg-shilo
Copy link
Owner

The MAX_PATH limitation has been known for more than a decade.
Though it's not a universal quality of WIN file system but only one of its subsystems (eg explorer).
To add even more confusion to this, consider the fact that I have two Win10 systems (different builds) where one has the code above failing to create the file and another one successfully creating it. However in this case the shell extension is failing even to start.

Only some applications, that can act on a very low level of file system can overcome this problem because the limit is imposed on file creation but not renaming (afaik).

You also can enable Win32 long path support from group policy, providing you have permission.
Then it is expected that all IO libraries will start supporting long-names.

Anyway, Win is still suffering from this problem so its' problematic to address it from the application like shell-x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Done
Projects
None yet
Development

No branches or pull requests

3 participants