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

Improve visibility of "can cast", "casting", "have cast" states #4

Closed
nwinter opened this Issue Jan 1, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@nwinter
Contributor

nwinter commented Jan 1, 2014

It's not obvious enough yet when the code is ready to cast, is casting, or has been cast. Beginners don't realize what's going on. Here are what things look like now:

screenshot 2013-12-31 19 22 03

screenshot 2013-12-31 19 22 15

screenshot 2013-12-31 19 21 58

Any improved design might think about improving more than just the cast button, like changing the look of the spell editor itself. And anything involving the cast button might consider a different button design, because this one falls over for i18n:

screenshot 2013-12-31 19 21 12

What I'd imagine we'd do is to come up with a new design for the cast button that gives it more space, more visual prominence, and a much more obvious animation. It would be good if it looked cool, too! See the Tome docs for info on how this stuff works.

Note that the cast button does include this kind of world-casting-progress fill as it's casting. That should either be preserved in any upgraded design or moved to some other area of the interface.

Relevant files

@nwinter

This comment has been minimized.

Show comment
Hide comment
@nwinter

nwinter Feb 22, 2014

Contributor

Having moved this down to its own area in the spell view, it's looking much more prominent now.

screenshot 2014-02-22 15 28 57

Contributor

nwinter commented Feb 22, 2014

Having moved this down to its own area in the spell view, it's looking much more prominent now.

screenshot 2014-02-22 15 28 57

@nwinter nwinter closed this Feb 22, 2014

nwinter pushed a commit that referenced this issue Jul 9, 2014

nwinter pushed a commit that referenced this issue Feb 14, 2016

clescot added a commit to clescot/codecombat that referenced this issue Jun 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment