Theme Editor: Co Used a more efficient/less error-prone system for rendering SBS documents underneath WPS documents

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27765 a1c6a512-1295-4272-9138-f99709370657

8 files changed, 51 insertions, 14 deletions

diff --git a/utils/themeeditor/graphics/rbrenderinfo.cpp b/utils/themeeditor/graphics/rbrenderinfo.cpp
index e39a4ce..bcccc4c 100644
--- a/utils/themeeditor/graphics/rbrenderinfo.cpp
+++ b/utils/themeeditor/graphics/rbrenderinfo.cpp
@@ -23,12 +23,11 @@
 
 RBRenderInfo::RBRenderInfo(ParseTreeModel* model, ProjectModel* project,
                            SkinDocument* doc, QMap<QString, QString>* settings,
-                           DeviceState* device, RBScreen* screen,
-                           RBScreen* sbsScreen)
+                           DeviceState* device, RBScreen* screen)
                                :mProject(project), mDoc(doc),
                                mSettings(settings),
                                mDevice(device), mScreen(screen),
-                               mSbsScreen(sbsScreen), mModel(model)
+                               mModel(model)
 {
 }
 
@@ -44,7 +43,6 @@ RBRenderInfo::RBRenderInfo(const RBRenderInfo &other)
     mDevice = other.mDevice;
     mDoc = other.mDoc;
     mScreen = other.mScreen;
-    mSbsScreen = other.mSbsScreen;
     mModel = other.mModel;
 }
 
@@ -55,7 +53,6 @@ const RBRenderInfo& RBRenderInfo::operator=(const RBRenderInfo& other)
     mDevice = other.mDevice;
     mDoc = other.mDoc;
     mScreen = other.mScreen;
-    mSbsScreen = other.mSbsScreen;
     mModel = other.mModel;
 
     return *this;
diff --git a/utils/themeeditor/graphics/rbrenderinfo.h b/utils/themeeditor/graphics/rbrenderinfo.h
index 17b87db..e246033 100644
--- a/utils/themeeditor/graphics/rbrenderinfo.h
+++ b/utils/themeeditor/graphics/rbrenderinfo.h
@@ -35,8 +35,7 @@ class RBRenderInfo
 public:
     RBRenderInfo(ParseTreeModel* model,  ProjectModel* project,
                  SkinDocument* doc, QMap<QString, QString>* settings,
-                 DeviceState* device, RBScreen* screen,
-                 RBScreen* sbsScreen = 0);
+                 DeviceState* device, RBScreen* screen);
     RBRenderInfo();
     RBRenderInfo(const RBRenderInfo& other);
     virtual ~RBRenderInfo();
@@ -48,7 +47,6 @@ public:
     SkinDocument* document() const{ return mDoc; }
     QMap<QString, QString>* settings() const{ return mSettings; }
     RBScreen* screen() const{ return mScreen; }
-    RBScreen* sbsScreen() const{ return mSbsScreen; }
     ParseTreeModel* model() const{ return mModel; }
 
 private:
@@ -57,7 +55,6 @@ private:
     QMap<QString, QString>* mSettings;
     DeviceState* mDevice;
     RBScreen* mScreen;
-    RBScreen* mSbsScreen;
     ParseTreeModel* mModel;
 };
 
diff --git a/utils/themeeditor/graphics/rbscene.cpp b/utils/themeeditor/graphics/rbscene.cpp
index 836abee..206f68f 100644
--- a/utils/themeeditor/graphics/rbscene.cpp
+++ b/utils/themeeditor/graphics/rbscene.cpp
@@ -20,6 +20,7 @@
  ****************************************************************************/
 
 #include <QGraphicsSceneMouseEvent>
+#include <QGraphicsItem>
 
 #include <QDebug>
 
@@ -33,4 +34,3 @@ RBScene::RBScene(QObject* parent)
 RBScene::~RBScene()
 {
 }
-
diff --git a/utils/themeeditor/graphics/rbscene.h b/utils/themeeditor/graphics/rbscene.h
index 2c4e0ee..4cb0dfc 100644
--- a/utils/themeeditor/graphics/rbscene.h
+++ b/utils/themeeditor/graphics/rbscene.h
@@ -24,6 +24,8 @@
 
 #include <QGraphicsScene>
 
