BUSY.ORG | Updated Description Below 'topics' Input Box to Make Accurate PR #1321

Words
226
Reading
2 min
Listen
Play
9y

I attempted to update the instructions ("Separate topics with spaces. Only lowercase letters, numbers and hyphen characters are permitted.") below the topics input box on the '/editor' route and found that the file change that I initially thought would fix the issue did not have the desired effect.

I then searched the repo for the phrase only to find that there were 26 other instances of the phrase... I then replaced all instances with "Separate topics with spaces. Only lowercase letters, numbers and hyphen characters are permitted." I then saved the files and the text rendered on the screen as expected.

Currently, below 'topics' in the post editor, there is some instruction that reads:

"Separate topics with commas. Only lowercase letters, numbers and hyphen character is permitted."

However, commas are not necessary:
commasnotnecessary.gif


I went through the repository and changed every instance of the above phrase to read:

"Separate topics with spaces. Only lowercase letters, numbers and hyphen characters are permitted."

Here is a screenshot of the updated application running from my local machine:
Screen Shot 2018-01-07 at 11.56.07 AM.png

Here is a link to the pull request:
https://github.com/busyorg/busy/pull/1321

My Computer:

  • OS: MacOS Sierra 10.12.4 (16E195)
  • Model: Mac mini Server (Late 2012)
  • Processor: 2.6 GHz Intel Core i7
  • Memory: 4 GB 1600 MHz DDR3

My Browser:

  • Google Chrome Version 63.0.3239.108 (Official Build) (64-bit)



Posted on Utopian.io - Rewarding Open Source Contributors

BUSY.ORG | Updated Description Below 'topics' Input Box to Make Acc... | Ecency