Supports selecting file extensions, and adds export to PDF and PNG

[Lutra-Fs]

Pull Request Details

What issue does this PR address

• Add native functionality to convert SVG to PDF and PNG in app.

• Link to associated issue: #198

Checklist

Complete the check-list below to ensure your branch is ready for PR.

Flutter Style Guide: https://survivor.togaware.com/gnulinux/flutter-style.html

• [ ] Screenshots included in linked issue
• [x] Changes adhere to the style and coding guideline
• [x] No confidential information
• [x] No duplicated content
• [x] No lint check errors related to your changes (make prep or flutter analyze lib)
• [ ] Pre-exisiting lint errors noted: [HERE]
• [ ] Integration test make qtest.tmp screenshot included in issue
• [x] Tested on device:
  • [ ] Linux
  • [ ] MacOS
  • [x] Windows
• [x] Added two reviewers

Finalising

Once PR discussion is complete and reviewers have approved:

• [ ] Merge dev into the branch
• [ ] Resolve any conflicts
• [ ] Add one line summary into CHANGELOG.md
• [ ] Bump appropriate version number in pubspec.yaml
• [ ] Push to git repository and review
• [ ] Merge PR into dev (gjwgit)

[Lutra-Fs]

Hey! I tried running make qtest on my Windows machine but ran into some environment issues. I wanted to get back to this project, but I've been swamped with assignments between mid-semester break and the end of teaching period. Haven't had a chance to look at it at all during that time. Finally now that classes are over, I can pick it up again!

[Lutra-Fs]

7ce9c69c9040a4ee93ac942ce5b5198cf6f0a254 introduce pipe uses date command, which I get this error: cmd:

E:\rattleng>date +%Y%m%d%H%M%S
The system cannot accept the date entered.
Enter the new date: (dd-mm-yy) 

pwsh:

date +%Y%m%d%H%M%S
Get-Date: Cannot bind parameter 'Date'. Cannot convert value "+%Y%m%d%H%M%S" to type "System.DateTime". Error: "String '+%Y%m%d%H%M%S' was not recognized as a valid DateTime."

make:

✦ ❯ make qtest.tmp
process_begin: CreateProcess(NULL, date +%Y%m%d%H%M%S, ...) failed.
support/flutter.mk:307: pipe: Bad file descriptor

[Lutra-Fs]

Also I think it would be better to turn https://github.com/gjwgit/rattleng/blob/dev/lib/main.dart#L66 alwaysOnTop off to avoid the annoying pop up for integration test. I have read the comment but still don't get the idea why we force the app on top of the beginning.

[Lutra-Fs]

when running qtest I found https://github.com/gjwgit/rattleng/issues/359 appears again.

[Lutra-Fs]

Ref: https://github.com/gjwgit/rattleng/issues/550 Currently on my device, multiple unrelated integration tests are failing.

[gjwgit]

Testing through the makefile is a little problematic on Windows - not developed/tested yet.

Tests are mostly working okay on Linux.

macOS on M1 is having issues with testing (and async issues) though older Mac is okay.

Thanks for the updates - I'll review shortly.

[gjwgit]

Also I think it would be better to turn https://github.com/gjwgit/rattleng/blob/dev/lib/main.dart#L66 alwaysOnTop off to avoid the annoying pop up for integration test. I have read the comment but still don't get the idea why we force the app on top of the beginning.

I'll review this again - perhaps create an issue for this. It's a good point re the testing getting focus all the time because of this. An end user's expectation is that on starting the app it should be visible. I found that on starting the app it often ended up behind other windows on Linux.