search

Hyperskill-Community / flashcards

Digital flashcard app
4 stars 0 forks source link

minor templates refactor #75

Closed codeofcarbon closed 7 months ago

codeofcarbon commented 7 months ago

refactor mainly touches expanded tags, which I removed where those weren't needed, pl[us some other minor stuff like gathering colors in one place. No features improvements here 😺

wisskirchenj commented 7 months ago

Yepp - better! Thanks for adding all those semicolons after the const lambdas - funny, that I seem to keep forgetting those :-/ Actually, this finding showed me, that we don't use the linter yet... otherwise it would have told ...

wisskirchenj commented 7 months ago

Hmm - did you see in discord-server: Linting throws errors for v-text / v-html use in components: The rule is: https://eslint.vuejs.org/rules/no-v-text-v-html-on-component And here is some (convincing IMO) discussion, why it makes sense: https://github.com/vuejs/eslint-plugin-vue/issues/1724

So I guess we should not use v-text and v-html... But I would not mind merging this anyway.. What do you think?