Skip to content

fix(controls): ToolTip taking parent line height and font weight into account#1525

Merged
pomianowski merged 1 commit intolepoco:mainfrom
Nuklon:patch-32
Nov 7, 2025
Merged

fix(controls): ToolTip taking parent line height and font weight into account#1525
pomianowski merged 1 commit intolepoco:mainfrom
Nuklon:patch-32

Conversation

@Nuklon
Copy link
Copy Markdown
Collaborator

@Nuklon Nuklon commented Aug 28, 2025

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?

ToolTip currently sets the internal FontSize to 12, that's OK, but the LineHeight and FontWeight should also be set to ensure that if you have a parent element (e.g., a TextBlock) with say FontWeight SemiBold and LineHeight 28, these don't get used by ToolTip, which looks like a huge amount of padding at the bottom due to the line height.

@Nuklon Nuklon requested a review from pomianowski as a code owner August 28, 2025 16:34
@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 Aug 28, 2025
@Nuklon Nuklon changed the title Fix ToolTip taking parent lineheight and size into account Fix ToolTip taking parent line height and font weight into account Aug 28, 2025
@pomianowski pomianowski changed the title Fix ToolTip taking parent line height and font weight into account fix(controls): ToolTip taking parent line height and font weight into account Nov 7, 2025
@pomianowski pomianowski merged commit b8ee7d4 into lepoco:main Nov 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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