Skip to content

Conversation

AlanFnz
Copy link
Member

@AlanFnz AlanFnz commented Aug 6, 2025

What

Properly controls top padding in TimeField by passing hideLabel={!label} to InnerWrapper. Padding is now only applied when a label actually exists

Media

timefield

@AlanFnz AlanFnz requested review from lloydaf and renejfc August 6, 2025 13:57
@AlanFnz AlanFnz self-assigned this Aug 6, 2025
Copy link

pkg-pr-new bot commented Aug 6, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@freenow/wave@554

commit: 541cfb7

Copy link
Member

@renejfc renejfc left a comment

Choose a reason for hiding this comment

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

lgtm

@AlanFnz AlanFnz merged commit ad2843f into freenowtech:main Aug 6, 2025
7 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 6, 2025
### [2.41.3](v2.41.2...v2.41.3) (2025-08-06)

### Bug Fixes

* pass hideLabel prop to TimeField InnerWrapper ([#554](#554)) ([ad2843f](ad2843f))
Copy link
Contributor

github-actions bot commented Aug 6, 2025

🎉 This PR is included in version 2.41.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants