-
Notifications
You must be signed in to change notification settings - Fork 8
Description
would be great to get markup adoptions and Improvements communicated and also moved to https://codepen.io/Grienauer/pen/NWPaKBd
I think I will create a github project for it too, so we can discuss/adopt/merge better!
Please let's discuss:
https://github.com/betterembed/neos-betterembed/blob/master/Resources/Private/Fusion/Component/Default/Default.fusion
Questions:
{props.title}
I think it would be better to use a h2 or even h3 due to https://developer.mozilla.org/en-US/docs/Web/HTML/Element/article it is possible, but as you can see, also in the demo it is a h2alt text
✅ "Thumbnail from the embeded URL" is a good option.
Extra wrapper and different markup
I don't know, why we need a top area or multiple spanm elements inside of the button element.
I think this is also a matter of taste.
I think we should use the same markup and work on one "official" base or completelz split it and implement it how it should be best for each system.
I am a fan of as less divs as possible and to use just js-* classnames if a JavaScript should listen/change something.
Thx for your wort it is well build and looks fantastic!
Nico