Skip to content

Add support for dominant-baseline attribute - #46

Open
georgefst wants to merge 3 commits into
reanimate:masterfrom
georgefst:dominant-baseline
Open

Add support for dominant-baseline attribute #46
georgefst wants to merge 3 commits into
reanimate:masterfrom
georgefst:dominant-baseline

Conversation

@georgefst

@georgefst georgefst commented Nov 25, 2023

Copy link
Copy Markdown
Collaborator

(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