Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pretty up the graphic #10

Open
ryepup opened this issue Jan 7, 2016 · 0 comments
Open

pretty up the graphic #10

ryepup opened this issue Jan 7, 2016 · 0 comments

Comments

@ryepup
Copy link
Owner

ryepup commented Jan 7, 2016

add more dot styling to make it a bit more appealing

ryepup added a commit that referenced this issue Jan 7, 2016
Had a few related problems:

* the order of items in the list mattered when generating SVG, and we
  weren't guaranteed the same ordering. This cause angular to throw
  errors because the value for the SVG string would not stabilze. Sorts
  by id to get a stable list for angular
* the tests were failing because the ids were generated by uuids, and
  sometimes the ordering would match our test expectation, sometimes it
  wouldn't. Permit setting ids manually
* pull out a custom jasmine matcher for trimming strings before checking
  equality

Also messed with the DOT rendering to make nicer looking graphs.

refs #10
ryepup added a commit that referenced this issue Jan 8, 2016
ryepup added a commit that referenced this issue Jan 8, 2016
* word wrap at 30 characters
* smaller font

refs #10
@ryepup ryepup assigned ryepup and unassigned ryepup Jun 10, 2016
ryepup added a commit that referenced this issue Jan 16, 2017
* much simpler DOT output
* tons of tests
* kinda simpler code; re-uses the IR produces by the parser so it does a
  lot less mapping and bookkeeping
* prettier dot output

refs #37
refs #10
ryepup added a commit that referenced this issue Jan 16, 2017
* removes the "preparedGraph" middle man between the parser and the dot
  writer
* delete the old code

refs #10
ryepup added a commit that referenced this issue Jan 27, 2017
ryepup added a commit that referenced this issue Feb 1, 2017
Very not-obvious UI, will need to cleanup in #10

fix #42
ryepup added a commit that referenced this issue Mar 22, 2017
Small indicator of what can be zoomed in on.

refs #10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant