Skip to content

Creates a new Collector Level Type#9313

Merged
Hamms merged 11 commits into
stagingfrom
collector-level-type
Jul 7, 2016
Merged

Creates a new Collector Level Type#9313
Hamms merged 11 commits into
stagingfrom
collector-level-type

Conversation

@Hamms

@Hamms Hamms commented Jul 5, 2016

Copy link
Copy Markdown
Contributor

Notes

  • Spec here
  • Depends on Creates a generic Maze Drawer class #9224
  • Note that the assets are currently those we use for artist, and should be regarded as temporary placeholder. Furthermore, this feature should be regarded as not ready for public consumption until we obtain new art assets.

Description

Collector Levels are a new Maze level type intended to emphasize the importance of block limits while providing a flexible number of in-level goals. Students will be successful if they are able to collect any number of the available collectibles within the block limits. Using too many blocks or collecting nothing will be a a failure, and managing to collect all collectibles will be considered "perfect".

Screenshots

pic description
image Level with sample art
image new blocks
image exceeded block limit
image collected none
image collected some
image collected all

@Hamms Hamms force-pushed the collector-level-type branch from d9bbfaa to 6cd4b17 Compare July 6, 2016 04:01
Comment thread apps/src/maze/collectorBlocks.js Outdated
name: 'collector_collect',
helpUrl: 'TODO',
title: 'collect',
tooltip: 'TODO',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO.

@joshlory

joshlory commented Jul 7, 2016

Copy link
Copy Markdown
Contributor

Hmm, it's been a while since I've built blockly-core. Even after ./build_with_core.sh debug I get blockly.js:14950 Uncaught Error: "collector_collect" is an unknown language block.

@Hamms

Hamms commented Jul 7, 2016

Copy link
Copy Markdown
Contributor Author

That's odd. In what context is that error?

@joshlory

joshlory commented Jul 7, 2016

Copy link
Copy Markdown
Contributor

Followed up offline. LGTM!

screen shot 2016-07-07 at 3 57 03 pm

@Hamms Hamms merged commit 43658cc into staging Jul 7, 2016
@Hamms Hamms deleted the collector-level-type branch July 7, 2016 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants