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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fries for crobs #5

Merged
merged 7 commits into from Apr 23, 2019
Merged

Fries for crobs #5

merged 7 commits into from Apr 23, 2019

Conversation

etcadinfinitum
Copy link
Collaborator

@etcadinfinitum etcadinfinitum commented Apr 22, 2019

Add reaction functionality for posts.

Could use a few more changes:

  • safety checking for presence + truth of all required form elements
  • add an icon for indicating fries have been given 馃崯 - fixed in 3874e2a
  • better styling - fixed in 3874e2a
  • align button and icon into a row instead of being stacked - fixed in 3874e2a

preliminary work for adding post reactions
src/main.py Outdated Show resolved Hide resolved
src/main.py Outdated Show resolved Hide resolved
src/main.py Outdated Show resolved Hide resolved
src/main.py Show resolved Hide resolved
src/post.py Show resolved Hide resolved
src/post.py Show resolved Hide resolved
src/templates/homepage.html Show resolved Hide resolved
src/templates/homepage.html Show resolved Hide resolved
Chris-Johnston and others added 5 commits April 22, 2019 15:01
it worked as is, but this is better.

Co-Authored-By: etcadinfinitum <lizzy.presland@gmail.com>
Crobs judge unnecessary standard output in _enterprise_ code

Co-Authored-By: etcadinfinitum <lizzy.presland@gmail.com>
DURRRRRRRR

Co-Authored-By: etcadinfinitum <lizzy.presland@gmail.com>
Added better stylings for the fries segment of the post
Also added the unicode 馃崯 icon for REDUNDANCY
@etcadinfinitum
Copy link
Collaborator Author

@Chris-Johnston let me know what you think about the naming conventions - I'm okay changing them if you really think we should, but I'm not inclined to make a keyword into a variable just in case something breaks down the line.

Regarding the page reload - I'm still not sure how to go about implementing this, but would be okay making this into its own feature branch & PR due to addition of JS.

In the interim, check out the majesty that is the new reaction section:
new_crobber_fry_stylings

@etcadinfinitum etcadinfinitum merged commit a72892f into master Apr 23, 2019
@etcadinfinitum etcadinfinitum deleted the fries_for_crobs branch April 23, 2019 06:58
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.

None yet

2 participants