vuejs / vue-hackernews-2.0

HackerNews clone built with Vue 2.0, vue-router & vuex, with server-side rendering
MIT License
10.96k stars 2.15k forks source link

remove redundant v-if #308

Closed mcculleydj closed 3 years ago

mcculleydj commented 6 years ago

Did not see a reason to have duplicate v-if="item" directives on lines 2 and 3. Checked the blame, which mentions adjusting spinner position, but the template tag should be unreachable if !item and transparent otherwise.