Giter Club home page Giter Club logo

Comments (10)

GulinSS avatar GulinSS commented on July 28, 2024

It will be refactored to using 3x3 matrix for transformations. May be it will solve issue 3 in one shot.

from boxchareditor.

GulinSS avatar GulinSS commented on July 28, 2024

It will have some DSL for support extensions drawings in future (double line, thick line).

from boxchareditor.

GulinSS avatar GulinSS commented on July 28, 2024

If it will not be have things above it will have overkill cyclomatic complexity and hard to support.

from boxchareditor.

ogt avatar ogt commented on July 28, 2024

I am not sure what you have in mind, but please go ahead and try (and refactor the simple type as well)
Feel free to take on Issues #3 ($40) and Issue #4 ($120) as well in one shot.
Can you give me an estimate on how long it will take you ?

from boxchareditor.

GulinSS avatar GulinSS commented on July 28, 2024

I try it already. There is an intermediate result. See https://github.com/GulinSS/boxchareditor/commits

from boxchareditor.

GulinSS avatar GulinSS commented on July 28, 2024

But no tests yet.

from boxchareditor.

ogt avatar ogt commented on July 28, 2024

Ok, I checkout your code. I see what you are doing - but it is still far from complete. Very few of the transformation work as expected. Here is some feedback:

  • Your editor rewrote the complete html (using 4-space tabs) (thats fine with me) but make sure that this stays consistent at either 4 or 2.
  • The page is a bit diff (bigger block cursor), everything moved a bit further down. Do you mind finding what caused that change and reverse it? (Given that the whole code appears different due to the tab size change I cannot tell what exactly caused it.)
  • As I draw the page expands in size. As if the block fonts have different height than the spaces (&nbsp) . Can you find what is causing it and fix it.
  • Make sure that you also run make lint. I see lots of lint errors.
  • Finally when you make comments in githib dont't use it like a chat or IRC. (ie single line per enter). Group together your lines in a single comment. If you want to say more on a comment you have made already, edit that comment and add more lines. (# of comments in a issue is used as an indication of activity)

My skype id is tsatalos. It may be easier to connect with me directly there if you think that some of the above comments are not clear.

from boxchareditor.

GulinSS avatar GulinSS commented on July 28, 2024

Thanks for feedback! All your instructions will be applied.

Some updates: https://github.com/GulinSS/boxchareditor/commits/gh-pages
Improved link engine.

Draft works, play :-)

from boxchareditor.

ogt avatar ogt commented on July 28, 2024

I just played with it. More of the transformations work now.
Keep in mind that that when I erase I look at the neighboring cells and I recalculate them, it leaves no hanging "tails" over the erased path. Still some key things don't work, like drawing a rectangle...

You should probably revert index.html to the original - no need to be messing with the HTML in this issue..
The only files that should need to be updated is blocklines.js and a tiny amount of drawingenging.js (unless if you want to refactor simplelines.js to make it consistent with the matrix transformation method.

Can you connect with me on skype to iterate a bit faster? My skype id is tsatalos.

from boxchareditor.

GulinSS avatar GulinSS commented on July 28, 2024

You should probably revert index.html

Html page should contains Content-Type, I added this. Probably now HTML-page looks like it was.

Still some key things don't work, like drawing a rectangle.

It works but tricky: you should delete a corner and press left/right on blank corner to get nice close.

from boxchareditor.

Related Issues (18)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.