* 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>
* add option to change pen size with the scroll wheel
* Update the cursor size immediately when changing the size
* Add a tooltip with the size change shortcut
Co-authored-by: ophir <pere.jobs@gmail.com>
* Added cursors
* Prepare cursor code for future settings to toggle cursors on or off.
Let cursor be the color the person has currently selected
* fix cursor on mobile (still won't display it in most cases as there is no hover on mobile but at least it won't throw errors)
* use correct size for cursor
* throttle cursor update rate to dramatically improve performance by eliminating congestion
* fix remote cursor size on desktop
* show own cursor by default and renove offset
* use svg as mouse cursor for pencil to be able to apply a reduced opacity to it and view our cursor
* don't throttle local cursor
* throttle local cursor at an independent higher rate. This could be made user adjustable for low power devices
* remove let and const from client-side code
* get emit count and emit count period from configuration
* reduce network cursor updates a lot to prevent instantly getting banned with the current defaults
* prevent eraser from deleting cursors
* use group inside of svg as drawing area and only delete elements inside it with the eraser
* use transform: translate to move cursors around instead of manipulating x and y directly
* fix: add socket ids to cursor messages
* fix incorrect remote cursor scaling and make local cursor visible again after it has been moved after being hidden due to inactivity
* create cursors in a proper fashion and keep them in a separate group
* scaling has been fixed in a1a5580
* move duplicated cursor creation code to function
* show cursors above content
* pass some of ther server configuration through to the client
* fix bug introduced in a833ce9
* allocate at most half of the allowed traffic to cursor updates
* remove debugging leftover
* use feature detection instead of ua sniffing
Co-Authored-By: Ophir LOJKINE <ophir.lojkine@auto-grid.com>
* fix regression where local cursor color was not updated on color change
* Define the cursor as a tool
* Remove the cursor tool from the UI
* Throttle remote cursor updates, not local ones
* Do not increment notification count on cursor move
* Use only one pencil icon
Use the same image for the pencil icon in the menu
and the pencil cursor that appears while drawing
* Add a test for the new cursor feature
* only stop drawing remote cursor when using some tools and always draw local cursor
* increase idle period before hiding cursor
* change idle duration back and set whether a cursor should be sent when using a tool in the respective tool
Co-authored-by: Robert Beach <rdbeach@gmail.com>
Co-authored-by: Ophir LOJKINE <ophir.lojkine@auto-grid.com>
Co-authored-by: ophir <pere.jobs@gmail.com>
Make the text field of the text tool appear at the position where the text will be written.
* improved Text tool
* ignore layers for now
* only assert value is not null or undefined / allow 0 as value
* remove unused layer code
* Reformat text.js
Co-authored-by: Robert Beach <rdbeach@gmail.com>
Co-authored-by: @finnboeger
* Initial Changes (#1)
* Move socket event handlers and initial connection to connect function
* Converted await statements to .then()
* Toggle tool if clicked again and toggling option exists
Co-authored-by: Finn Böger <finnboeger@googlemail.com>
Co-authored-by: Robert Beach <rdbeach@gmail.com>
* Updated client styles
- disable selecting text in tool menu
- change design of tool boxes to be square and smaller
- reduce text size in sliders
Co-authored-by: Finn Böger <finnboeger@googlemail.com>
* Changed Apple Touch Icon
* Update size and opacity icon, include fontawesome for future use
Co-authored-by: Finn Böger <finnboeger@googlemail.com>
* Changed favicon svg
* replace font-family with valid value
* Added icons and icomoon font
* change eraser and zoom icon
Co-authored-by: Finn Böger <finnboeger@googlemail.com>
* Revert icon change for now
* Revert "Converted await statements to .then()"
This reverts commit b35e0207df.
* Revert "Revert icon change for now"
This reverts commit 34e354f00f.
* Restored new Icons
* Restored favicon
* Remove fontawesome
* Increase icon sizes
* Update the zoom icon
See https://github.com/lovasoa/whitebophir/pull/39#issuecomment-619524098
* set display inline block for tool names, otherwise firefox doesn't respect margin-bottom
* Rework tool icons
* fix color presets position
Co-authored-by: finnboeger <finnboeger@users.noreply.github.com>
Co-authored-by: Robert Beach <rdbeach@gmail.com>
Co-authored-by: Finn Böger <finnboeger@googlemail.com>
I hope this will nudge people towards watching what is already on the board
before starting to draw.
It may encourage people to be constructive.
This is a (very) small step towards #6
