forked from qt-creator/qt-creator
CMake: Some UI/tr fixes
Avoid the order "Label [ ]" for checkboxes, we changed all cases to the standard "[ ] Label" because it looks very ugly and confusing the other way around. Also move the corresponding directory setting to the next row, which is not ideal but still looks better. Improve message about reloading presets. Change-Id: Ibcd26e24a2a4cd3ea5ca819f74e78edb40f46072 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Cristian Adam <cristian.adam@qt.io>
This commit is contained in:
@@ -194,7 +194,7 @@ CMakeBuildStep::CMakeBuildStep(BuildStepList *bsl, Id id) :
|
||||
|
||||
m_useStaging = addAspect<BoolAspect>();
|
||||
m_useStaging->setSettingsKey(USE_STAGING_KEY);
|
||||
m_useStaging->setLabelPlacement(BoolAspect::LabelPlacement::AtCheckBox);
|
||||
m_useStaging->setLabel(Tr::tr("Stage for installation"), BoolAspect::LabelPlacement::AtCheckBox);
|
||||
m_useStaging->setDefaultValue(!buildAndRunOnSameDevice(kit()));
|
||||
|
||||
m_stagingDir = addAspect<FilePathAspect>();
|
||||
@@ -503,8 +503,10 @@ QWidget *CMakeBuildStep::createConfigWidget()
|
||||
|
||||
m_stagingDir->setEnabled(m_useStaging->value());
|
||||
if (m_useStaging->value()) {
|
||||
summaryText.append(" " + Tr::tr("and stage at %2 for %3")
|
||||
.arg(currentStagingDir(), currentInstallPrefix()));
|
||||
//: Stage (for installation) at <staging_dir> for <installation_dir>
|
||||
summaryText.append(
|
||||
"; "
|
||||
+ Tr::tr("Stage at %2 for %3").arg(currentStagingDir(), currentInstallPrefix()));
|
||||
}
|
||||
|
||||
if (!m_buildPreset.isEmpty()) {
|
||||
@@ -567,7 +569,8 @@ QWidget *CMakeBuildStep::createConfigWidget()
|
||||
Layouting::Form builder;
|
||||
builder.addRow({m_cmakeArguments});
|
||||
builder.addRow({m_toolArguments});
|
||||
builder.addRow({Tr::tr("Stage for installation:"), Layouting::Row{m_useStaging, m_stagingDir}});
|
||||
builder.addRow({m_useStaging});
|
||||
builder.addRow({m_stagingDir});
|
||||
|
||||
if (m_useiOSAutomaticProvisioningUpdates)
|
||||
builder.addRow({m_useiOSAutomaticProvisioningUpdates});
|
||||
|
||||
Reference in New Issue
Block a user