-
Notifications
You must be signed in to change notification settings - Fork 8
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
KBHBIB-24 Share buttons on material #1587
base: develop
Are you sure you want to change the base?
Conversation
To leverage the advantages of ´svg´ in the DOM and to change the color on hover, we need to have it as a component. This would not be possible with an `img` tag
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work on this feature! I have a few comments for you to consider.
I also agree with you that it’s important to make a ticket for how our “cards” appear when shared on platforms like Facebook.
@@ -0,0 +1,23 @@ | |||
import React from "react"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Icons should live in the design system and be used like this:
import ExpandMoreIcon from "@danskernesdigitalebibliotek/dpl-design-system/build/icons/collection/ExpandMore.svg";
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How would you go around rendering an SVG without an img
tag this way?
…hen the user has scrolled past the share buttons. This is because we don't want to show the fixed share buttons in the footer
6ac1175
to
230f95d
Compare
Link to issue
https://reload.atlassian.net/browse/KBHBIB-24
Description
Screenshot of the result
Test
https://varnish.pr-1851.dpl-cms.dplplat01.dpl.reload.dk/