thenickdude / chickenpaint

An HTML5 Port of the ChibiPaint multi-layer Oekaki painting tool
GNU General Public License v3.0
117 stars 21 forks source link

Fixed deprecated JavaScript and jQuery. Fixed shortcut keys that were no longer working. Add shortcut keys. #56

Closed satopian closed 8 months ago

satopian commented 9 months ago

$("a").click(function(e)is deprecated. Therefore, I replaced it with $("a").on('click',function(e). KeyboardEvent: keyCode property - Web APIs | MDN keycode is deprecated. I also modified "keymaster.js" and replaced it with e.key to get the key. I also replaced the parts other than "Bootstrap4" with e.key. dependabot[bot] often point out vulnerabilities in npm packages, so we removed the part about icomoon in Make and adjusted the package-lock.json so that there were no vulnerabilities at build time. In Google Chrome, it is no longer possible to rotate the canvas using "alt+space+left click", so I replaced it with "R+left-click". Simply hold down the R key and drag to rotate the canvas. Rotation by "R key + drag" is a shortcut key that is also implemented in other Paint software. "Alt+Mousewheel" worked, but moving the mousewheel to the left would return you to the previous screen in the browser and potentially cause you to lose your drawing, so I removed the "Alt+Mousewheel" shortcut key. I discovered an issue where all shortcut keys stop working immediately after pressing the "alt" key. The default behavior of the "alt key" was conflicting. After canceling the default behavior of the "alt" key, it now works as expected. In some painting software, the shortcut key "H key" is assigned to flip horizontally. Some people often use horizontal flip to check their drawings. That's me. You can easily flip it left and right, making your work easier. You can flip horizontally by simply pressing the "H key". In addition, we made it so that the "A key" switches to airbrush, and the "W key" switches to watercolor, but these may deviate from ChickenPaint's development policy, so please switch back if necessary. I think it needs to be restored.

@thenickdude These are just updates that I thought were necessary for my use and I apologize if they stray from the development policy.