https://www.ultimatepp.org/redmine/https://www.ultimatepp.org/redmine/redmine/favicon.ico2013-11-28T00:05:57ZRedmineU++ - Bug #591: Ctrl+Q shortcut in SelectPackageDlg is slogan only.https://www.ultimatepp.org/redmine/issues/591?journal_id=12152013-11-28T00:05:57ZZbigniew Rebacz
<ul><li><strong>File</strong> deleted (<del><i>SelectPkg.cpp</i></del>)</li></ul> U++ - Bug #591: Ctrl+Q shortcut in SelectPackageDlg is slogan only.https://www.ultimatepp.org/redmine/issues/591?journal_id=12162013-11-28T00:11:39ZZbigniew Rebacz
<ul><li><strong>File</strong> <a href="/redmine/attachments/download/224/SelectPkg.cpp">SelectPkg.cpp</a><a href="/redmine/attachments/224/SelectPkg.cpp"><img alt="Magnifier" src="/redmine/images/magnifier.png" /></a> added</li></ul><p>Personally, I think we should also change shortcut combination (CTRL+Q to CTRL+K), because CTRL+K is more neutral for search. Moreover this shortcut is widely used for example in Mozilla Firefox or Thunderbird. While CTRL+Q is good shortcut for exiting the program.</p>
<pre>
// In Constructor:
search.NullText("Search (Ctrl+K)", StdFont().Italic(), SColorDisabled());
// Changed shortcut:
bool SelectPackageDlg::Key(dword key, int count)
{
if(key == K_ALT_ENTER) {
ChangeDescription();
return true;
}
else if(key == K_CTRL_K) {
search.SetFocus();
return true;
}
if((clist.HasFocus() || alist.HasFocus()) && search.Key(key, count))
return true;
return TopWindow::Key(key, count);
}
</pre> U++ - Bug #591: Ctrl+Q shortcut in SelectPackageDlg is slogan only.https://www.ultimatepp.org/redmine/issues/591?journal_id=12312013-12-03T08:14:30ZMiroslav Fidlercxl@ntllib.org
<ul><li><strong>Status</strong> changed from <i>Ready for QA</i> to <i>Patch ready</i></li></ul> U++ - Bug #591: Ctrl+Q shortcut in SelectPackageDlg is slogan only.https://www.ultimatepp.org/redmine/issues/591?journal_id=12452013-12-09T07:30:01ZMiroslav Fidlercxl@ntllib.org
<ul><li><strong>Status</strong> changed from <i>Patch ready</i> to <i>Approved</i></li></ul>