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

added svg formatter #286

Merged
merged 1 commit into from Sep 29, 2019
Merged

added svg formatter #286

merged 1 commit into from Sep 29, 2019

Conversation

rsteube
Copy link
Contributor

@rsteube rsteube commented Sep 28, 2019

basic version without any options. colors and font-styles seem to be ok.
rough support for text background in styles like murphy using predrawn
rectangles (svg has no text background attribute).

things to improve:

  • svg width attribute (<svg width="")
  • linenumbers
  • highlighting
  • embedded font
  • tabwidth option
  • margins?
  • better position/width calculation (rectangles not correctly drawn on
    resize)

@rsteube
Copy link
Contributor Author

rsteube commented Sep 28, 2019

related to #245
@alecthomas code is still a bit messy, but output looks pretty decent so might as well use this as initial version. just tell me what you want to be improved/fixed for this PR.

basic version without any options. colors and font-styles seem to be ok.
rough support for text background in styles like `murphy` using predrawn
rectangles (svg has no text background attribute).

things to improve:
- svg width attribute (`<svg width=""`)
- linenumbers
- highlighting
- embedded font
- tabwidth option
- margins?
- better position/width calculation (rectangles not correctly drawn on
resize)
@alecthomas
Copy link
Owner

Looks pretty good to me for a first pass, thanks very much.

@alecthomas alecthomas merged commit 22511fb into alecthomas:master Sep 29, 2019
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

Successfully merging this pull request may close these issues.

None yet

2 participants