Fix(directive):remove the dom operation in v-permission which will ca…

[j-joker]

What kind of change does this PR introduce?

bugifx

More information:
v-permission remove the the elemnt of dom which will arise an DomException when el-table updata its source and the vue try to patch the change ,so I suggest to change the display property instead of doing dom manipulation

[Armour]

Is there any possibility that the user can simply remove the CSS "display:none" style to show the content that their permission should not be able to see? I think this case is not acceptable.

[j-joker]

Is there any possibility that the user can simply remove the CSS "display:none" style to show the content that their permission should not be able to see? I think this case is not acceptable.

I think the essential of fontend privilege check is just for visibility.Of course you can remove the display property ,but that means nothing because we always have the privilege check in the backend.

[j-joker]

And I think The role of v-permission is not to prevent users from seeing secret content which is never suggested to do in front-end, but to prevent users from seeing elements that are not related to them.
The responsibility of developer is not to show features that users have no privilege,but when users try to edit the css in the client,they give up the benefit we provide by default

[Armour]

Thanks for detail explanation :)