Cache ChatLine layout data for visible chatlines

[quassel.git] / src / qtui / chatview.cpp

diff --git a/src/qtui/chatview.cpp b/src/qtui/chatview.cpp
index 453ac04..e61dc9e 100644 (file)
--- a/src/qtui/chatview.cpp
+++ b/src/qtui/chatview.cpp
@@ -55,6 +55,8 @@ void ChatView::init(MessageFilter *filter) {
   _bufferContainer = 0;
   _currentScaleFactor = 1;
   _invalidateFilter = false;
+  _markerLineVisible = true;
+  _markedLine = 0;
 
   setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
   setVerticalScrollBarPolicy(Qt::ScrollBarAsNeeded);
@@ -79,6 +81,8 @@ void ChatView::init(MessageFilter *filter) {
   connect(verticalScrollBar(), SIGNAL(valueChanged(int)), this, SLOT(verticalScrollbarChanged(int)));
   _lastScrollbarPos = verticalScrollBar()->value();
 
+  connect(Client::networkModel(), SIGNAL(markerLineSet(BufferId,MsgId)), SLOT(markerLineSet(BufferId,MsgId)));
+
   // only connect if client is synched with a core
   if(Client::isConnected())
     connect(Client::ignoreListManager(), SIGNAL(ignoreListChanged()), this, SLOT(invalidateFilter()));
@@ -119,6 +123,8 @@ bool ChatView::event(QEvent *event) {
 void ChatView::resizeEvent(QResizeEvent *event) {
   QGraphicsView::resizeEvent(event);
 
+  // FIXME: do we really need to scroll down on resize?
+
   // we can reduce viewport updates if we scroll to the bottom allready at the beginning
   verticalScrollBar()->setValue(verticalScrollBar()->maximum());
   scene()->updateForViewport(viewport()->width(), viewport()->height());
@@ -126,6 +132,8 @@ void ChatView::resizeEvent(QResizeEvent *event) {
 
   _lastScrollbarPos = verticalScrollBar()->maximum();
   verticalScrollBar()->setValue(verticalScrollBar()->maximum());
+
+  checkChatLineCaches();
 }
 
 void ChatView::adjustSceneRect() {
@@ -218,6 +226,7 @@ bool chatLinePtrLessThan(ChatLine *one, ChatLine *other) {
   return one->row() < other->row();
 }
 
+// TODO: figure out if it's cheaper to use a cached list (that we'd need to keep updated)
 QSet<ChatLine *> ChatView::visibleChatLines(Qt::ItemSelectionMode mode) const {
   QSet<ChatLine *> result;
   foreach(QGraphicsItem *item, items(viewport()->rect().adjusted(-1, -1, 1, 1), mode)) {
@@ -256,6 +265,45 @@ ChatLine *ChatView::lastVisibleChatLine() const {
   return 0;
 }
 
+void ChatView::setMarkerLineVisible(bool visible) {
+  if(visible != _markerLineVisible) {
+    _markerLineVisible = visible;
+  }
+}
+
+void ChatView::setMarkedLine(ChatLine *line) {
+  if(_markedLine == line)
+    return;
+
+  if(!scene()->isSingleBufferScene())
+    return;
+
+  if(line) {
+    BufferId bufId = scene()->singleBufferId();
+    Client::setMarkerLine(bufId, line->msgId());
+  }
+}
+
+void ChatView::markerLineSet(BufferId buffer, MsgId msg) {
+  if(!scene()->isSingleBufferScene() || scene()->singleBufferId() != buffer)
+    return;
+
+  ChatLine *newLine = scene()->chatLine(msg);
+  if(_markedLine == newLine)
+    return;
+
+  ChatLine *oldLine = _markedLine;
+  _markedLine = newLine;
+
+  if(oldLine)
+    oldLine->update();
+
+  if(newLine) {
+    setMarkerLineVisible(true);
+    newLine->update();
+  }
+}
+
 void ChatView::addActionsToMenu(QMenu *menu, const QPointF &pos) {
   // zoom actions
   BufferWidget *bw = qobject_cast<BufferWidget *>(bufferContainer());
@@ -295,3 +343,29 @@ void ChatView::invalidateFilter() {
     _invalidateFilter = true;
   }
 }
+
+void ChatView::scrollContentsBy(int dx, int dy) {
+  QGraphicsView::scrollContentsBy(dx, dy);
+  checkChatLineCaches();
+}
+
+void ChatView::setHasCache(ChatLine *line, bool hasCache) {
+  if(hasCache)
+    _linesWithCache.insert(line);
+  else
+    _linesWithCache.remove(line);
+}
+
+void ChatView::checkChatLineCaches() {
+  qreal top = mapToScene(viewport()->rect().topLeft()).y() - 10; // some grace area to avoid premature cleaning
+  qreal bottom = mapToScene(viewport()->rect().bottomRight()).y() + 10;
+  QSet<ChatLine *>::iterator iter = _linesWithCache.begin();
+  while(iter != _linesWithCache.end()) {
+    ChatLine *line = *iter;
+    if(line->pos().y() + line->height() < top || line->pos().y() > bottom) {
+      line->clearCache();
+      iter = _linesWithCache.erase(iter);
+    } else
+      ++iter;
+  }
+}