MantisBT - NetSurf
View Issue Details
0002185NetSurfAmiga-specificpublic2014-08-23 18:182016-02-16 15:27
ReporterSamir Hawamdeh 
Assigned ToChris Young 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformAmigaOSOSOS Version4.1
Product Version3.1 
Target Version3.4Fixed in Version3.4 
Fixed in CI build #2801
Reported in CI build #
URL of problem page
Summary0002185: Save as IFF option still save images as PNG
DescriptionThe Save as IFF option in reality still save images as PNG
TagsNo tags attached.
Attached Files

Notes
(0000451)
Chris Young   
2014-08-23 18:58   
"Save as IFF" is definitely saving bitmaps as IFF ILBM (it should be saving SVGs as IFF DR2D too, but I haven't double-checked that). However, the default filename in the requester is the file's original filename with extension. This needs stripping off and changing to .iff (or less generically, .ilbm or .dr2d).
(0000507)
Chris Young   
2014-10-10 15:19   
The larger issue here is that generating filenames should probably be done using url_nice(), which none of the Amiga frontend code is using.
(0000512)
Chris Young   
2014-10-23 19:00   
Now using url_nice to generate initial filenames. Note this is stripping the extension always, not just when saving in a different format. I'm undecided as to whether this is a problem or not (as AmigaOS doesn't care).
(0000859)
Chris Young   
2015-06-22 18:43   
Now strips the file extension (if not saving as source) and adds a more appropriate one.
(0001310)
Vincent Sanders   
2016-02-16 15:27   
Confirmed resolved in 3.4 release

Issue History
2014-08-23 18:18Samir HawamdehNew Issue
2014-08-23 18:54Chris YoungAssigned To => Chris Young
2014-08-23 18:54Chris YoungStatusnew => assigned
2014-08-23 18:58Chris YoungNote Added: 0000451
2014-08-23 18:58Chris YoungStatusassigned => confirmed
2014-08-23 18:58Chris YoungTarget Version => 4.0
2014-10-10 15:19Chris YoungNote Added: 0000507
2014-10-23 19:00Chris YoungNote Added: 0000512
2014-10-25 22:46Chris YoungStatusconfirmed => feedback
2015-06-22 18:43Chris YoungFixed in CI build # => 2801
2015-06-22 18:43Chris YoungNote Added: 0000859
2015-06-22 18:43Chris YoungStatusfeedback => resolved
2015-06-22 18:43Chris YoungFixed in Version => 3.4
2015-06-22 18:43Chris YoungResolutionopen => fixed
2015-06-23 10:14Vincent SandersTarget Version4.0 => 3.4
2016-02-16 15:27Vincent SandersNote Added: 0001310
2016-02-16 15:27Vincent SandersStatusresolved => closed