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

Add support for dominant-baseline attribute #46

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

georgefst
Copy link
Collaborator

@georgefst georgefst commented Nov 25, 2023

(Based on #45 for now since I can't easily build otherwise)

I need this in order to process some basic SVGs output by the diagrams library.

There are a few things that could maybe be improved. I'm not totally sure about:

  • the best way to split up the types - there's some overlap with text-anchor, as well as alignment-baseline (which is not yet supported by this library)
  • where to put these new definitions - I've put it all below the existing text stuff, but maybe there's some logical order to the types and to the fields of DrawAttributes?
  • what to put in the haddocks, e.g. on DominantBaseline constructors - are these maybe copied from some source (seemingly not the SVG spec, or MDN)?

The old bound of 300 is not high enough on GHC 9.6.3. A loose bound has been chosen for the sake of simplicity and to prevent further churn.
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.

1 participant