Skip to content

Commit

Permalink
Remove out-of-palette use of blue color, improve constraints label
Browse files Browse the repository at this point in the history
  • Loading branch information
nirvn committed Nov 14, 2024
1 parent a6066da commit 35c166e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -77,16 +77,14 @@ EditorWidgetBase {
Text {
visible: isEnabled
color: Theme.secondaryTextColor
text: isEnabled && !constraintsHardValid ? qsTr('Ensure contraints') : ''
text: isEnabled && !constraintsHardValid ? qsTr('Ensure contraints are met') : ''
anchors {
leftMargin: 10
left: parent.left
right: addButtonRow.left
verticalCenter: parent.verticalCenter
}
font.bold: true
font.italic: true
font.pointSize: Theme.tipFont.pointSize
font: Theme.tipFont
}

Row {
Expand All @@ -106,7 +104,7 @@ EditorWidgetBase {

round: false
iconSource: Theme.getThemeVectorIcon('ic_add_white_24dp')
bgcolor: parent.enabled ? nmRelationId ? 'blue' : 'black' : 'grey'
bgcolor: parent.enabled ? 'black' : 'grey'
}
}

Expand Down
8 changes: 3 additions & 5 deletions src/qml/editorwidgets/relationeditors/relation_editor.qml
Original file line number Diff line number Diff line change
Expand Up @@ -87,16 +87,14 @@ EditorWidgetBase {
Text {
visible: isEnabled
color: Theme.secondaryTextColor
text: isEnabled && !constraintsHardValid ? qsTr('Ensure contraints') : ''
text: isEnabled && !constraintsHardValid ? qsTr('Ensure contraints are met') : ''
anchors {
leftMargin: 10
left: parent.left
right: addButtonRow.left
verticalCenter: parent.verticalCenter
}
font.bold: true
font.italic: true
font.pointSize: Theme.tipFont.pointSize
font: Theme.tipFont
}

Row {
Expand All @@ -116,7 +114,7 @@ EditorWidgetBase {

round: false
iconSource: Theme.getThemeVectorIcon('ic_add_white_24dp')
bgcolor: parent.enabled ? nmRelationId ? 'blue' : 'black' : 'grey'
bgcolor: parent.enabled ? 'black' : 'grey'
}
}

Expand Down

1 comment on commit 35c166e

@qfield-fairy
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.