Commit 4585f215 authored by Fushan Wen's avatar Fushan Wen 💬
Browse files

Move "Default" option to the front in "Select Remote Charset" menu

The ordering of encodings is different from other KDE apps in
"Select Remote Charset" menu. We should keep the UI consistency.

BUG: 445148
parent cdb4a790
Pipeline #101598 passed with stage
in 4 minutes and 58 seconds
......@@ -90,7 +90,7 @@ void DolphinRemoteEncoding::fillMenu()
QMenu* menu = m_menu->menu();
menu->clear();
menu->addAction(i18n("Default"), this, SLOT(slotDefault()), 0)->setCheckable(true);
for (int i = 0; i < m_encodingDescriptions.size();i++) {
QAction* action = new QAction(m_encodingDescriptions.at(i), this);
action->setCheckable(true);
......@@ -100,7 +100,6 @@ void DolphinRemoteEncoding::fillMenu()
menu->addSeparator();
menu->addAction(i18n("Reload"), this, SLOT(slotReload()), 0);
menu->addAction(i18n("Default"), this, SLOT(slotDefault()), 0)->setCheckable(true);
m_idDefault = m_encodingDescriptions.size() + 2;
connect(menu, &QMenu::triggered, this, &DolphinRemoteEncoding::slotItemSelected);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment