NoRedInk/noredink-ui
1
1
Fork 0
mirror of https://github.com/NoRedInk/noredink-ui.git synced 2024-11-27 13:02:42 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix current color implementation

Browse Source
This commit is contained in:
Tessa Kelly 2022-07-08 18:03:02 -07:00
parent b584b3a916
commit 2ff116155e
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/Nri/Ui/Svg/V1.elm
View File

@ -16,7 +16,6 @@ import Accessibility.Styled.Aria as Aria
import Accessibility.Styled.Role as Role
import Css exposing (Color)
import Html.Styled.Attributes
import Nri.Ui.Colors.Extra exposing (toCssString)
import Svg.Styled
import Svg.Styled.Attributes exposing (..)
@ -109,14 +108,14 @@ toHtml (Svg record) =
|> Maybe.withDefault (Css.height (Css.pct 100))
color =
fill (Maybe.withDefault "currentcolor" (Maybe.map toCssString record.color))
Maybe.map Css.color record.color
|> Maybe.withDefault (Css.batch [])
in
Svg.Styled.svg
(List.filterMap identity
[ Just (viewBox record.viewBox)
, Just color
, -- TODO: what css is supported on this svg node?
Just (css (width :: height :: record.css))
, Just (fill "currentcolor")
, Just (css (width :: height :: color :: record.css))
, -- TODO: Use a title svg node instead
Maybe.map Aria.label record.label
|> Maybe.withDefault (Aria.hidden True)