Excellent Rob, you implemented all my wishes! Thanks a lot!
Very good that you shows also a pop-up 'Manually entered GPS data already exists.'
Thanks.
That's all folks!
Not so fast! I still see possibilities to make it even more user friendly:
Sorry, but it’s good enough for me and I have other, non-jAlbum, things that need to be done over the next few months.
The pop-up 'Create a Google Maps URL link for the selected video?' is superfluously, after all the user did select a tool named 'Create GMaps link for video'.
Anyone can rename the tool to what they want - I just haven’t done it, I may do much later.
Instead in the 'Processed video' window I think you should add text below the reported values:
'Save these values?' and add a Cancel button.
The whole point was to extract gps data, why would you not want to save the data? So, that stays as is.
I think too that you should remove the '+' characters in the Latitude and Longitude, like with currency these character are normally not used. See also the notation in the Google Earth way-points window. And I am sure that 'gpsLocation=51.800198, 5.247233' is accepted in all skins.
Ffmpeg extracts the data and it always shows a + or -, I just kept to what is given - and it saves bloating the code with more tests.
I like of course the new window 'Past GPS latitude and longitude into the fields below'. But if I enter values without a + sign, the next message 'Sorry, latitude needs to start with + or -' is incorrect, these values are normally reported without a + sign, only a – sign might be needed. All GPS applications will accept GPS values without a + sign.
As above, I’m just keeping the internal code simple by following the ffmpeg presented coordinate format - I can just use those pasted values without additional tests - is it likely to be a valid ordinate and not ‘my dog has fleas’.
And after you close this error message window, you should go back to the 'Past GPS latitude ...' window to give the user the opportunity to correct his input'
It is only a one off event, unless of course the user repeatedly pastes the same information expecting a different result.
But as said already the current version is already very good and I hope that David will add it to the distributed tools in the next jAlbum version!
Highly unlikely, especially given the name of the coder
