sagalbot/vue-select

Setting initial value via value prop locks the selection

Open

#1,270 opened on Aug 21, 2020

View on GitHub
 (2 comments) (1 reaction) (0 assignees)JavaScript (1,307 forks)batch import
bughelp wantedhigh priority

Repository metrics

Stars
 (4,548 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Describe the bug in 3.. you can no longer set the selection via the value prop without vue-select locking its selection to that initial value. That is fine if you put a vue/vuex data object and update that but if you set the value prop as a string (ex. your data comes from PHP)

To Reproduce https://codepen.io/roennow/pen/yLOVbwE Try to select something different than foo

Expected behavior https://codepen.io/roennow/pen/bGpBWJr When you select something different than foo it actually does it

Desktop:

  • OS: macOS Catalina (10.15.6)
  • Browser Firefox
  • Version 78.0.2

Contributor guide

Research direction
Investigate the behavior of the value prop in vue select. Check the source code for how the initial value is used to set the selection. The issue suggests that setting value as a string causes lock. Look at the component's prop handling and event emission. Compare with working version (3.x vs earlier). Consider fixing by ensuring the component allows external changes to value prop to update selection.
Tech stack
vuejavascript
Domain
frontend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Basic Vue.jsUnderstanding of component props
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70