Commit ab8a25ad authored by Henrik Fehlauer's avatar Henrik Fehlauer

Polish wording in crop ratio combobox

Summary:
576cfa68 enabled the clear button in the {nav Crop} ratio combobox.
This resulted in an even wider toolbar at the bottom, because the
combobox automatically adjusts its width to its widest entry,
potentially cutting off the toolbar at the right even more than before.

As all other entries also lack the "size" moniker, we can just drop it
for {nav ISO size (A4, A3, …)}. Even the original entry before the
change in a028ea5c lacked "size" in the naming.

Test Plan
Open {nav Crop} tool in {nav Advanced} mode and inspect the ratio
combobox (now with consistent wording) as well as the width of the crop
toolbar (now less wide).

Reviewers: #gwenview, ngraham, huoni

Reviewed By: ngraham

Differential Revision: https://phabricator.kde.org/D11579
parent ef8bfd40
......@@ -190,14 +190,14 @@ struct CropWidgetPrivate : public Ui_CropWidget
Q_FOREACH(const QSizeF& size, ratioList) {
addRatioToComboBox(size);
}
addRatioToComboBox(QSizeF(sqrt2, 1), i18n("ISO Size (A4, A3...)"));
addRatioToComboBox(QSizeF(sqrt2, 1), i18n("ISO (A4, A3...)"));
addRatioToComboBox(QSizeF(11, 8.5), i18n("US Letter"));
addSectionHeaderToComboBox(i18n("Portrait"));
Q_FOREACH(QSizeF size, ratioList) {
size.transpose();
addRatioToComboBox(size);
}
addRatioToComboBox(QSizeF(1, sqrt2), i18n("ISO Size (A4, A3...)"));
addRatioToComboBox(QSizeF(1, sqrt2), i18n("ISO (A4, A3...)"));
addRatioToComboBox(QSizeF(8.5, 11), i18n("US Letter"));
ratioComboBox->setMaxVisibleItems(ratioComboBox->count());
......
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