Fixed a bug where the canvas would get transparent bg upon erase

This commit is contained in:
Ondřej Hruška 2017-10-13 18:13:00 +02:00
parent f1f6d8220a
commit 0cca5ac567

View file

@ -17,10 +17,15 @@ export default class DoodleModal extends ImmutablePureComponent {
handleKeyUp = (e) => { handleKeyUp = (e) => {
if (e.key === 'Delete' || e.key === 'Backspace') { if (e.key === 'Delete' || e.key === 'Backspace') {
this.sketcher.clear(); this.clearScreen();
} }
} }
clearScreen () {
this.sketcher.context.fillStyle = 'white';
this.sketcher.context.fillRect(0, 0, this.canvas.width, this.canvas.height);
}
componentDidMount () { componentDidMount () {
window.addEventListener('keyup', this.handleKeyUp, false); window.addEventListener('keyup', this.handleKeyUp, false);
} }
@ -36,9 +41,7 @@ export default class DoodleModal extends ImmutablePureComponent {
if (elem) { if (elem) {
this.sketcher = new Atrament(elem, 500, 500, 'black'); this.sketcher = new Atrament(elem, 500, 500, 'black');
// pre-fill with white this.clearScreen();
this.sketcher.context.fillStyle = 'white';
this.sketcher.context.fillRect(0, 0, elem.width, elem.height);
// .smoothing looks good with mouse but works really poorly with a tablet // .smoothing looks good with mouse but works really poorly with a tablet
this.sketcher.smoothing = false; this.sketcher.smoothing = false;