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

[Bug Report] Bottom border persist validation error color in IE #10314

Closed
nclemeur opened this issue Jan 23, 2020 · 11 comments
Closed

[Bug Report] Bottom border persist validation error color in IE #10314

nclemeur opened this issue Jan 23, 2020 · 11 comments
Assignees
Labels
C: VTextField P: low The issue is of low importance platform specific The issue only occurs on a specific platform T: bug Functionality that does not work as intended/expected

Comments

@nclemeur
Copy link

Environment

Vuetify Version: 2.2.6
Last working version: 1.5.20
Vue Version: 2.6.11
Browsers: Internet Explorer
OS: Windows 10

Steps to reproduce

Note: this is an Internet Explorer only problem! Codepen is provided for the code, but the issue can't be reproduced in codepen
Use switch to turn error to true, then turn error back to false using the switch.

Expected Behavior

Bottom border should go back to it's normal state

Actual Behavior

The bottom border will remain red

Reproduction Link

https://codepen.io/nclemeur/pen/dyPaEyO

@johnleider
Copy link
Member

I ran into this the other day. It's a rendering issue (you can check and uncheck any display property in the dev tools and it fixes.

@johnleider johnleider added P: low The issue is of low importance platform specific The issue only occurs on a specific platform T: bug Functionality that does not work as intended/expected and removed S: triage labels Jan 27, 2020
@nclemeur
Copy link
Author

nclemeur commented Jan 28, 2020

It should be noted that this can be easily reproduced in the v-text-field documentation (https://vuetifyjs.com/en/components/text-fields) in the validation example. Just type in a correct email in the e-mail field then move the focus away, the bottom border is red again.

@Mikilll94
Copy link
Contributor

@nclemeur
In docs, everything works fine on IE. How is that possible? Even if you go to the forms section
https://vuetifyjs.com/en/components/forms#forms there are no problems with the red bottom border.

@johnleider
Do you know any workaround for this bug? I will be very glad for any help.

@kinkinkk
Copy link

kinkinkk commented Feb 3, 2020

@Mikilll94
Anyway, I

loading = true
sleep 50ms
loading = false

Check the improvement of the phenomenon.

The phenomenon was stopped by switching the loading.

@nclemeur
Copy link
Author

nclemeur commented Feb 4, 2020

@Mikilll94 yes that is really strange I can't reproduce it either. But I still see the issue on my side

@kinkinkk I am not sure I understand what do you mean? Could you elaborate a bit more?

@kinkinkk
Copy link

kinkinkk commented Feb 4, 2020

@nclemeur
It is a provisional correspondence method after reproducing.

You can fix the red border by turning loading on / off in the properties of the target v-textfiled.

@Mikilll94
Copy link
Contributor

Mikilll94 commented Feb 4, 2020

@kinkinkk
But how to detect that the field is now valid and we should refresh it? Does anyone have a fully working workaround?

@kinkinkk
Copy link

kinkinkk commented Feb 5, 2020

@Mikilll94
I'm currently working locally.
For example, a rule is set in the computed property, and only in the case of IE, loading switching is performed every time when inputting.

@nclemeur
Copy link
Author

nclemeur commented Feb 5, 2020

I believe this is now fixed with 2.2.9. Probably alon fixing #10388?

@Mikilll94
Copy link
Contributor

I confirm that this is fixed in 2.2.9 👍

@MajesticPotatoe
Copy link
Member

Will close given confirmation. If you believe that this issue has not been correctly resolved, please create a new issue showing the regression or reach out to us in our Discord community.

Thank you for your contribution and interest in improving Vuetify.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VTextField P: low The issue is of low importance platform specific The issue only occurs on a specific platform T: bug Functionality that does not work as intended/expected
Projects
None yet
Development

No branches or pull requests

7 participants