UI/Qt: Simplify setting some action shortcuts

These were copied from their previous instantiations, but we don't need
to go through QKeySequence::keyBindings for standard shortcuts.
This commit is contained in:
Timothy Flynn 2025-09-13 08:49:44 -04:00 committed by Tim Flynn
parent dbce62bc0d
commit 255c4c2d9b

View file

@ -78,11 +78,11 @@ static void initialize_native_control(WebView::Action& action, QAction& qaction,
switch (action.id()) {
case WebView::ActionID::NavigateBack:
qaction.setIcon(create_tvg_icon_with_theme_colors("back", palette));
qaction.setShortcuts(QKeySequence::keyBindings(QKeySequence::StandardKey::Back));
qaction.setShortcut(QKeySequence::StandardKey::Back);
break;
case WebView::ActionID::NavigateForward:
qaction.setIcon(create_tvg_icon_with_theme_colors("forward", palette));
qaction.setShortcuts(QKeySequence::keyBindings(QKeySequence::StandardKey::Forward));
qaction.setShortcut(QKeySequence::StandardKey::Forward);
break;
case WebView::ActionID::Reload:
qaction.setIcon(create_tvg_icon_with_theme_colors("reload", palette));
@ -91,15 +91,15 @@ static void initialize_native_control(WebView::Action& action, QAction& qaction,
case WebView::ActionID::CopySelection:
qaction.setIcon(load_icon_from_uri("resource://icons/16x16/edit-copy.png"sv));
qaction.setShortcuts(QKeySequence::keyBindings(QKeySequence::StandardKey::Copy));
qaction.setShortcut(QKeySequence::StandardKey::Copy);
break;
case WebView::ActionID::Paste:
qaction.setIcon(load_icon_from_uri("resource://icons/16x16/paste.png"sv));
qaction.setShortcuts(QKeySequence::keyBindings(QKeySequence::StandardKey::Paste));
qaction.setShortcut(QKeySequence::StandardKey::Paste);
break;
case WebView::ActionID::SelectAll:
qaction.setIcon(load_icon_from_uri("resource://icons/16x16/select-all.png"sv));
qaction.setShortcuts(QKeySequence::keyBindings(QKeySequence::StandardKey::SelectAll));
qaction.setShortcut(QKeySequence::StandardKey::SelectAll);
break;
case WebView::ActionID::SearchSelectedText:
@ -163,7 +163,7 @@ static void initialize_native_control(WebView::Action& action, QAction& qaction,
}
case WebView::ActionID::ZoomOut:
qaction.setIcon(load_icon_from_uri("resource://icons/16x16/zoom-out.png"sv));
qaction.setShortcuts(QKeySequence::keyBindings(QKeySequence::StandardKey::ZoomOut));
qaction.setShortcut(QKeySequence::StandardKey::ZoomOut);
break;
case WebView::ActionID::ResetZoom:
qaction.setIcon(load_icon_from_uri("resource://icons/16x16/zoom-reset.png"sv));