Solved issue #411

[sarthak-g]

Added fullscreen and unfullscreen button in Gears Activity as mentioned in issue #411

[llaske]

That's the idea.
But there is an error in the console when clicking on fullscreen/unfullscreen button:

activity.js:126 Uncaught ReferenceError: resizeHandler is not defined
    at HTMLButtonElement.<anonymous> (activity.js:126)

[sarthak-g]

I've updated the code and made a commit.

[llaske]

There is another issue: in fullscreen mode, the drawing is not aligned with the cursor. There is vertical shift to the size of the toolbar.

[sarthak-g]

If vertical shift will not be done then blank space will display as below:

@canvasOffsetY in gearsketch_main.coffee gets value only when the page is displayed for first time (with toolbar), so when fullscreen is done then size of the canvas is not updated.
What I think is we've to call @updateCanvasSize() when fullscreen as well as unfullscreen button is pressed but for doing so we need jquery.

[llaske]

Of course the vertical shift should be done but it should be take into account when mouse position is computed.
You're right, I guess the issue is related to the canvasOffsetY initialized here. This value should be updated each time there is a click on fullscreen/unfullscreen button.
Calling updateCanvasSize method of the gearSketch object, from the activity, could be a solution. I don't think you need jQuery since the gearSketch object is available from the activity.

[sarthak-g]

Fullscreen is working as expected.
Although without calling updateCanvasSize, it is working fine but I add updateCanvasSize for future safety.

[llaske]

Nice job. It works now.
Could you fix indentation issue in the activity.js file.

[sarthak-g]

Indentation issue resolved.

[llaske]

Sorry, there is still a TAB instead of white spaces before lines document.getElementById("fullscreen-button"), }); and document.getElementById("unfullscreen-button")

See the screen capture of Atom editor below:

[sarthak-g]

It is fine now I guess.

[llaske]

Cool. That's fine now.
Thanks.