Improve the UI for MainWindow::askToSave
ClosedPublic

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

Details

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


Diff Detail

Branch
ImproveSaveDialogUI
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 5286
Build 5304: 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.
src/mainwindow.cpp
432–436

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