cooperuser / blockade

A minimal but challenging puzzle game, inspired by the ice puzzles in The Legend of Zelda: Twilight Princess.
2 stars 0 forks source link

General Discussion #9

Closed grady404 closed 7 years ago

grady404 commented 8 years ago

Not an issue, but as far as I'm aware this is the only way for us to talk in a general thread. Kind of a hacky fix, but whatever.

grady404 commented 8 years ago

How am I supposed to create a label? I think it would be nicer to add labels for each priority level, than have them in the name of the issue.

cooperuser commented 8 years ago

I was about to create a discussion thread.

cooperuser commented 8 years ago

I was planning on using Labels as a way of marking severity

grady404 commented 8 years ago

Severity, as a separate thing from priority?

grady404 commented 8 years ago

And by discussion thread did you mean there was a non-hacky way to make one, or you were just going to do this

cooperuser commented 8 years ago

Yes...

cooperuser commented 8 years ago

I suppose that the severity part is already included, so I can just make priority ones

cooperuser commented 8 years ago

using the star system you proposed?

grady404 commented 8 years ago

"Yes" isn't an answer to a multiple choice question, but if there's a way to make a real thread, then do it, we'll delete this crap

grady404 commented 8 years ago

And yeah, use the star thing (go from 1 to 5, we don't really need zero)

cooperuser commented 8 years ago

Yeah.

Well the possibly-temporary-possibly-not star system has been implemented

grady404 commented 8 years ago

Alright, seems good, but I would make them all different colors (I guess red for five stars, green for one, and shades of yellow/orange/whatever for the intermediary ones)

cooperuser commented 8 years ago

Linearly interpolating? or rough estimation between colors

cooperuser commented 8 years ago

I feel like they should all be a shade of red actually, cause that makes it easier to spot all of the stars on different threads. Different shades though

grady404 commented 8 years ago

Alright, you can change what I did to them, I agree with you

grady404 commented 8 years ago

Oh but be sure to make them obviously different shades of red, not just slightly different

cooperuser commented 8 years ago

okay I changed the values, but I had to keep them somewhat similar or else the color of the text would automatically change

grady404 commented 8 years ago

Oh yeah I see what you mean, I think it's fine how you did it, also are you working on the level name/number thing? I think that should be your top priority right now, since I want to get the main functionality of the game to a point where I can get some people playtesting it

cooperuser commented 8 years ago

Good point, where should this information be displayed?

grady404 commented 8 years ago

Bottom left corner of the screen. See the issue I posted about it earlier, but ignore what I said about making the number a circle - instead look at the comment, I think it should be a rounded rectangle.

grady404 commented 8 years ago

How exactly do I download a directory? (don't send me a lmgtfy link, I'm looking it up also)

grady404 commented 8 years ago

What's "thing.json"?

grady404 commented 8 years ago

You can go ahead and replace the directory (see the other thread), all the levels that were stored in the old version of the folder have been carried over to the new version, except this "thing.json", but I have that on my local drive, what is it though?

cooperuser commented 8 years ago

thing.json? I don't recall making a file by that name...

cooperuser commented 8 years ago

ohh wait nevermind, it was a test

grady404 commented 8 years ago

Check the repository, I'm sure someone else made it, it's a weird one for sure.

On Wednesday, September 7, 2016, Cooper Anderson notifications@github.com wrote:

thing.json? I don't recall making a file by that name...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cooper-anderson/Blockade/issues/9#issuecomment-245490208, or mute the thread https://github.com/notifications/unsubscribe-auth/AJyH7uoQOdI_jVvJ2vEKLxpDZQrKRD4xks5qn45igaJpZM4J3k6Q .

cooperuser commented 8 years ago

well... it was your design

cooperuser commented 8 years ago

your original "bring one guy to rescue the other guy"

grady404 commented 8 years ago

Nope, I didn't make that, maybe it was something you/someone else made to "test" my design?

cooperuser commented 8 years ago

Well, I made the actual file, but the grid design was yours

grady404 commented 8 years ago

Ok, makes sense

grady404 commented 8 years ago

Go ahead and replace the levels directory though (don't merge it)

cooperuser commented 8 years ago

Just fixed the P glitch by the way

cooperuser commented 8 years ago

but why dont merge it?

grady404 commented 8 years ago

Because I renamed a lot of the levels in the user folder and/or moved them out of the user folder and/or changed the number of levels already outside of the user folder, so if you merge them you'll get a lot of unnecessary duplicates. The new version of the folder has all the same levels as the old version, just possibly in different places or with different names.

cooperuser commented 8 years ago

if you removed a file, i'm pretty sure it would remove the master one too, but i'm not too sure about edited files.., i'll have to look into that

grady404 commented 8 years ago

Well, I already know that GitHub doesn't recognize two files as the same if you rename it... and yeah, if you merge the two branches, it will basically overwrite all the levels in the master that also appear in the other branch, but all the levels that only appear in the master branch will stay there.

grady404 commented 8 years ago

Did you have trouble replacing it? Anyway I deleted temp.json from the repo, should I delete all the duplicate levels also?

cooperuser commented 8 years ago

If we need to, I guess we can, but I think some time tomorrow or the next day we should get the whole level-sorting system down in person. But I did "merge" the branches in the way you said.

cooperuser commented 8 years ago

Also, for running the game, are you using the new method, or the old method

grady404 commented 8 years ago

Wait huh?

cooperuser commented 8 years ago

For running the actual game, do you use the old Blockade.app file or the new run_osx.sh file

grady404 commented 8 years ago

Oh, dude, I just realized something... you know how when you sent me the game it gave me an error because you had a line of code in there that added dev tools for /Users/cooperanderson/[something...]? Well I think you forgot to take that out, because when Jack Warner tried to download your game off GitHub, it gave him the same error... so I would fix that ASAP

cooperuser commented 8 years ago

i've already fixed that one

grady404 commented 8 years ago

I guess I use Blockade.app, because I haven't heard anything about the "new" method... what is it?

cooperuser commented 8 years ago

if (__dirname == "/Users/cooperanderson/Google Drive/Other/Programming/JetBrains/WebStorm/Blockade/app"){ BrowserWindow.addDevToolsExtension("/Users/cooperanderson/Library/Application\ Support/Google/Chrome\ Canary/Default/Extensions/bomhdjeadceaggdgfoefmpeafkjhegbo/2.1.7_0"); }

grady404 commented 8 years ago

I mean, I run the .app file that you AirDropped to me.

cooperuser commented 8 years ago

ahh i see

cooperuser commented 8 years ago

do you know if you have npm and node installed on your computer?

grady404 commented 8 years ago

Oh jeez, that's a really hacky fix, but it works