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

Talking about the Markup #8

Open
Grienauer opened this issue Mar 22, 2020 · 1 comment
Open

Talking about the Markup #8

Grienauer opened this issue Mar 22, 2020 · 1 comment

Comments

@Grienauer
Copy link
Collaborator

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 h2

alt 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

@davidspiola
Copy link
Collaborator

yes, that was the intension. The adoptions are not final and need to be discussed. Thx for opening an issue

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

No branches or pull requests

2 participants