* Add prettier formatting feature and also add it as a job in CI.yml file
* Update CI.yml file
* Add format job to CI.yml file
* Fix formatting issues in CI workflow file
There is now a separate size for the main pencil and the whiteout tool. This is helpful when using "White-Out" as an eraser and switching frequently, as one typically wants eraser to have larger radius.
* Remember and restore white-out size
* Isolate size toggle in pencil.js
* Undo toggle back to Pencil on tool change, replace Pencil size concept with Drawing size
Co-authored-by: ishabalin <ishabalin@apple.com>
When using an iPad with an apple stylus, finger touches now trigger the whiteout tool.
This behavior can be disabled by setting the environment variable AUTO_FINGER_WHITEOUT=disabled
* Auto white-out with a finger when using stylus
* Add AUTO_FINGER_WHITEOUT config variable
* Use local variable to remember if stylus was used and make behavior configurable
* Delete "stylus" property
* Make AUTO_FINGER_WHITEOUT on by default
Co-authored-by: Ophir LOJKINE <pere.jobs@gmail.com>
* Delete parseBool, add note about iPad/Pencil
Co-authored-by: ishabalin <ishabalin@apple.com>
Co-authored-by: Ophir LOJKINE <pere.jobs@gmail.com>
* Add mover skeleton
Atm it is just the eraser with new icons and renamed variables
* The mover not detect the object to move
* We can move ellipses and rectangles
* The mover now moves all types of objects
* Add the move everything functionality
* Aesthetic changes
* Replace the use of const and let with classic var
* Undo few commits
* Start the transform-translate implementation
* Mover now moves with transform translate
* Put the function to get the translation matrix in the Tools
* Shapes (ellipse, line, pencil, rect, and text) now properly load translate coords
* Add the transform-translate to the createSVG functions
* Done movement using transform-translate
* Fix parenthesization bug in the SVG create
* Fix comment about mover states
* Undo translation loading from Ellipse, Mover, Pencil, and Text tools
* Makes the board send update message to the mover as needed
* Remove the old code from the line tool too
* Simplify the mover tool
See #86
* update the mover icon
* Merge the mover and the hand tools
* Improve preview for pre-rendered elements
* v1.6.0
Co-authored-by: Paolo Bolzoni <paolo.bolzoni1@studenti.unipr.it>
* added whiteout icon made by Freeplk on flaticon.com
* added whiteout fluid icon (public domain).
* Added whiteout tool as secondary function of pencil tool.
Co-authored-by: Robert Beach <rdbeach@gmail.com>
* shrink icon width using stroke
* shrink whiteout icon properly
* shrink whiteout fluid icon
* reduce number of allowed messages
* simplify
* remove layer code
* renmove show/hide marker code
* change white to #ffffff
* make toggle function compatible with current implementation
* Revert changes unrelated to the whiteout tool
* Refactor secondary tool logic between ellipse and pencil
* Add translations for the whiteout pen
* Fix tests
Co-authored-by: Robert Beach <rdbeach@gmail.com>
Co-authored-by: ophir <pere.jobs@gmail.com>
(@rdbeach)
* make live version render single-point paths the same as preview does
* implemented smoothing in server side rendering
* separate preview from export
* ensure correct handling of paths with only one child
* break out rendering of 3rd point and up into separate function
* changed test for 2 points to math current implementation and added a test for a bezier curve with 3 lines
* deduplicate code to extrapolate points for bezier curves
* fix merge
* Factor more code between the client and the server
Completely remove the straight line approximation foor paths
Vastly improve preview performance
* Add support for opacity in preview
* Stream board previews to the browser
Previously, svg board previews were generated in memory
and then pushed to the client.
They are now streamed to the client without blocking the main thread.
# Performance
## Before
$ ab -n 30 -c 15 http://localhost:8080/preview/ic3
This is ApacheBench, Version 2.3 <$Revision: 1843412 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/
Benchmarking localhost (be patient).....done
Server Software:
Server Hostname: localhost
Server Port: 8080
Document Path: /preview/ic3
Document Length: 14352691 bytes
Concurrency Level: 15
Time taken for tests: 19.499 seconds
Complete requests: 30
Failed requests: 0
Total transferred: 430588920 bytes
HTML transferred: 430580730 bytes
Requests per second: 1.54 [#/sec] (mean)
Time per request: 9749.374 [ms] (mean)
Time per request: 649.958 [ms] (mean, across all concurrent requests)
Transfer rate: 21565.33 [Kbytes/sec] received
Connection Times (ms)
min mean[+/-sd] median max
Connect: 0 0 0.1 0 0
Processing: 727 8885 1561.6 9220 9817
Waiting: 718 5380 2668.8 5569 9549
Total: 727 8885 1561.6 9220 9817
Percentage of the requests served within a certain time (ms)
50% 9220
66% 9239
75% 9245
80% 9248
90% 9560
95% 9811
98% 9817
99% 9817
100% 9817 (longest request)
## After
$ ab -n 30 -c 15 http://localhost:8080/preview/ic3
This is ApacheBench, Version 2.3 <$Revision: 1843412 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/
Benchmarking localhost (be patient).....done
Server Software:
Server Hostname: localhost
Server Port: 8080
Document Path: /preview/ic3
Document Length: 14352687 bytes
Concurrency Level: 15
Time taken for tests: 17.647 seconds
Complete requests: 30
Failed requests: 0
Total transferred: 430588020 bytes
HTML transferred: 430580610 bytes
Requests per second: 1.70 [#/sec] (mean)
Time per request: 8823.254 [ms] (mean)
Time per request: 588.217 [ms] (mean, across all concurrent requests)
Transfer rate: 23828.86 [Kbytes/sec] received
Connection Times (ms)
min mean[+/-sd] median max
Connect: 0 0 0.1 0 0
Processing: 719 8212 1440.6 8657 8810
Waiting: 701 4666 2324.8 4788 8769
Total: 719 8212 1440.6 8657 8811
Percentage of the requests served within a certain time (ms)
50% 8657
66% 8670
75% 8691
80% 8790
90% 8802
95% 8808
98% 8811
99% 8811
100% 8811 (longest request)
* Write the element id and opacity only when needed
* Fix a bug in wbo_pencil_point
* Rework the integration tests for the pencil tool
* Test server-side rendering in integration tests
* Remove unused function
* Remove unused id fallback
* set return types
* fix rectangle
* set fill for rect in stylesheet instead
Co-authored-by: Ophir LOJKINE <pere.jobs@gmail.com>
* include promise polyfill
* remove es6
* add viewBox to every icon
* change self closing to explicitly closing tags
* replace window.scroll.. with document.documentElement.scroll..
* verify document.activeElement.blur() is a function before calling
* fix visuals of color chooser (colorpicker still sometimes buggy)
* do not attach to mouseleave event in IE
* attempt removal with usecapture true in IE as well, this fixes the tool switching issue
* fix text tool for IE
* fix cursor for IE
* change curMode to boolean and rename to 'active'
* replace minified canvascolor with unminified code
* replace if else structure with switch statement
* compile polyfill for each browser individually
* remove static polyfills from board.js
* Fix broken js reference
* Fix unused variable and empty console log
* Allow serving non-minified polyfills for easier debugging
* Add proper caching for polyfill.js
This request is on the critical path, we should try to make it fast
* Include more polyfills
All the DOM polyfills were missing
* Update dependencies
* Add missing polyfill
Our custom polyfill was deleted, but the one from polyfill-library hadn't been added
* Remove feature detection for Node.contains
We now have the function in the polyfills
* Update polyfill caching logic
* Remove content-Length header
Co-authored-by: @lovasoa
* Add oneTouch option when creating tool. These tools won't be selected, rather their onstart action will be executed once.
Useful for undo/redo/clear/upload/zoom-in/zoom-out
* added grid
* move pattern to dedicated file
* Reformat code in grid tool
* Restore the gradient over the active tool
* Remove some duplicated code in oneTouch handling
Co-authored-by: Robert Beach <rdbeach@gmail.com>
Co-authored-by: Ophir LOJKINE <pere.jobs@gmail.com>
* Added Circle tool as alternative function of Rectangle tool
* change toggle function to work with new icon structure and toggle name
* split circle into separate tool and added ellipse as secondary function. Also added server side rendering for svg export
* change ellipse to be default
* change circle behaviour to same as inkscape
* toggle to alternative while holding shift
* append to drawing area instead of svg root
* don't update shape after it has been drawn
* Stop logging all rectangles
* Remove es6
* Remove more es6
* Fix a few bugs with circle to ellipse toggling
* Calculate the circle params at creation time, not at rendering time
* Make the behavior of the shift key consistent
When shift is pressed, the current function is reversed: the circle tool draws an ellipse and the ellipse tool draws a circle.
This makes the tool easier to grasp.
* Add required translations
* german translation
* Update the shape immediately when pressing shift
* Fix server-side ellipse rendering
* Add a test for the circle tool
Co-authored-by: Robert Beach <rdbeach@gmail.com>
Co-authored-by: Ophir LOJKINE <pere.jobs@gmail.com>
