search

kazupon / vue-validator

:white_check_mark: Validator component for Vue.js
MIT License
2.19k stars 431 forks source link

Directives that change the DOM #242

Closed blocka closed 8 years ago

blocka commented 8 years ago

vue & vue-validator version

1.0.24, 2.1.1

Reproduction Link

If really necessary will, provide, but am limited in time.

Steps to reproduce

Add a validator on to an element that changes the elements DOM...either by wrapping it or adding to it.

What is Expected?

The validator will use the intended form element (this.el)

What is actually happening?

The validator is using the fragment instance, which is modified by directive. Since this is a terminal directive, the priority does not seem to make a difference....custom directive is always called first. This results in all sorts of bugs.

kazupon commented 8 years ago

Thank you for your feedback. Sorry, I could understand your explanation.

If you can reproduce this issue, Can you provide the reproduction code please?

blocka commented 8 years ago

I think it was fixed here #243