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

NP++ 7.9.4 File doesn't save as extension type (win 7) #9657

Closed
EatDesk opened this issue Mar 15, 2021 · 10 comments
Closed

NP++ 7.9.4 File doesn't save as extension type (win 7) #9657

EatDesk opened this issue Mar 15, 2021 · 10 comments
Assignees

Comments

@EatDesk
Copy link

EatDesk commented Mar 15, 2021

Description of the Issue

Same issue as #9510

Should have been fixed by f96d81a unless I misunderstand?

Unfortunately the commit did not fix the issue on win 7 at least.

In Notepad++ 7.9.2 I can do "Save as", file name: "lol" and "Save as type": "Normal text file (*.txt)" and the output file will be "lol.txt". Choosing the same options in 7.9.4 and the output file is named "lol" with no extension.

I don't have the option "Set Save dialog file extension filter to *.*" checked.

Steps to Reproduce the Issue

Edit for exact steps:
ALT -> F -> A -> this for for the "Save As..." menu, then I type my filename and hit Return.

The CTRL+ALT+S shortcut will also replicate the issue.

Expected Behavior

Above

Actual Behavior

Above

Debug Information

Notepad++ v7.9.4 (64-bit)
Build time : Mar 15 2021 - 01:03:10
Path : C:\Program Files\Notepad++\notepad++.exe
Admin mode : ON
Local Conf mode : OFF
OS Name : Windows 7 Ultimate (64-bit)
OS Build : 7601.0
Current ANSI codepage : 1252
Plugins : mimeTools.dll NppConverter.dll NppExport.dll

@mere-human
Copy link
Contributor

Yes, it should have been fixed.
I've just tried on Windows 7 VM and I can't reproduce it. Unless, you didn't specify some important steps.
Here is what I did:

Windows7.mp4

Could you, please, record a video?

@EatDesk
Copy link
Author

EatDesk commented Mar 15, 2021

Yes, it should have been fixed.
I've just tried on Windows 7 VM and I can't reproduce it. Unless, you didn't specify some important steps.

Apologies, I should have noted the exact process. I always use keyboard shortcuts so the exact process to reproduce is:

ALT -> F -> A -> this for for the "Save As..." menu, then I type my filename and hit Return.

The CTRL+ALT+S shortcut will also replicate the issue.

@ArkadiuszMichalski
Copy link
Contributor

@EatDesk Can you try with fresh portable version? Maybe something is wrong with the settings. I can't reproduce in:

Notepad++ v7.9.4   (32-bit)
Build time : Mar 15 2021 - 01:00:14
Path : D:\_test\NPPClean\notepad++.exe
Admin mode : ON
Local Conf mode : ON
OS Name : Windows 7 Ultimate (64-bit) 
OS Build : 7601.0
Current ANSI codepage : 1250
Plugins : mimeTools.dll NppConverter.dll NppExport.dll 

@EatDesk
Copy link
Author

EatDesk commented Mar 15, 2021

@EatDesk Can you try with fresh portable version? Maybe something is wrong with the settings. I can't reproduce in:

Notepad++ v7.9.4   (32-bit)
Build time : Mar 15 2021 - 01:00:14
Path : D:\_test\NPPClean\notepad++.exe
Admin mode : ON
Local Conf mode : ON
OS Name : Windows 7 Ultimate (64-bit) 
OS Build : 7601.0
Current ANSI codepage : 1250
Plugins : mimeTools.dll NppConverter.dll NppExport.dll 

You're right it doesn't occur with a fresh portable version. However that allowed me to find the setting that was causing this, it was the default save file directory apparently: Settings > Preferences > Default Directory: C:\Users\EatDesk\Desktop

If I switch portable to that directory the bug also occurs as it does with my installed version.

If I change the Default Directory to C:\Users\EatDesk\Desktop\Test the bug doesn't occur in either my installed version or portable.

@mere-human
Copy link
Contributor

I've just tried the scenario with setting Desktop as the Default Directory locally.
I don't see the issue on Windows 10.
But I see the same issue on Windows 7.

Looking into it...

@mere-human
Copy link
Contributor

@EatDesk One more question.
Could you, please, check if the issue occurs when you:

  • Open a save dialog by any means (e.g. Alt > F > S)
  • Instead of pressing Return key, press Alt + S or press Save button with a mouse?

Because for me the issue occurs only when I press Return key.

mere-human added a commit to mere-human/notepad-plus-plus that referenced this issue Mar 19, 2021
Initialize file dialog window procedures later so that
all the controls have been created by this moment.

Fix notepad-plus-plus#9657
@mere-human
Copy link
Contributor

@EatDesk You can check the issue again with the build artifacts from the PR above (if you want).
The process is described here: https://github.com/notepad-plus-plus/notepad-plus-plus/wiki/Testing

@EatDesk
Copy link
Author

EatDesk commented Mar 19, 2021

@EatDesk You can check the issue again with the build artifacts from the PR above (if you want).
The process is described here: https://github.com/notepad-plus-plus/notepad-plus-plus/wiki/Testing

I tested the build and it is resolved from my testing, thanks for looking into it. It's quite amusing that my normal usage was the perfect chain to trigger the bug.

@EatDesk One more question.
Could you, please, check if the issue occurs when you:

* Open a save dialog by any means (e.g. _Alt > F > S_)

* Instead of pressing _Return_ key, press _Alt + S_ or press _Save_ button with a mouse?

Because for me the issue occurs only when I press Return key.

I don't know if you needed an answer to this anymore but yes I have the same findings.

@EatDesk EatDesk closed this as completed Mar 19, 2021
@mere-human
Copy link
Contributor

Thanks for confirmation.
But I think this issue should be closed after the PR got merged.

@EatDesk EatDesk reopened this Mar 19, 2021
@sasumner
Copy link
Contributor

Yes, if/when the change is accepted via the PR (and it should be), this issue will get closed as well.

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

Successfully merging a pull request may close this issue.

5 participants