1580158
/
canvas-zoom
mirror of https://github.com/richrobber2/canvas-zoom.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Replaced the multiple equality checks with a single Array.prototype.includes call for better readability. 

Used the nullish coalescing operator (??) to simplify the assignment of result[key] when userValue is undefined.
testing_refactoring
richrobber2 2024-06-12 02:56:04 -07:00
parent 260be2a9cc
commit e00b3f58a4
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
javascript/canvas-zoom.js
View File

@ -166,10 +166,10 @@ onUiLoaded(async () => {
return Object.entries(defaultHotkeysConfig).reduce((result, [key, defaultValue]) => {
const userValue = hotkeysConfigOpts[key];
if (key === "canvas_zoom_undo_extra_key" || key === "canvas_zoom_inc_brush_size" || key === "canvas_zoom_dec_brush_size") {
if (["canvas_zoom_undo_extra_key", "canvas_zoom_inc_brush_size", "canvas_zoom_dec_brush_size"].includes(key)) {
result[key] = userValue;
} else if (userValue === undefined || invalidTypes.has(typeof userValue) || userValue.startsWith("#") || userValue === "disable") {
result[key] = userValue === undefined ? defaultValue : userValue;
result[key] = userValue ?? defaultValue;
} else if (isValidHotkey(userValue)) {
const normalizedUserValue = normalizeHotkey(userValue);