Bug 344760

Summary:	"save as" menu doesnt enforce filetypes from dropdown in a predictable manner
Product:	[Applications] krita	Reporter:	repto2 <f836524>
Component:	File formats	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	minor	CC:	halla
Priority:	NOR
Version:	2.9
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	http://commits.kde.org/calligra/4d3aa4addad4c6252e91a468ac61a8df39a303c1	Version Fixed In:

Description repto2 2015-03-02 20:45:17 UTC

Hello folks, i want to file a minor bug report. Using the 2.9 windows build when you save a .kra and then "save as", leave the filename the same (no suffix) and change the file type in the dropdown, it will ask if you want to overwrite the .kra.
The expected behavior is to enforce the dropdown filetype. 

Reproducible: Always

Steps to Reproduce:
1. Save a .kra
2. Go to Save As, change the filetype of the new file but keep the filename (no suffix) the same
3. in the filetype dropdown choose another format, e.g. PNG

Actual Results:  
Krita asks to overwrite the .kra

Expected Results:  
Save a .png

Comment 1 Halla Rempt 2015-03-24 19:44:14 UTC

Hi,

Thanks for your report. To be honest, I don't know how to properly fix this. The integration of the native Windows file dialog gives a lot of problems (as does the integration of the native Gnome and OSX file dialogs -- if I weren't sure people would complain, I'd force the KDE file dialog on all platforms...) But I can confirm the issue in any case.

Comment 2 Halla Rempt 2015-08-25 14:38:43 UTC

Git commit 4d3aa4addad4c6252e91a468ac61a8df39a303c1 by Boudewijn Rempt.
Committed on 25/08/2015 at 14:38.
Pushed by rempt into branch 'calligra/2.9'.

File dialog: Set the full filename as default directory
Related: bug 342555

A bit of api confusion. Where the documentation talks about
default "default directory", it means "default filename including
path".

M  +3    -8    libs/widgets/KoFileDialog.cpp

http://commits.kde.org/calligra/4d3aa4addad4c6252e91a468ac61a8df39a303c1