Skip to content

Commit

Permalink
[core] fix: do not colour FormGroup sub label based on intent (#6811)
Browse files Browse the repository at this point in the history
Co-authored-by: svc-changelog <svc-changelog@palantir.com>
  • Loading branch information
berryde and svc-changelog authored May 20, 2024
1 parent 79010df commit 772e44c
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
5 changes: 5 additions & 0 deletions packages/core/changelog/@unreleased/pr-6811.v2.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
type: improvement
improvement:
description: Do not colour FormGroup sub label based on intent
links:
- https://github.com/palantir/blueprint/pull/6811
2 changes: 0 additions & 2 deletions packages/core/src/components/forms/_form-group.scss
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ Styleguide form-group
/* stylelint-disable-next-line order/declaration-block-order */
@each $intent, $color in $pt-intent-text-colors {
&.#{$ns}-intent-#{$intent} {
.#{$ns}-form-group-sub-label,
.#{$ns}-form-helper-text {
color: $color;
}
Expand Down Expand Up @@ -102,7 +101,6 @@ Styleguide form-group
.#{$ns}-dark & {
@each $intent, $color in $pt-dark-intent-text-colors {
&.#{$ns}-intent-#{$intent} {
.#{$ns}-form-group-sub-label,
.#{$ns}-form-helper-text {
color: $color;
}
Expand Down
3 changes: 1 addition & 2 deletions packages/core/src/components/forms/formGroup.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,7 @@ export interface FormGroupProps extends IntentProps, Props {

/**
* Optional text for `label`. The given content will be wrapped in
* `Classes.FORM_GROUP_SUB_LABEL` and displayed beneath `label`. The text color
* is determined by the `intent`.
* `Classes.FORM_GROUP_SUB_LABEL` and displayed beneath `label`.
*/
subLabel?: React.ReactNode;
}
Expand Down

1 comment on commit 772e44c

@svc-palantir-github
Copy link

Choose a reason for hiding this comment

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

[core] fix: do not colour FormGroup sub label based on intent (#6811)

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

Please sign in to comment.