Improve the UI for MainWindow::askToSave

Authored by rizzitello on Nov 23 2018, 4:21 AM.


Change "Ignore" to "Discard Changes" to be consistent with kate
Disable Save Selected until item is selected
Test Plan

Diff Detail

R231 Atelier
No Linters Available
No Unit Test Coverage
Build Status
Buildable 5326
Build 5344: arc lint + arc unit
rizzitello requested review of this revision.Nov 23 2018, 4:21 AM
rizzitello created this revision.
rizzitello edited the summary of this revision. (Show Details)Nov 23 2018, 4:23 AM
rizzitello edited the test plan for this revision. (Show Details)
rizzitello added a project: Atelier.
rizzitello added a subscriber: Atelier.
tcanabrava requested changes to this revision.Nov 23 2018, 10:33 AM
tcanabrava added inline comments.

sabeBtn->setEnabled(currentRow >= 0); ?

This revision now requires changes to proceed.Nov 23 2018, 10:33 AM
rizzitello updated this revision to Diff 46077.Nov 23 2018, 2:47 PM
rizzitello marked an inline comment as done.
  • Tomaz Suggestion
rizzitello set the repository for this revision to R231 Atelier.Nov 23 2018, 2:55 PM
tcanabrava accepted this revision.Nov 23 2018, 2:57 PM
This revision is now accepted and ready to land.Nov 23 2018, 2:57 PM
patrickelectric accepted this revision.Nov 24 2018, 5:56 PM
rizzitello updated this revision to Diff 46149.Nov 24 2018, 6:28 PM
  • Remove new icon for delete use standard icon instead for fallback
rizzitello edited the summary of this revision. (Show Details)Nov 24 2018, 6:29 PM
rizzitello closed this revision.Nov 24 2018, 6:30 PM