# Assessment wanted for Events 2 and Events 3

**URL:** <https://discourse.mozilla.org/t/assessment-wanted-for-events-2-and-events-3/106218>\
**Category:** Learn\
**Created:** [October 5, 2022, 9:56am UTC](https://discourse.mozilla.org/t/assessment-wanted-for-events-2-and-events-3/106218 "2022-10-05T09:56:23Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![meg\_hidey](https://avatars.discourse-cdn.com/v4/letter/m/df788c/32.png) [@meg\_hidey](https://discourse.mozilla.org/u/meg_hidey)\
**Post date:** [October 5, 2022, 9:56am UTC](https://discourse.mozilla.org/t/assessment-wanted-for-events-2-and-events-3/106218/1 "2022-10-05T09:56:23Z")

</div>

Assessments completed for [https://developer.mozilla.org/en-US/docs/Learn/JavaScript/Building\_blocks/Test\_your\_skills:\_Events#events\_2](https://developer.mozilla.org/en-US/docs/Learn/JavaScript/Building_blocks/Test_your_skills:_Events#events_2) Seeking feedback

Events 2:  
[https://jsfiddle.net/kfe7qphc/2/](https://jsfiddle.net/kfe7qphc/2/)  
Events 3:  
[https://jsfiddle.net/kfe7qphc/](https://jsfiddle.net/kfe7qphc/)

---

<div class="post-metadata">

**Author:** ![mikoMK](https://sea1.discourse-cdn.com/flex001/user_avatar/discourse.mozilla.org/mikomk/32/50456_2.png) [@mikoMK](https://discourse.mozilla.org/u/mikoMK)\
**Post date:** [October 11, 2022, 3:26pm UTC](https://discourse.mozilla.org/t/assessment-wanted-for-events-2-and-events-3/106218/2 "2022-10-11T15:26:22Z")

</div>

Hi @meg_hidey and welcome to the community 👋

Well done! Here are my comments:

## Task 2

You could make the `case`s simpler by negating the conditions and putting the x/y change inside. For example:

```auto
if (x > 50) {
  x -= 1;
}

```

Since `drawCircle(x, y, size);` is called in every `case` we can move it to out of the `switch`.

## Task 3

The click should color the `buttonBar` and not just the button. Otherwise your code is correct. 👍

I hope that helps. 🙂  
Michael
