Skip to content

Three changes#5

Open
jennazee wants to merge 3 commits intortunney:masterfrom
jennazee:master
Open

Three changes#5
jennazee wants to merge 3 commits intortunney:masterfrom
jennazee:master

Conversation

@jennazee
Copy link

Hi Rob!

Three changes:

  1. There's a python function to return a frequency distribution of all the non-stop words used in the reflection. It's called generate_popular( ) in reflection_scraper.py
  2. I added functionality that when you click on a matched person's name in the wheel, it displays snippets of their reflections that included that word
  3. I fixed the name submission system so that pushing enter only calls the generate tree function once. Before, hitting 'enter' ran the function twice

@forklady42
Copy link

Brief expansion on 3 for Rob's benefit, as he wasn't here for the conversation:

html already submits a form when enter is pressed. Thus, it's not necessary to handle the submit via button and keypress separately in jQuery. Simply calling generate_tree upon "submit" works in both cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants