<table><tr><td style="">gszymaszek requested changes to this revision.<br />gszymaszek added inline comments.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15383">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D15383#inline-83902">View Inline</a><span style="color: #4b4d51; font-weight: bold;">.editorconfig:1</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"># http://editorconfig.org
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">root = true
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;"><tt style="background: #ebebeb; font-size: 13px;">https</tt>, please 🙂</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D15383#inline-83900">View Inline</a><span style="color: #4b4d51; font-weight: bold;">.editorconfig:14</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">[*.md]
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">max_line_length = 0
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">trim_trailing_whitespace = false
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I think it’s better to suggest some line length limit in Markdown files as well, as it makes these files readable in plain text (incapable of Markdown previewing or of text soft wrapping, like mcedit) editors. Like it’s in <a href="https://cgit.kde.org/syntax-highlighting.git/tree/README.md" class="remarkup-link" target="_blank" rel="noreferrer">cgit/syntax-highlighting/tree/README.md</a></p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D15383#inline-83901">View Inline</a><span style="color: #4b4d51; font-weight: bold;">.editorconfig:18</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">[COMMIT_EDITMSG]
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Is it really a good idea?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15383">https://phabricator.kde.org/D15383</a></div></div><br /><div><strong>To: </strong>lbergdoll, hindenburg, tcanabrava, gszymaszek<br /><strong>Cc: </strong>gszymaszek, dhaumann, konsole-devel, herrold, ngraham, maximilianocuria, hindenburg<br /></div>