Skip to content

Theming fixes#1236

Merged
pomianowski merged 2 commits intolepoco:mainfrom
reincubate:theming-fixes
Feb 1, 2025
Merged

Theming fixes#1236
pomianowski merged 2 commits intolepoco:mainfrom
reincubate:theming-fixes

Conversation

@Difegue
Copy link
Copy Markdown
Contributor

@Difegue Difegue commented Sep 24, 2024

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

The placeholder text brush for TextControls is incorrect.
We've also seen that when using a single-line InfoBar, the text isn't centered vertically within the control. (This is also the case for multi-line InfoBars, but it's less jarring..)

image

Issue Number: N/A

What is the new behavior?

  • Fixed brush for placeholder text
  • Fixed margins for InfoBar

Other information

@github-actions github-actions bot added controls Changes to the appearance or logic of custom controls. styles Topic is related to styles PR Pull request release labels Sep 24, 2024
@pomianowski pomianowski merged commit 2afbe94 into lepoco:main Feb 1, 2025
pomianowski pushed a commit that referenced this pull request Feb 1, 2025
* Fix TextControl placeholders not using the proper color

* Fix InfoBar padding so it's centered for single-line messages
@Difegue Difegue deleted the theming-fixes branch February 10, 2025 14:26
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

controls Changes to the appearance or logic of custom controls. PR Pull request release styles Topic is related to styles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants