iamSahdeep / seo_renderer

A Flutter Web Plugin to display Text Widget as Html for SEO purpose
MIT License
114 stars 15 forks source link

Any reason for the DivElement only? #5

Open t1gu1 opened 3 years ago

t1gu1 commented 3 years ago

Hi and first, thanks for your workaround!

I want to know if there is any reason why it use DivElement only?

We can find it all here: https://api.dart.dev/stable/2.14.2/dart-html/LinkElement-class.html

Here is why Heading tags and Link tags can be important: https://www.greengeeks.ca/blog/html-tags-for-seo/

Thanks to share on this with me. Maybe I'll do a PR on that if there is no specefic reason. :)

iamSahdeep commented 3 years ago

Hi @t1gu1 πŸ’― There is no specific reason to it, Actually I had very little knowledge of HTML and SEO but was learning and about to make changes for heading tags. You are right we should be using specific tags for specific use case. I am planning on it first stable release v1.0.0 and would love to have your PR on this.

If you can do PR let me know :)πŸ₯‡

t1gu1 commented 3 years ago

Thanks for your quick reply @iamSahdeep!

I could do the PR. I'm simply really busy lately so it could take a week or two before I start that.

I will let you know in that thread if I got some question for you. πŸ€œπŸ€›

t1gu1 commented 3 years ago

With that last PR we can now use whatever we want to render text. πŸ₯‡

Next step I could check for the link element that I didn't touch in my PR.

iamSahdeep commented 3 years ago

That’s Awesome ✨ Thanks for the PRs πŸ™Œ

mgaceanu commented 3 years ago

Hi, I saw you included this PR... when will you do a new release? Thanks

iamSahdeep commented 3 years ago

@mgaceanu Hi, I am pushing v1.0.0 soon. Meanwhile these changes are pushed as version v0.3.0