| gregormi | |
| ngraham |
| Spectacle |
If Save As was never used before and the Save button was
showing the Save As action, Spectacle would return Pictures by
default as the last used "Save As" location, while in all other cases it
would be Folder/. The missing / resulted in the filemanager
highlighting the folder only, instead of opening it.
The fix simply makes lastSaveAsLocation() consistently return URLs
with a / at the end.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Huh, interesting corner case. Can confirm original problem as well as that it's fixed with this patch. Thanks!
Actually just a chaotic codebase, sometimes returning QUrl, sometimes QString, sometimes with /, sometimes without… That's why it's so important to review in depth and only commit "nice" code.