Bug 404162 File/Save acts always acts like "Save as..."
ClosedPublic

Authored by krutovmikhail on Mar 14 2019, 7:44 PM.

Details

Summary

Slight reformatting and fixing the Save As junior job bug

Diff Detail

Repository
R337 KTurtle
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
krutovmikhail created this revision.Mar 14 2019, 7:44 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptMar 14 2019, 7:44 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
krutovmikhail requested review of this revision.Mar 14 2019, 7:44 PM

Updating D19769: Bug 404162 File/Save acts always acts like "Save as..."

Updating D19769: Bug 404162 File/Save acts always acts like "Save as..."

aacid added a subscriber: aacid.Mar 14 2019, 8:51 PM
aacid added inline comments.
src/editor.cpp
262–263

indentation on lines 263 and 265 feels weird, there's two } on the same horizontal line, probably the one above needs to be indented?

apol added a subscriber: apol.Mar 15 2019, 6:00 PM
apol added inline comments.
src/editor.cpp
248

We shouldn't use exclamation marks on GUIs.
https://hig.kde.org/style/writing/wording.html

  • Update per phabricator comments: message wording & indention
aacid accepted this revision.Mar 17 2019, 10:54 PM

Awesome job :)

I'll land it just now.

This revision is now accepted and ready to land.Mar 17 2019, 10:54 PM
This revision was automatically updated to reflect the committed changes.