Skip to content
This repository has been archived by the owner on Mar 8, 2022. It is now read-only.

Solution using a more Flux-like Redux and React #49

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

Conversation

samiskin
Copy link

Here's my submission. I use a bit of a mix between Facebook's Flux and Redux, since I really like Store classes acting as a barrier between the data needed and the data representation in the state variable, while also really liking the Immutability of Redux.

@staltz
Copy link
Owner

staltz commented Dec 9, 2015

Hi, thanks for submitting.

README.md:

Your solution should be executable by opening submissions/username/index.html in the browser with no build steps.

@samiskin
Copy link
Author

samiskin commented Dec 9, 2015

Ah sorry, completely forgot! (Also turns out I had a bug anyway).
I've updated the repo 👍

@staltz
Copy link
Owner

staltz commented Feb 27, 2016

Hi, sorry for the very late review, but this seems to still be missing the compiled bundle.js.

@samiskin
Copy link
Author

Updated 👍

@samiskin samiskin changed the title Solution using an Unorthodox Redux and React Solution using a more Flux-like Redux and React Feb 27, 2016
@staltz
Copy link
Owner

staltz commented Mar 2, 2016

Hi @samiskin

Thanks. It's almost perfect, but I found a bug: I clicked wildly on the scroll buttons, and I managed to make the app have 6 rows:
screen shot 2016-03-02 at 19 42 34

It should have 5 rows always.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants