configure_ui: Address some review comments from the previous PR
This commit is contained in:
parent
347b50ad43
commit
0072003a14
|
@ -29,23 +29,8 @@ constexpr std::array row_text_names{
|
||||||
|
|
||||||
ConfigureUi::ConfigureUi(QWidget* parent) : QWidget(parent), ui(new Ui::ConfigureUi) {
|
ConfigureUi::ConfigureUi(QWidget* parent) : QWidget(parent), ui(new Ui::ConfigureUi) {
|
||||||
ui->setupUi(this);
|
ui->setupUi(this);
|
||||||
ui->language_combobox->addItem(tr("<System>"), QString{});
|
|
||||||
ui->language_combobox->addItem(tr("English"), QStringLiteral("en"));
|
|
||||||
QDirIterator it(QStringLiteral(":/languages"), QDirIterator::NoIteratorFlags);
|
|
||||||
while (it.hasNext()) {
|
|
||||||
QString locale = it.next();
|
|
||||||
locale.truncate(locale.lastIndexOf(QLatin1Char{'.'}));
|
|
||||||
locale.remove(0, locale.lastIndexOf(QLatin1Char{'/'}) + 1);
|
|
||||||
QString lang = QLocale::languageToString(QLocale(locale).language());
|
|
||||||
ui->language_combobox->addItem(lang, locale);
|
|
||||||
}
|
|
||||||
|
|
||||||
// Unlike other configuration changes, interface language changes need to be reflected on the
|
InitializeLanguageComboBox();
|
||||||
// interface immediately. This is done by passing a signal to the main window, and then
|
|
||||||
// retranslating when passing back.
|
|
||||||
connect(ui->language_combobox,
|
|
||||||
static_cast<void (QComboBox::*)(int)>(&QComboBox::currentIndexChanged), this,
|
|
||||||
&ConfigureUi::OnLanguageChanged);
|
|
||||||
|
|
||||||
for (const auto& theme : UISettings::themes) {
|
for (const auto& theme : UISettings::themes) {
|
||||||
ui->theme_combobox->addItem(QString::fromUtf8(theme.first),
|
ui->theme_combobox->addItem(QString::fromUtf8(theme.first),
|
||||||
|
@ -120,6 +105,25 @@ void ConfigureUi::RetranslateUI() {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void ConfigureUi::InitializeLanguageComboBox() {
|
||||||
|
ui->language_combobox->addItem(tr("<System>"), QString{});
|
||||||
|
ui->language_combobox->addItem(tr("English"), QStringLiteral("en"));
|
||||||
|
QDirIterator it(QStringLiteral(":/languages"), QDirIterator::NoIteratorFlags);
|
||||||
|
while (it.hasNext()) {
|
||||||
|
QString locale = it.next();
|
||||||
|
locale.truncate(locale.lastIndexOf(QLatin1Char{'.'}));
|
||||||
|
locale.remove(0, locale.lastIndexOf(QLatin1Char{'/'}) + 1);
|
||||||
|
const QString lang = QLocale::languageToString(QLocale(locale).language());
|
||||||
|
ui->language_combobox->addItem(lang, locale);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Unlike other configuration changes, interface language changes need to be reflected on the
|
||||||
|
// interface immediately. This is done by passing a signal to the main window, and then
|
||||||
|
// retranslating when passing back.
|
||||||
|
connect(ui->language_combobox, QOverload<int>::of(&QComboBox::currentIndexChanged), this,
|
||||||
|
&ConfigureUi::OnLanguageChanged);
|
||||||
|
}
|
||||||
|
|
||||||
void ConfigureUi::InitializeIconSizeComboBox() {
|
void ConfigureUi::InitializeIconSizeComboBox() {
|
||||||
for (const auto& size : default_icon_sizes) {
|
for (const auto& size : default_icon_sizes) {
|
||||||
ui->icon_size_combobox->addItem(QString::fromUtf8(size.second), size.first);
|
ui->icon_size_combobox->addItem(QString::fromUtf8(size.second), size.first);
|
||||||
|
|
|
@ -34,6 +34,7 @@ private:
|
||||||
void changeEvent(QEvent*) override;
|
void changeEvent(QEvent*) override;
|
||||||
void RetranslateUI();
|
void RetranslateUI();
|
||||||
|
|
||||||
|
void InitializeLanguageComboBox();
|
||||||
void InitializeIconSizeComboBox();
|
void InitializeIconSizeComboBox();
|
||||||
void InitializeRowComboBoxes();
|
void InitializeRowComboBoxes();
|
||||||
|
|
||||||
|
|
Reference in New Issue