Date filter in advanced mode doesn't use "ShowTimeForDateTimeFilter()" #838

Closed
opened 2026-01-29 17:45:07 +00:00 by claunia · 0 comments
Owner

Originally created by @PWyrostek on GitHub (May 2, 2023).

In grid advanced filter mode, the date picker for date columns doesn't use "ShowTimeForDateTimeFilter()" for the "ShowTime" property - instead it's true, so the time is always displayed on the filter in this mode (note: it is taken into consideration only for the second date filter in popup)

To Reproduce
Steps to reproduce the behavior:

  1. Create a custom column, overriding "ShowTimeForDateTimeFilter()" to false
  2. Create a DataGrid with Advanced filter mode and a DateTime column
  3. Activate the filter, the first filter will still have time picker.

Expected behavior
We should use ShowTimeForDateTimeFilter() in ShowTime in this line

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Chrome
  • Version 112.0.5615.138
Originally created by @PWyrostek on GitHub (May 2, 2023). In grid advanced filter mode, the date picker for date columns doesn't use "ShowTimeForDateTimeFilter()" for the "ShowTime" property - instead it's true, so the time is always displayed on the filter in this mode (note: it is taken into consideration **only** for the second date filter in popup) **To Reproduce** Steps to reproduce the behavior: 1. Create a custom column, overriding "ShowTimeForDateTimeFilter()" to false 2. Create a DataGrid with Advanced filter mode and a DateTime column 3. Activate the filter, the first filter will still have time picker. **Expected behavior** [We should use ShowTimeForDateTimeFilter() in ShowTime in this line](https://github.com/radzenhq/radzen-blazor/blob/a7df5c7facf5030d17c1269d60d1b9cb6fd0e98c/Radzen.Blazor/RadzenDataGridHeaderCell.razor#L84) **Desktop (please complete the following information):** - OS: Windows 10 - Browser: Chrome - Version 112.0.5615.138
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/radzen-blazor#838