+class RBScreen;
+
 class RBScene : public QGraphicsScene
 {
     Q_OBJECT
diff --git a/utils/themeeditor/graphics/rbscreen.cpp b/utils/themeeditor/graphics/rbscreen.cpp
index 1b1adc8..2b63a6b 100644
--- a/utils/themeeditor/graphics/rbscreen.cpp
+++ b/utils/themeeditor/graphics/rbscreen.cpp
@@ -32,7 +32,7 @@
 RBScreen::RBScreen(const RBRenderInfo& info, bool remote,
                    QGraphicsItem *parent)
                        :QGraphicsItem(parent), backdrop(0), project(project),
-                       albumArt(0), customUI(0)
+                       albumArt(0), customUI(0), defaultView(0)
 {
 
     setAcceptHoverEvents(true);
@@ -221,11 +221,29 @@ void RBScreen::makeCustomUI(QString id)
     }
 }
 
+void RBScreen::endSbsRender()
+{
+    sbsChildren = children();
+
+    QList<int> keys = fonts.keys();
+    for(QList<int>::iterator i = keys.begin(); i != keys.end(); i++)
+    {
+        if(*i > 2)
+            fonts.remove(*i);
+    }
+
+    images.clear();
+    namedViewports.clear();
+    displayedViewports.clear();
+}
+
 void RBScreen::breakSBS()
 {
-    width = fullWidth;
-    height = fullHeight;
-    setParentItem(0);
+    for(QList<QGraphicsItem*>::iterator i = sbsChildren.begin()
+        ; i != sbsChildren.end(); i++)
+        (*i)->hide();
+    if(defaultView)
+        defaultView->makeFullScreen();
 }
 
 QColor RBScreen::stringToColor(QString str, QColor fallback)
diff --git a/utils/themeeditor/graphics/rbscreen.h b/utils/themeeditor/graphics/rbscreen.h
index c9f6ae2..c9ae2cf 100644
--- a/utils/themeeditor/graphics/rbscreen.h
+++ b/utils/themeeditor/graphics/rbscreen.h
@@ -86,6 +86,8 @@ public:
         }
     }
 
+    void setDefault(RBViewport* view){ defaultView = view; }
+    void endSbsRender();
     void breakSBS();
 
 protected:
@@ -111,6 +113,9 @@ private:
 
     RBAlbumArt* albumArt;
     RBViewport* customUI;
+    RBViewport* defaultView;
+
+    QList<QGraphicsItem*> sbsChildren;
 };
 
 #endif // RBSCREEN_H
diff --git a/utils/themeeditor/graphics/rbviewport.cpp b/utils/themeeditor/graphics/rbviewport.cpp
index 2c6fb9d..a986127 100644
--- a/utils/themeeditor/graphics/rbviewport.cpp
+++ b/utils/themeeditor/graphics/rbviewport.cpp
@@ -56,6 +56,16 @@ RBViewport::RBViewport(skin_element* node, const RBRenderInfo& info,
         customUI = false;
         font = screen->getFont(1);
 
+        screen->setDefault(this);
+
+        if(screen->getCustomUI())
+        {
+            RBViewport* cui = screen->getCustomUI();
+            size = cui->boundingRect();
+            setPos(cui->pos());
+
+        }
+
         /* Making sure the default viewport can't be graphically manipulated */
         setFlag(ItemIsSelectable, false);
         setFlag(ItemIsMovable, false);
@@ -299,6 +309,12 @@ void RBViewport::showPlaylist(const RBRenderInfo &info, int start,
     }
 }
 
+void RBViewport::makeFullScreen()
+{
+    size = screen->boundingRect();
+    setPos(screen->pos());
+}
+
 void RBViewport::saveGeometry()
 {
     QRectF bounds = boundingRect();
diff --git a/utils/themeeditor/graphics/rbviewport.h b/utils/themeeditor/graphics/rbviewport.h
index 9bbceb4..8544ad3 100644
--- a/utils/themeeditor/graphics/rbviewport.h
+++ b/utils/themeeditor/graphics/rbviewport.h
@@ -81,6 +81,8 @@ public:
     void showPlaylist(const RBRenderInfo& info, int start, skin_element* id3,
                       skin_element* noId3);
 
+    void makeFullScreen();
+
 protected:
     void saveGeometry();