Allow comma in array setting value
Closed this issue · 0 comments
UziTech commented
Prerequisites
- Put an X between the brackets on this line if you have done all of the following:
- Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
- Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
- Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
- Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
- Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages
Description
An array setting cannot have a comma value
Steps to Reproduce
- Set an array config value to
["1", ",", "2"]
inconfig.cson
- Settings view will show '1, ,, 2'
- Add a array item in the settings view '1, ,, 2, 3'
Expected behavior:
Settings view should show '1, ,, 2'
After changing the setting config.cson
should be ["1", ",", "2", "3"]
Actual behavior:
Settings view shows '1, ,, 2'
After changing the setting config.cson
is ["1", "", "2", "3"]
Reproduces how often:
100%
Versions
Atom : 1.35.0-beta0
Electron: 2.0.16
Chrome : 61.0.3163.100
Node : 8.9.3
apm 2.1.3
npm 6.2.0
node 8.9.3 x64
atom 1.35.0-beta0
python 2.7.14
git 2.20.0.windows.1
visual studio 2015
Additional Information
#1103 will fix this