Skip to content

Conversation

@emmanuelbaa
Copy link
Member

fixes #9109

added a time picker to the add event dialogue.

timePicker

@emmanuelbaa emmanuelbaa changed the title added supporting time picker to add event (fixes #9109) add Event: added a timepicker (fixes #9109) Sep 23, 2025
Copy link
Member

@jessewashburn jessewashburn left a comment

Choose a reason for hiding this comment

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

I like the idea, but currently I'm seeing some weird margins.

On Firefox:
image

On Chrome the issue is more subtle, but you can see the top of the 0 and the 2 are slightly cut off.
image

@emmanuelbaa
Copy link
Member Author

I like the idea, but currently I'm seeing some weird margins.

On Firefox: image

On Chrome the issue is more subtle, but you can see the top of the 0 and the 2 are slightly cut off. image

good catch.

@dogi dogi changed the title add Event: added a timepicker (fixes #9109) dashboard: smoother events creation (fixes #9109) Sep 24, 2025
@Saby-Bishops
Copy link
Member

Hi @emmanuelbaa,
I think double-checking the CSS for padding/line-height on the input field might be a good idea? It might just need a small adjustment to ensure consistent rendering across browsers.
Otherwise, the functionality looks solid.

google-labs-jules bot and others added 5 commits October 17, 2025 23:38
This change fixes a visual bug in the time picker input fields where the text was being cut off due to incorrect margins and padding. The fix adds a new CSS class to the time picker fields and applies a padding to the input element to ensure the text is displayed correctly in all browsers.
This change fixes a visual bug in the time picker input fields where the text was being cut off due to incorrect margins and padding. The fix adds a new CSS class to the time picker fields and applies a height to the input element to ensure the text is displayed correctly in all browsers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

community events time field

6 participants