I had no idea what you were talking about here; had to look into code to understand. I'll rewrite this ticket so that someone could understand it without looking at the code.
We can do this. I also think we should be able to produce HTML that's readable without a DOM inspector, Mr. I. Hate GUIs :). Note that the wrap achieves not only comments but also appropriate white space handling so that humans can understand the data. Gimme that old time unix religion.
There are places where this is absolutely a bug. I recently ran into it when trying to use inclusions in some embed code. I was trying to do a quick example of how to make a cardtype template to embed a YouTube video, and the comments invaded the URL.
Readability is overrated for generated HTML. For debugging javascript, css and html generated, there really isn't any other way than to use the dev-tools of a modern browser.
Then I suspect you were using the wrong view. You only get that html comment if you get a slot (a div). If you wanted the raw url and were just doing a default (content view) inclusion, you were screwed with or without the comment.
I'm also noticing that content seems to be wrapped twice in a div with class=card-content. It's beginning to matter a bit to me because I'm playing with code to do 'live-editing' on the HTML directly. This is the 'better than just grafting on etherpad' idea.
Don't Wrap Slots with Html Comments by Default+discussion
I had no idea what you were talking about here; had to look into code to understand. I'll rewrite this ticket so that someone could understand it without looking at the code.
We can do this. I also think we should be able to produce HTML that's readable without a DOM inspector, Mr. I. Hate GUIs :). Note that the wrap achieves not only comments but also appropriate white space handling so that humans can understand the data. Gimme that old time unix religion.
There are places where this is absolutely a bug. I recently ran into it when trying to use inclusions in some embed code. I was trying to do a quick example of how to make a cardtype template to embed a YouTube video, and the comments invaded the URL.
Readability is overrated for generated HTML. For debugging javascript, css and html generated, there really isn't any other way than to use the dev-tools of a modern browser.
Then I suspect you were using the wrong view. You only get that html comment if you get a slot (a div). If you wanted the raw url and were just doing a default (content view) inclusion, you were screwed with or without the comment.
I'm also noticing that content seems to be wrapped twice in a div with class=card-content. It's beginning to matter a bit to me because I'm playing with code to do 'live-editing' on the HTML directly. This is the 'better than just grafting on etherpad' idea.
I think I tried a raw view, but that was a few weeks back now.
I think I know how to do this, you want me to make a PR for it?
Sure you could make a PR. I won't really be back at work until Jan 6.
The card wrapping is quite clean now, so if you've got two card-content divs, they're probably both right.
will change this to a ticket. Gerry already made a PR.