Closed sheeit closed 4 years ago
Thanks for the report and for the merge request.
We try not to include css inside html. Can you please add the inline css into the separate css file?
Sure!
The code seems now ok. I have to check and test it.
Thank you.
Ok, the code is ok and I tested it. Nothing to say about it.
Can you please squash and rebase your commit to follow the contributing guide (this helps a lot auto-generating the changelog):
The commit message of each commit of the merge request does not follow the conventional commits standard.
In my opinion, there should only be 2 commits in the end:
Alright, I've done that. I had to do some reading on the git documentation, but I figured it out at the end. Cheers.
That's great, apaxy gained a better code, and on your part you learned something!
Fixes #138