From 3c95dbb20859debe99edc874c57ddaa6f3acf3d2 Mon Sep 17 00:00:00 2001
From: Robert Bieber <robby@bieberphoto.com>
Date: Fri, 11 Jun 2010 19:51:34 +0000
Subject: Theme Editor: Fixed bug in parser handling empty lines and made
 ParseTreeModel handle the new VIEWPORT element properly

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@26792 a1c6a512-1295-4272-9138-f99709370657
---
 utils/themeeditor/skindocument.cpp | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

(limited to 'utils/themeeditor/skindocument.cpp')

diff --git a/utils/themeeditor/skindocument.cpp b/utils/themeeditor/skindocument.cpp
index b67c70d..8547aaf 100644
--- a/utils/themeeditor/skindocument.cpp
+++ b/utils/themeeditor/skindocument.cpp
@@ -146,10 +146,8 @@ void SkinDocument::settingsChanged()
     palette.setColor(QPalette::All, QPalette::Text, fg);
     editor->setPalette(palette);
 
-    errorColor = QTextCharFormat();
     QColor highlight = settings.value("errorColor", Qt::red).value<QColor>();
-    errorColor.setBackground(highlight);
-    errorColor.setProperty(QTextFormat::FullWidthSelection, true);
+    editor->setErrorColor(highlight);
 
     /* Setting the font */
     QFont def("Monospace");
@@ -175,10 +173,19 @@ void SkinDocument::codeChanged()
     if(skin_error_line() > 0)
     {
         editor->addError(skin_error_line());
-    }
-    else
-    {
-        editor->setExtraSelections(QList<QTextEdit::ExtraSelection>());
+
+        /* Now we're going to attempt parsing again at each line, until we find
+           one that won't error out
+        QTextDocument doc(editor->document()->toPlainText());
+        if(skin_error_line() > 0)
+        {
+            QTextCursor rest(&doc);
+
+            for(int i = 0; i < skin_error_line(); i++)
+                rest.movePosition(QTextCursor::NextBlock,
+                                  QTextCursor::KeepAnchor);
+            rest.clearSelection();
+        }*/
     }
 
     if(editor->document()->toPlainText() != saved)
-- 
cgit v1.1