Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jessica's Scrabble Sinatra! #8

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Jessica's Scrabble Sinatra! #8

wants to merge 19 commits into from

Conversation

jweeber
Copy link

@jweeber jweeber commented Apr 10, 2016

Solo project
Completed all primary requirements and the first and third optional enhancements

Gems:
sinatra
rerun (which is the worst)
tilt
pry (for debugging)

Cool stuff:

  • Picture of Frank Sinatra in the footer. 🎤 He stares at you while you enter in words to score and it's amazing. I spent hours trying to get him in there and in the right spot and it was totally worth it.
  • For the third optional ( /score/:word ), I figured out how to check the word entry in my handler to see if it matches a regex pattern and word length before scoring the word, and if it doesn't it prints out an error to the user in the browser. I learned a lot spending time on this part. I'm really proud of it!

jweeber added 19 commits April 6, 2016 15:00
…he score. also removed the argument error and instead put a maxlength attribute in score.erb to limit chars to 7. cooooool
…field so user has to enter at least one letter and only up to 7 letters. the field is also required so they have to enter something for the score to show up
…that number of text boxes will work. so cool
…ed word. also got it to print out that there is a 50 point bonus when a 7 letter word is given
… clear the float of the form, and change the sizing of the elements so it all lines up on the right side.
…works. also home page fixed after testing styling things that didn't work.
… if the word entered in the url matches pattern
…the letters and points for each, but now fixed
…e/word and fixed it cause i'm the bomb dot com
…natra now watches you score your words and it's amazing
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.

1 participant