added pathSide property to text on path

[melchiar]

Here's my newest implementation of the side property for text on a path. Note that I've used the name side to be consistent with the related SVG property, although SVG side is currently supported only in Firefox.

Feel free to test:
https://jsfiddle.net/melchiar/oLq91r2t/

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   83.29 |    76.93 |   85.99 |   83.01 |                                               
 fabric.js |   83.29 |    76.93 |   85.99 |   83.01 | ...,29823,29948,30028-30093,30216,30315,30532 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   83.29 |    76.93 |   85.99 |   83.01 |                                               
 fabric.js |   83.29 |    76.93 |   85.99 |   83.01 | ...,29823,29948,30028-30093,30216,30315,30532 
-----------|---------|----------|---------|---------|-----------------------------------------------

[asturur]

for the previous discussion, should we call this pathSide?

[melchiar]

sounds good. I think let's change startOffset to pathStartOffset as well for the same reason

[asturur]

yes i agree is fresh enough that no one noticed it yet.

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   83.27 |     76.9 |   85.99 |   82.99 |                                               
 fabric.js |   83.27 |     76.9 |   85.99 |   82.99 | ...,29816,29941,30021-30086,30209,30308,30525 
-----------|---------|----------|---------|---------|-----------------------------------------------

[asturur]

i'm merging.
Can you follow up with a visual test?