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

New level select sidebars #70

Closed grady404 closed 7 years ago

grady404 commented 8 years ago

Although I don't know what layout we would use, the new sidebars on the level select should include:

grady404 commented 8 years ago

blockboy12345: 9 / 10 grady404: 10 / 10 bobbydude444: 12 / 10

cooperuser commented 8 years ago

Is this what we have decided the sidebar to look like? Any changes?

grady404 commented 8 years ago

Uh, I didn't know we had anything finalized... but I would focus on implementing showMoveButtonsOnLoad first, for playtesting (just remember that the game shouldn't throw an error if the flags variable doesn't exist)

cooperuser commented 8 years ago

Okay

grady404 commented 8 years ago

And of course, for levels with that flag the buttons shouldn't go away until the user clicks one of them, rather than disappearing as soon as the user hovers over the box

cooperuser commented 8 years ago

Actually, I have been watching some people play the first level, and they see the buttons but do not know what they are, they hover over them and nothing happens and they do not click, but then they move off of it and it disappears and they go "Oh, I see" and move back on and click a button.

grady404 commented 8 years ago

If that's the case, I actually have the feeling that we won't want to show the move buttons at all, because I think that for the most part players will have the natural instinct to try hovering over one of the boxes (and then they will see the buttons appear, and know that they are significant). We'll have to experiment with all three different possibilities, so for now I suggest you make two flags: showMoveButtonsUntilClick and showMoveButtonsUntilHover (and I would say that the former will override the latter). When we finalize which version of the flag we will be using, we can delete the other one.

cooperuser commented 8 years ago

Okay

grady404 commented 8 years ago

By the way, are you happy with the current code for the level select screen, or do you plan to rewrite it?

cooperuser commented 8 years ago

I did plan to rewrite it, but now that you fixed the but I don't think I will need to.

cooperuser commented 8 years ago

I re-added the flag by the way

grady404 commented 8 years ago

Alright, although you might consider moving the JS code itself to a separate levelselect.js file if that keeps it in line with the rest of the game's code. If you do that it shouldn't interfere with the structure of the code, so it doesn't warrant a rewrite.

grady404 commented 8 years ago

What flag?

cooperuser commented 8 years ago

showMoveButtonsOnLoad

grady404 commented 8 years ago

And that shows it until hover? So the showMoveButtonsOnLoadUntilClicked will be a separate flag?

cooperuser commented 8 years ago

Yes, but I am still thinking of how to implement that one. Because it goes against how the selector currently works

cooperuser commented 8 years ago

Ah, I know

cooperuser commented 8 years ago

Done

grady404 commented 8 years ago

Minor thing, if you include both showMoveButtonsUntilClick and showMoveButtonsUntilHover in the flags section, a very strange bug occurs. It's not a huge deal, but I suggest you make the system foolproof

cooperuser commented 8 years ago

Ah, okay. I will post a new issue explaining this and fix it.

cooperuser commented 8 years ago

Fixed

grady404 commented 7 years ago

A new level info sidebar was implemented in 0.3.28. See #84 regarding leaderboards.