Autres/audacity
1
0
Fork 0
mirror of https://github.com/cookiengineer/audacity synced 2025-04-30 15:49:41 +02:00
Code Issues Releases Wiki Activity

Extended import filter prefs GUI usability improvements

Browse Source 
* Select newly added item and switch focus to the table
* Show confirmation dialog when deleting items
* Be smart about keeping cursor and selection position when deleting items
This commit is contained in:
LRN1986 2010-04-04 22:07:57 +00:00
parent d2c0a72597
commit ded848d66e
1 changed files with 18 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/prefs/ExtImportPrefs.cpp
View File

@ -35,7 +35,7 @@ enum ExtImportPrefsControls
BEGIN_EVENT_TABLE(ExtImportPrefs, PrefsPanel) BEGIN_EVENT_TABLE(ExtImportPrefs, PrefsPanel)
EVT_LIST_KEY_DOWN(EIPPluginList,ExtImportPrefs::OnPluginKeyDown) EVT_LIST_KEY_DOWN(EIPPluginList,ExtImportPrefs::OnPluginKeyDown)
EVT_KEY_DOWN (ExtImportPrefs::OnRuleTableKeyDown) //EVT_KEY_DOWN (ExtImportPrefs::OnRuleTableKeyDown)
EVT_GRID_EDITOR_HIDDEN (ExtImportPrefs::OnRuleTableEdit) EVT_GRID_EDITOR_HIDDEN (ExtImportPrefs::OnRuleTableEdit)
EVT_GRID_SELECT_CELL (ExtImportPrefs::OnRuleTableSelect) EVT_GRID_SELECT_CELL (ExtImportPrefs::OnRuleTableSelect)
EVT_GRID_RANGE_SELECT (ExtImportPrefs::OnRuleTableSelectRange) EVT_GRID_RANGE_SELECT (ExtImportPrefs::OnRuleTableSelectRange)
@ -396,7 +396,10 @@ void ExtImportPrefs::OnAddRule(wxCommandEvent& event)
ExtImportItem *item = wxGetApp().mImporter->CreateDefaultImportItem(); ExtImportItem *item = wxGetApp().mImporter->CreateDefaultImportItem();
items->Add (item); items->Add (item);
AddItemToTable (RuleTable->GetNumberRows (), item); AddItemToTable (RuleTable->GetNumberRows (), item);
RuleTable->SelectRow(RuleTable->GetNumberRows () - 1);
RuleTable->SetGridCursor (RuleTable->GetNumberRows () - 1, 0);
RuleTable->SetFocus();
} }
void ExtImportPrefs::OnDelRule(wxCommandEvent& event) void ExtImportPrefs::OnDelRule(wxCommandEvent& event)
@ -404,9 +407,22 @@ void ExtImportPrefs::OnDelRule(wxCommandEvent& event)
if (last_selected < 0) if (last_selected < 0)
return; return;
ExtImportItems *items = wxGetApp().mImporter->GetImportItems(); ExtImportItems *items = wxGetApp().mImporter->GetImportItems();
int msgres = wxMessageBox (_("Do you really want to delete selected rule?"),
_("Rule deletion confirmation"), wxYES_NO, RuleTable);
if (msgres == wxNO || msgres != wxYES)
return;
RuleTable->DeleteRows (last_selected); RuleTable->DeleteRows (last_selected);
items->RemoveAt (last_selected); items->RemoveAt (last_selected);
RuleTable->AutoSizeColumns (); RuleTable->AutoSizeColumns ();
if (last_selected >= RuleTable->GetNumberRows ())
last_selected = RuleTable->GetNumberRows () - 1;
if (last_selected >= 0)
{
RuleTable->SelectRow(last_selected);
RuleTable->SetGridCursor (last_selected, 0);
}
} }
// Indentation settings for Vim and Emacs and unique identifier for Arch, a // Indentation settings for Vim and Emacs and unique identifier for Arch, a