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

Art Editor #20

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

Art Editor #20

wants to merge 23 commits into from

Conversation

maddox
Copy link
Owner

@maddox maddox commented Aug 27, 2014

🚧 🚧 🚧 🚧 🚧 🚧 🚧 WIP 🚧 🚧 🚧 🚧 🚧 🚧 🚧

This is early, but I'm gonna push it up while I work on it.

This lets you edit the art for a rom right from inside Kart. It adds a new button to the game card that prompts a modal giving choices of art. I wrote a quick consolegrid.com scraper in Ruby and threw it up on Heroku so it had something simple to call out to as an API. It works pretty well.

edit-art

To Do

  • Add loading UI while the api request is still processing.
  • Add credit to consolegrid.com in the modal, linking to them.
  • Add form for adding art via a URL
  • Clean up the modal to just look a little better
  • Some error handling
  • Fix loader to be adjusted to where the art will appear
  • Make sure the art results content can scroll with an overflow
  • Add loader when updating art via url
  • Figure out why events stop working after the first time the modal is shown

@maddox
Copy link
Owner Author

maddox commented Aug 27, 2014

/cc #15

Show the modal immediately, with the spinner, then do the fetch, and
once we get data, append the cards into the dom and stop the spinner.
After updating atom-shell, it looks like the new version of chrome in
it has better caching. We need to be sure we break it.
@maddox
Copy link
Owner Author

maddox commented Aug 28, 2014

spinner

@maddox
Copy link
Owner Author

maddox commented Aug 28, 2014

Added some attribution. The problem is, the links don't open an external browser. I had thought that _blank made it work in the past, but maybe updating atom-shell broke it.

I'd hate to think we have to listen for the click and use shell.openExternal()

screen shot 2014-08-28 at 12 26 53 am

@maddox
Copy link
Owner Author

maddox commented Aug 28, 2014

Yup, we do.

@jrsconfitto
Copy link
Contributor

Nice 😄. They blend reasonably well so far...

image

@jrsconfitto
Copy link
Contributor

i like how this works. WAY easier than downloading files and making sure they get named right and put in the right folder. 🤘

@maddox
Copy link
Owner Author

maddox commented Aug 29, 2014

Niceee. Let's get #21 working and landed first, then we'll get #20 landed after that.

@maddox
Copy link
Owner Author

maddox commented Sep 10, 2014

Got this close tonight, but ran into some more issues. I added them to the task list.

screen shot 2014-09-10 at 1 30 51 am

@maddox
Copy link
Owner Author

maddox commented Sep 10, 2014

There's also a problem where the hooked events don't work anymore after the first time the modal is showed. They work the first time, but after that they don't.

This includes clicks on links to art providers in the footer of the modal and the form submission events.

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