Skip to content

Commit de3758c

Browse files
committed
Set PdmUiButton default alignment to right
Removed manual setAlignment(Qt::AlignRight) calls for UI buttons, as PdmUiButton now defaults to right alignment. Also removed unused cafPdmUiButton.h includes from several files.
1 parent caaf799 commit de3758c

File tree

7 files changed

+12
-25
lines changed

7 files changed

+12
-25
lines changed

ApplicationLibCode/Commands/ExportCommands/RicExportSectorModelUi.cpp

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@
3131
#include "RimTools.h"
3232
#include "Tools/RimEclipseViewTools.h"
3333

34-
#include "cafPdmUiButton.h"
3534
#include "cafPdmUiCheckBoxEditor.h"
3635
#include "cafPdmUiFilePathEditor.h"
3736
#include "cafPdmUiGroup.h"
@@ -229,13 +228,12 @@ void RicExportSectorModelUi::defineUiOrdering( QString uiConfigName, caf::PdmUiO
229228
else if ( uiConfigName == m_pageNames[WizardPageEnum::KeywordAdjustments] )
230229
{
231230
uiOrdering.add( &m_keywordsToRemove );
232-
auto button = uiOrdering.addNewButton( "Reset to Default",
233-
[this]()
234-
{
235-
setDefaultKeywordsToRemove();
236-
updateConnectedEditors();
237-
} );
238-
button->setAlignment( Qt::AlignRight );
231+
uiOrdering.addNewButton( "Reset to Default",
232+
[this]()
233+
{
234+
setDefaultKeywordsToRemove();
235+
updateConnectedEditors();
236+
} );
239237
}
240238
else if ( uiConfigName == m_pageNames[WizardPageEnum::SimulationJob] )
241239
{

ApplicationLibCode/ProjectDataModel/AnalysisPlots/RimAnalysisPlot.cpp

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@
5353
#include "qwt_scale_draw.h"
5454

5555
#include "cafCmdFeatureMenuBuilder.h"
56-
#include "cafPdmUiButton.h"
5756
#include "cafPdmUiCheckBoxEditor.h"
5857
#include "cafPdmUiComboBoxEditor.h"
5958
#include "cafPdmUiGroup.h"
@@ -517,10 +516,7 @@ void RimAnalysisPlot::defineUiOrdering( QString uiConfigName, caf::PdmUiOrdering
517516

518517
caf::PdmUiGroup* selVectorsGrp = uiOrdering.addNewGroup( "Selected Vectors" );
519518
selVectorsGrp->addNewLabel( createLabelText(), { .newRow = false, .totalColumnSpan = 4 } );
520-
auto showVectorSelection = selVectorsGrp->addNewButton( "...",
521-
[this]() { RimAnalysisPlot::showSelectVariablesDialog(); },
522-
{ .newRow = false, .totalColumnSpan = 1 } );
523-
showVectorSelection->setAlignment( Qt::AlignRight );
519+
selVectorsGrp->addNewButton( "...", [this]() { RimAnalysisPlot::showSelectVariablesDialog(); }, { .newRow = false, .totalColumnSpan = 1 } );
524520

525521
selVectorsGrp->add( &m_referenceCase );
526522

ApplicationLibCode/ProjectDataModel/Completions/RimMswSegmentCollection.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@
3333
#include "RimWellPathCompletionSettings.h"
3434
#include "RimWellPathCompletions.h"
3535

36-
#include "cafPdmUiButton.h"
37-
3836
#include <algorithm>
3937
#include <map>
4038

ApplicationLibCode/ProjectDataModel/Jobs/RimOpmFlowJob.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -360,8 +360,7 @@ void RimOpmFlowJob::defineUiOrdering( QString uiConfigName, caf::PdmUiOrdering&
360360

361361
if ( createOpenPostionButton )
362362
{
363-
auto openSelectButton = wellGrp->addNewButton( "Select Open Keyword Position", [this]() { selectOpenWellPosition(); } );
364-
openSelectButton->setAlignment( Qt::AlignRight );
363+
wellGrp->addNewButton( "Select Open Keyword Position", [this]() { selectOpenWellPosition(); } );
365364
}
366365
}
367366
}
@@ -399,8 +398,7 @@ void RimOpmFlowJob::defineUiOrdering( QString uiConfigName, caf::PdmUiOrdering&
399398
auto advGrp = uiOrdering.addNewGroup( "Advanced" );
400399
advGrp->setCollapsedByDefault();
401400
advGrp->add( &m_currentRunId );
402-
auto resetRunIdButton = advGrp->addNewButton( "Reset Ensemble Run Id", [this]() { resetEnsembleRunId(); } );
403-
resetRunIdButton->setAlignment( Qt::AlignRight );
401+
advGrp->addNewButton( "Reset Ensemble Run Id", [this]() { resetEnsembleRunId(); } );
404402

405403
uiOrdering.skipRemainingFields();
406404
}

ApplicationLibCode/ProjectDataModel/WellPath/RimWellPathCollection.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@
7070
#include "cafDataLoader.h"
7171
#include "cafPdmFieldScriptingCapability.h"
7272
#include "cafPdmObjectScriptingCapability.h"
73-
#include "cafPdmUiButton.h"
7473
#include "cafPdmUiEditorHandle.h"
7574
#include "cafPdmUiTreeOrdering.h"
7675
#include "cafProgressInfo.h"
@@ -557,8 +556,7 @@ void RimWellPathCollection::defineUiOrdering( QString uiConfigName, caf::PdmUiOr
557556
caf::PdmUiGroup* mswSegmentGroup = uiOrdering.addNewGroup( "MSW Segment Visualization" );
558557
mswSegmentGroup->add( &m_mswShowBands );
559558
mswSegmentGroup->add( &m_mswEclipseCase );
560-
auto updateButton = mswSegmentGroup->addNewButton( "Update Segments", [this]() { updateMswSegments(); } );
561-
updateButton->setAlignment( Qt::AlignRight );
559+
mswSegmentGroup->addNewButton( "Update Segments", [this]() { updateMswSegments(); } );
562560
uiOrdering.skipRemainingFields();
563561
}
564562

Fwk/AppFwk/cafProjectDataModel/cafPdmUiCore/cafPdmUiButton.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,15 +43,15 @@ namespace caf
4343
///
4444
//--------------------------------------------------------------------------------------------------
4545
PdmUiButton::PdmUiButton()
46-
: m_alignment( Qt::AlignLeft | Qt::AlignTop )
46+
: m_alignment( Qt::AlignRight )
4747
{
4848
}
4949

5050
//--------------------------------------------------------------------------------------------------
5151
///
5252
//--------------------------------------------------------------------------------------------------
5353
PdmUiButton::PdmUiButton( const QString& buttonText, const ClickCallback& callback )
54-
: m_alignment( Qt::AlignLeft | Qt::AlignTop )
54+
: m_alignment( Qt::AlignRight )
5555
, m_clickCallback( callback )
5656
{
5757
setUiName( buttonText );

Fwk/AppFwk/cafTests/cafTestApplication/LabelsAndHyperlinks.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,6 @@ void LabelsAndHyperlinks::defineUiOrdering( QString uiConfigName, caf::PdmUiOrde
8080
msgBox->show();
8181
} );
8282
iconButton->setIconFromResourceString( ":/cafCommandFeatures/Delete.svg" );
83-
iconButton->setAlignment( Qt::AlignRight );
8483
}
8584
else
8685
{

0 commit comments

Comments
 (0)