vuejs/vue

Warn if colon shorthand is used on v-if/v-html/etc.

Open

#10.191 geöffnet am 25. Juni 2019

Auf GitHub ansehen
 (15 Kommentare) (23 Reaktionen) (0 zugewiesene Personen)TypeScript (209.764 Stars) (33.833 Forks)batch import
contribution welcomefeature requestgood first issuehas PRwarnings

Beschreibung

What problem does this feature solve?

I just spent way too long debugging something really weird until I realized I accidentally wrote :v-if instead of v-if.

A warning when wrongly using shorthands like : on "native" vue attributes could prevent this bad experience easily.

What does the proposed API look like?

:v-if="foo"

--> console.warn("You specified v-bind:/ v-on: or a corresponding shorthand on a Vue attribute like v-if or similar. Usually this does not make sense.)

Contributor Guide

Tech Stack
vuetypescript
Domain
frontend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Familiarity with Vue 2 template compiler
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
Investigate Vue 2's template compiler in `src/compiler/` to find where directives are parsed. Look for the handling of `v bind` shorthand (`:`) and how native directives like `v if` are distinguished. Add a warning when a shorthand is used on a directive that does not accept bindings. The warning can be emitted in the parser or codegen phase. Check linked issues and comments for prior discussion on implementation details.