[Active] Skills - Selecting a different class doesn't change an active skill pane

[Cronax]

Tested in:
Google Chrome 18.0.1025.168 m

Steps to reproduce:
- Open the hero planner
- Select an initial class (for instance, Wizard)
- Open one of the skill panes but do not choose a skill
- Change your character class (to Demon Hunter for instance)

Expected result:
The open skill pane either changes to reflect the newly chosen class, or closes upon selecting a new class

Actual result:
The skill pane for the initially chosen class remains open, clicking it generates the error "Error: Skill not valid for this Hero."
__________

It's good that an error message is generated and the current implementation works, but I think that maybe it would be more user friendly and better security wise to change the behaviour to either of the expected results I defined?

The alternative, having the pane update as the class selection changes, might be a good option but I imagine that may generate more data traffic, which might limit the scalability of your application.

I have screenshots available if needed, but I can't upload them here for some reason...

P.s. I'm a software tester by trade so I've used the template I use to report bugs at work, hope that's not a problem ^^

[Dorjan]

Hey Cronax, totally! You are 110% correct.

I believe this happens for Items already I just didn't get round to doing this for the skills. As you can see server security / consistancy checks work but the userablity isn't the best.

I'll make this change when I can

Thank you Cronax!

- Dorjan

edit: I just said to myself "that's a very good bug report" so yes, that was fantastic