Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved: Add a date range picker form field (OFBIZ-13209) #879

Open
wants to merge 6 commits into
base: trunk
Choose a base branch
from

Conversation

florianMo
Copy link
Contributor

@florianMo florianMo commented Feb 12, 2025

Improved: Add a date range picker form field

This is a wrapper around https://www.daterangepicker.com/, exposing (almost) all its parameters as field attributes.

It can be used as a date picker (like ), or as a date range picker (like ).

Its main purpose is to provide a modern UX to pick date ranges, with 2 calendars where the user can click start date, then end date, instead of dealing with two unrelated date fields and operator selectors.

Visit /webtools/control/WebtoolsLayoutDemo and /framework/webtools/widget/MiscForms.xml#LayoutDemoForm for a showcase.

(OFBIZ-13209)

Thanks: Néréide team

This is a wrapper around https://www.daterangepicker.com/, exposing
(almost) all its parameters as field attributes.

It can be used as a date picker (like <date-time/>), or as a date range
picker (like <date-find/>).

Its main purpose is to provide a modern UX to pick date ranges, with
2 calendars where the user can click start date, then end date, instead
of dealing with two unrelated date fields and operator selectors.

OFBIZ-13209
@florianMo
Copy link
Contributor Author

@JacquesLeRoux I see failing jobs, but it does not seem related to the actual code ?

@JacquesLeRoux
Copy link
Contributor

@JacquesLeRoux I see failing jobs, but it does not seem related to the actual code ?

Few days ago I sent a message about that but atm I can't find it. Anyway, indeed it's general and unreltated to your work here. :)

@JacquesLeRoux
Copy link
Contributor

BTW, I got a bit sick hence less attention last days, feel better today :)

@florianMo
Copy link
Contributor Author

Few days ago I sent a message about that but atm I can't find it. Anyway, indeed it's general and unreltated to your work here. :)

Thanks @JacquesLeRoux !

@JacquesLeRoux
Copy link
Contributor

Found it in my email client, weridly nothing at https://lists.apache.org/
Image 043

Maybe it's time to alert them... Though it's not a big deal. I'm currently working on the "old" codeDL alerts list (less than a month old) and it has not changed for months if not years.

@JacquesLeRoux
Copy link
Contributor

JacquesLeRoux commented Feb 14, 2025

Ha, I also sent a message to the security ML where I put that at the bottom (no secret, so no pb)

*    Note that currently CodeQL is failing: https://pipelinesghubeus2.actions.githubusercontent.com/fUrDAyzwWugzCCqnOZoRTpKuxUHxXGBhCKQFOpXdXNmBahQfog/_apis/pipelines/workflows/13244470954/artifacts?api-version=6.0-preview
      As we did not change anything nor were suggested to do so, seems due to an use of Azure: https://stackoverflow.com/questions/60021558/tf400813-the-user-is-not-authorized-to-access-this-resource
      And not https://github.com/actions/deploy-pages/issues/236

It remember me that : https://citations.ouest-france.fr/citation-henri-queuille/aucun-probleme-assez-urgent-politique-107513.html :D

You just have to replace politique by informatique

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

Successfully merging this pull request may close these issues.

2 participants