Tradeshift / tradeshift-ui

The Tradeshift UI Library & Framework
https://ui.tradeshift.com
Other
33 stars 45 forks source link

Can't create Blanket Purchase Order in IE 11 #276

Closed thorup911 closed 6 years ago

thorup911 commented 7 years ago

Choose the section that applies to you and use the template to help us figure out how we can help you.


Bug report

Tradeshift UI version affected

v7.0.0

Expected Behavior

It should be possible to add a Blanket Purchase Order (BPO) to a Purchase Request (PR)

Actual Behavior

It is not possible to add a BPO to a PR. The button "Add to purchase request" is not active. The issue seems to be related to the date fields.

Steps to reproduce

Environment

https://sandbox.tradeshift.com

User

tho+a+r_ch1@tradeshift.com

Screenshots (optional)

image

wiredearp commented 7 years ago

The UI component appears to correctly synchronize the value from the visible field (which could in theory be regionalized) to the hidden field (which is what Angular is using), so this appears to be a problem with Angular's validation :sob: It could potentially be related to https://stackoverflow.com/questions/25183217/valid-input-not-recognized-by-ie-11, in which case we might have a problem in other apps. Perhaps we see it here because in other apps the validation takes place on some kind of "submit-button" whereas in this app it is tied to input field focus?

cc @RafalRasztemborski

sampi commented 6 years ago

Can't be reproduced. Closing.