make sure scrollbar doesn't request messages out of scope

This commit is contained in:
Rasmus Karlsson 2017-02-06 11:38:26 +01:00
parent 759e0aea50
commit 5dbe6990bd
2 changed files with 15 additions and 2 deletions

View file

@ -4,6 +4,7 @@
#include "messages/limitedqueuesnapshot.h"
#include <boost/signals2.hpp>
#include <memory>
#include <mutex>
#include <vector>
@ -102,7 +103,8 @@ private:
int limit;
int buffer;
};
}
}
} // namespace messages
} // namespace chatterino
#endif // LIMITEDQUEUE_H

View file

@ -49,6 +49,17 @@ ChatWidgetView::layoutMessages()
int start = this->scrollbar.getCurrentValue();
if (messages.getLength() <= start) {
// The scrollbar wants to show more values than we can offer
// just return for now
return false;
// Lower start value to the last message
// start = messages.getLength() - 1;
}
int y = -(messages[start].get()->getHeight() *
(fmod(this->scrollbar.getCurrentValue(), 1)));