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
Fix indentation of Dynamic Data Sources to be consistent #769
Comments
We must be careful with this. Up to now dynamic datasources will even load badly formatted (i.e. invalid) RSS feeds like Facebook feeds (and somehow "repair" certain problems). I don't know about the inner workings though. |
We must be careful with indentation or with handling external data at all? Not sure about your position here … |
I don't remember exactly, unfortunately. Once upon a time I had severe problems reading RSS feeds from Facebook as XML (in custom scripts) and decided to use dynamic datasources and output Symphony pages (then read those pages in custom scripts to further parse the content). |
Perhaps if perfomance is the problem, we could put a checkbox in the DS editor that enables/disables indentation. Would that make sense? |
Minor issue IMO, needs a test case. |
What kind of test case do you need? – It happens for all dynamic data source that I create. |
Just a sample URL & screenshot should do it :) |
Should this go into the debug devkit tracker? Sounds like a problem with Bitter. |
I think so, it's the result of this commit, which speeds up the page and prevents the devkit from crashing with some XML. I think Bitter definitely needs some TLC, last I remember @rowan-lewis was looking at http://pygments.org/ (Github's syntax highlighter) |
Dynamic data sources are not correctly indented. I guess this is because the external XML is appended as value and not converted to an internal XML object.
Nevertheless, the wrong indentation makes the XML hard to read in debug mode.
The text was updated successfully, but these errors were encountered: