Skip to content

Commit

Permalink
HMA-5257 Reduced minHeight for InformationMessageCardView buttons fro…
Browse files Browse the repository at this point in the history
…m 60dp to 0dp so they do not look so big in the context of the InformationMessageCardView. (#72)
  • Loading branch information
jamiescode authored Feb 3, 2022
1 parent cba9801 commit 2e659ff
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 0 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ Allowed headings:

## [Unreleased]

### Changed

* Updated `InformationMessageCardView` buttons so that the button `minHeight` is reset from the default 60dp to 0dp. This means that buttons now only occupy the vertical space that they need and no more.

## [3.21.0] - 2022-01-25

### Added
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,7 @@ class InformationMessageCardView @JvmOverloads constructor(

binding.headlineButtonsContainer.visibility = if (buttons.isNotEmpty()) View.VISIBLE else View.GONE
buttons.forEach {
it.minHeight = 0
it.setMargins(spacing16, spacing8, spacing16, 0)
it.backgroundTintList = ContextCompat.getColorStateList(context, R.color.hmrc_white)
it.gravity = Gravity.CENTER
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 2e659ff

Please sign in to comment.