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

Display level name and number in play mode #4

Closed grady404 closed 8 years ago

grady404 commented 8 years ago

Use the "name" attribute in the level file (under "data") to display the level name and number within the play screen. I suggest displaying it in the bottom left corner, the number followed by the name, with the number in a little circle (EDIT: see comment below) to make it look nicer. Example: "⓬ Jailbreak". I'd say white with gray text (same as background color) for the circle and white text for the level name, don't know how you'd exactly make the circle, but I trust you'll be able to. And obviously, make everything Hasklig.

cooperuser commented 8 years ago
screen shot 2016-09-07 at 8 09 06 pm
grady404 commented 8 years ago

Did you also forget to get rid of "test" at the bottom? haha

grady404 commented 8 years ago

But- you're not actually going to use the unicode characters, right? Because there are only finitely many of those...

grady404 commented 8 years ago

You know what? I would actually use a rounded rectangle instead of a circle, that way it will display correctly no matter how many digits are in the level number. That will also make it easier for you to implement.

cooperuser commented 8 years ago

Have you gotten to a point in adding your levels where you think we should merge the branch?

grady404 commented 8 years ago

Let me extract the current branch so I can sort through it and see what's in there, and once I do I would just recommend replacing the old branch with the new one including the user levels (don't do it yet though).

cooperuser commented 8 years ago

Let me know when, cause merging the two is as simple as pressing a button and the only changes that will be made to the master branch are only the changes you made in levels-main

grady404 commented 8 years ago

Yeah but I think you should completely REPLACE the level directory, as opposed to merging the two versions (which would imply keeping files from both), but like I said not yet, I'm trying to figure out how to download the directory

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

Let me know when, cause merging the two is as simple as pressing a button and the only changes that will be made to the master branch are only the changes you made in levels-main

— 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/4#issuecomment-245487611, or mute the thread https://github.com/notifications/unsubscribe-auth/AJyH7sGyVf7SrsazpBCr9YMVswybZb_9ks5qn4idgaJpZM4J3kIc .

grady404 commented 8 years ago

I like where you're going with this, but I think the bottom left corner of the screen is too cluttered. You don't need to show the creators inside of the level, show that on the level select screen. And instead of having separate lines for each thing, just make it something like this: <button [whatever button stuff you do to make it look nice]>NUM</button> Level Name

cooperuser commented 8 years ago

Makes sense, I'll do that

cooperuser commented 8 years ago

ooh!! bootstrap labels!

cooperuser commented 8 years ago
screen shot 2016-09-07 at 10 40 27 pm
cooperuser commented 8 years ago

any particular color?

grady404 commented 8 years ago

Oh yeah, of course. Those would look really nice. I'd go with info (that's the same color as the blocks right?)

cooperuser commented 8 years ago

indeed

grady404 commented 8 years ago

Then that for sure

grady404 commented 8 years ago

Minor thing, upon completing the last available level the "Continue" button shouldn't be disabled, it should just take you back to the level select when you click it

cooperuser commented 8 years ago
screen shot 2016-09-07 at 10 43 24 pm
grady404 commented 8 years ago

YES! That looks great. The game already seems a lot more complete

cooperuser commented 8 years ago

yeah definitely!

cooperuser commented 8 years ago

ooh! a thing we should add to the todo list, fix all the TODO things

cooperuser commented 8 years ago

(but that doesnt belong in this thread)

grady404 commented 8 years ago

I'll add that haha

cooperuser commented 8 years ago

k good

cooperuser commented 8 years ago

So would you say that considers this thread closed?

grady404 commented 8 years ago

Yep, soon as you add it to the repo.

cooperuser commented 8 years ago

awesome

grady404 commented 8 years ago

Did you do my "minor thing"? ^^^