Skip to content

Add workaround for datetime-local input in firefox+safari - #39261

Merged
bbovenzi merged 1 commit into
apache:mainfrom
astronomer:fix-firefox-datetime-local
Apr 25, 2024
Merged

bbovenzi merged 1 commit into
apache:mainfrom
astronomer:fix-firefox-datetime-local

Conversation

@bbovenzi

@bbovenzi bbovenzi commented Apr 25, 2024 •

Copy link
Copy Markdown
Contributor

We use datetime-local to set base dates, but safari and firefox don't fully support it. Instead, we detect the browser and render separate "date" and "time" inputs side-by-side.

Screenshot 2024-04-25 at 11 03 16 AM Screenshot 2024-04-25 at 11 03 23 AM Screenshot 2024-04-25 at 11 03 19 AM

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@bbovenzi bbovenzi added this to the Airflow 2.9.1 milestone Apr 25, 2024
@bbovenzi
bbovenzi requested a review from jscheffl April 25, 2024 15:16
@boring-cyborg boring-cyborg Bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Apr 25, 2024
@bbovenzi
bbovenzi merged commit 5638bc3 into apache:main Apr 25, 2024
@bbovenzi
bbovenzi deleted the fix-firefox-datetime-local branch April 25, 2024 17:21
@jedcunningham jedcunningham added the type:bug-fix Changelog: Bug Fixes label Apr 26, 2024
jedcunningham pushed a commit that referenced this pull request Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants