-
Notifications
You must be signed in to change notification settings - Fork 155
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
text alignment? #62
Comments
Could you perhaps paste/provide your erd file here? |
@mmzx I see it even with the example sample.er |
@jonathan-chin , currently the text alignment is hard coded, that is, node specific alignment is not supported. You could e.g. do something like the following, which is only a partial solution - because globally applied for all nodes. |
@mmzx I'm also curious because my keys and node titles don't get the same styling that you do (bold, underline, italics). Is it because of my OS? because I compiled it from Stack? |
@jonathan-chin , compiling it via stack or something else would not affect outcome. |
@mmzx here is my ERD file (it's the one from the repo; just including it for completeness sake)
here (I believe) is the dot file generated by
|
The dot file is correct, generated as it should be. So my assumption will be that it is your local setup that contributes to the differences. |
Thanks. I do notice that in the dot file, there's my dot version is I also just tried some other output formats (png, jpg, gif, svg, pdf) and it's the same. right aligned, no other styling on keys. |
Other than that, if all above fails I would say it has something to do with the local setup of yours. Have you tried the above workaround with sed? |
@mmzx the above workaround works for left aligning but the primary and foreign keys are not being formatted. I noticed this in the README:
However, after doing what it suggested ( |
following up on this. I installed ERD on a new machine (completely different from previous). Now running Ubuntu 18.04LTS color and styling work but everything is right aligned like before. this happens, again, with just the sample ERD file. |
Okay. I had some commits that were not pushed from recent days. It is now explicitly defined to align to the left, which is the default anyway.
|
thanks @mmzx ! my ubuntu system works perfectly now. any idea why? it's not a dealbreaker; honestly, the text alignment was a deal breaker and I was just about to learn graphwiz myself, but now I think I'll stick with erd. I'd love to help but I dived into the source code yesterday and realized I have no idea how Haskell works. |
Yeah, stick with erd! :P
Interpreting the dot file goes beyond I am happy that your alignment issue is resolved then! |
here's what I get with the same command:
So our dot's are the same. What's the next step in processing? |
I have just done some searching in the hope there will be straightforward answer(s). I believe we are dealing with the same type of situation as described in [1]. The content of the dot file match up against the expected content: that's how far My current recommendation for the short therm would be to stick with something that works as expected. Alternatively docker or the statically built version of In [2] now, there are binaries for 64bit systems statically and dynamically linked ones. I do understand that it would not solve osx related issues, but the static binary is there for the purpose to have something which behaves equally well. |
thanks! the other stylings aren't deal breakers but the alignment was.
… On Dec 8, 2019, at 11:54 AM, Akos Marton ***@***.***> wrote:
I have just done some searching in the hope there will be straightforward answer(s). I believe we are dealing with the same type of situation as described in [1].
The content of the dot file match up against the expected content: that's how far erd itself can go; the rendered result may differ based on platform related differences.
My current recommendation for the short therm would be to stick with something that works as expected. Alternatively docker or the statically built version of erd [2]. On the long-term, we would need to see whether this can be influenced and, first of all, to be reproduced.
In [2] now, there are binaries for 64bit systems statically and dynamically linked ones. I do understand that it would not solve osx related issues, but the static binary is there for the purpose to have something which behaves equally well.
1, stackoverflow; graphviz
2, v0.2.1.0-RC1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
If you could help finding out the dirty details that would be great at least to know the reasons to have an explanation. |
I'm not sure where to begin. what happens to the dot file after it's generated? if I know that, I could pick up the line of investigation.
… On Dec 8, 2019, at 1:10 PM, Akos Marton ***@***.***> wrote:
If you could help finding out the dirty details that would be great at least to know the reasons to have an explanation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
I have created a ticket, hopefully we will gain some insights. |
@jonathan-chin , would you mind if you (we) continue the discussion on the gitlab ticket, specifically anything that is related to dot itself? |
Hell, yeah! Thank you! I am going to update the readme based on the inputs you have kindly provided. |
my issues have been solved!
… On Dec 11, 2019, at 4:52 PM, Akos Marton ***@***.***> wrote:
Hell, yeah! Thank you!
I am going to update the readme based on the inputs you have kindly provided.
If you think all of your issues has been resolved, I would like to close this. Please let me know!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
I'm running this on OSX High Sierra, installed via Stack.
My attributes seem to be right aligned, whereas the sample pdf is left aligned. I think left aligned makes it so much easier to read.
How can I specify this? I don't think there's an option in the README.
This is what I get
The text was updated successfully, but these errors were encountered: