summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2017-06-26 19:10:28 +0300
committerAndrei Karas <akaras@inbox.ru>2017-06-26 19:37:23 +0300
commitd599ae437a5966d3730939c80811763b62e709b1 (patch)
tree4bc20b96914553aeea18de292628815bf24c4236
parentef6a173d875114f31509823541c972667d987f87 (diff)
downloadplus-d599ae437a5966d3730939c80811763b62e709b1.tar.gz
plus-d599ae437a5966d3730939c80811763b62e709b1.tar.bz2
plus-d599ae437a5966d3730939c80811763b62e709b1.tar.xz
plus-d599ae437a5966d3730939c80811763b62e709b1.zip
Imrove staticbrowserbox.
Fix wrong width in some cases. Improved performance. Removed useless code.
-rw-r--r--src/gui/popups/speechbubble.cpp1
-rw-r--r--src/gui/widgets/staticbrowserbox.cpp49
-rw-r--r--src/gui/widgets/staticbrowserbox.h12
3 files changed, 16 insertions, 46 deletions
diff --git a/src/gui/popups/speechbubble.cpp b/src/gui/popups/speechbubble.cpp
index 382d3c93c..1ae67fb99 100644
--- a/src/gui/popups/speechbubble.cpp
+++ b/src/gui/popups/speechbubble.cpp
@@ -79,7 +79,6 @@ void SpeechBubble::setText(const std::string &text, const bool showName)
int width = mCaption->getWidth();
mSpeechBox->clearRows();
mSpeechBox->addRow(text);
- mSpeechBox->setWidth(mSpeechBox->getDataWidth());
mSpeechBox->updateHeight();
const int speechWidth = mSpeechBox->getWidth();
diff --git a/src/gui/widgets/staticbrowserbox.cpp b/src/gui/widgets/staticbrowserbox.cpp
index 3efcda283..801ec0724 100644
--- a/src/gui/widgets/staticbrowserbox.cpp
+++ b/src/gui/widgets/staticbrowserbox.cpp
@@ -60,7 +60,6 @@ StaticBrowserBox::StaticBrowserBox(const Widget2 *const widget,
const std::string &skin) :
Widget(widget),
MouseListener(),
- WidgetListener(),
mTextRows(),
mTextRowLinksCount(),
mLineParts(),
@@ -75,7 +74,6 @@ StaticBrowserBox::StaticBrowserBox(const Widget2 *const widget,
mPadding(0),
mNewLinePadding(15U),
mItemPadding(0),
- mDataWidth(0),
mHighlightColor(getThemeColor(ThemeColorId::HIGHLIGHT)),
mHyperLinkColor(getThemeColor(ThemeColorId::HYPERLINK)),
mOpaque(opaque),
@@ -90,7 +88,6 @@ StaticBrowserBox::StaticBrowserBox(const Widget2 *const widget,
setFocusable(true);
addMouseListener(this);
- addWidgetListener(this);
mBackgroundColor = getThemeColor(ThemeColorId::BACKGROUND);
@@ -270,7 +267,7 @@ void StaticBrowserBox::addRow(const std::string &row,
plain.erase(idx1, 3);
}
- // Adjust the StaticBrowserBox size
+ // Adjust the StaticBrowserBox size. This need only for implementing "---"
const int w = startBold ?
boldFont->getWidth(plain) : font->getWidth(plain) + 2 * mPadding;
if (w > getWidth())
@@ -301,7 +298,6 @@ void StaticBrowserBox::clearRows()
setWidth(0);
setHeight(0);
mSelectedLink = -1;
- mDataWidth = 0;
}
void StaticBrowserBox::mousePressed(MouseEvent &event)
@@ -345,7 +341,7 @@ void StaticBrowserBox::draw(Graphics *const graphics)
if (mDimension.width != mWidth)
{
updateHeight();
- reportAlways("browserbox resize in draw");
+ reportAlways("browserbox resize in draw: %d, %d", mDimension.width, mWidth);
}
if (mOpaque == Opaque_true)
@@ -423,17 +419,13 @@ void StaticBrowserBox::safeDraw(Graphics *const graphics)
StaticBrowserBox::draw(graphics);
}
-int StaticBrowserBox::calcHeight()
+void StaticBrowserBox::updateHeight()
{
unsigned int y = CAST_U32(mPadding);
int moreHeight = 0;
- int maxWidth = mDimension.width - mPadding;
- if (maxWidth < 0)
- return 1;
-
int link = 0;
bool bold = false;
- const unsigned int wWidth = CAST_U32(maxWidth);
+ const unsigned int wWidth = CAST_U32(mDimension.width - mPadding);
const Font *const font = getFont();
const int fontHeight = font->getHeight() + 2 * mItemPadding;
const int fontWidthMinus = font->getWidth("-");
@@ -441,6 +433,7 @@ int StaticBrowserBox::calcHeight()
Color selColor[2] = {mForegroundColor, mForegroundColor2};
const Color textColor[2] = {mForegroundColor, mForegroundColor2};
mLineParts.clear();
+ uint32_t dataWidth = 0;
FOR_EACH (TextRowCIter, i, mTextRows)
{
@@ -479,8 +472,8 @@ int StaticBrowserBox::calcHeight()
selColor[0], selColor[1], img));
y += CAST_U32(img->getHeight() + 2);
moreHeight += img->getHeight();
- if (img->getWidth() > maxWidth)
- maxWidth = img->getWidth() + 2;
+ if (img->getWidth() + mPadding + 2 > CAST_S32(dataWidth))
+ dataWidth = img->getWidth() + 2 + mPadding;
}
continue;
}
@@ -659,8 +652,8 @@ int StaticBrowserBox::calcHeight()
start += 3;
if (start == row.size())
{
- if (x > mDataWidth)
- mDataWidth = x;
+ if (x > dataWidth)
+ dataWidth = x;
break;
}
}
@@ -687,23 +680,16 @@ int StaticBrowserBox::calcHeight()
width = font->getWidth(part);
x += CAST_U32(width);
- if (x > mDataWidth)
- mDataWidth = x;
+ if (x > dataWidth)
+ dataWidth = x;
}
y += CAST_U32(fontHeight);
}
- if (CAST_S32(wWidth) != maxWidth)
- setWidth(maxWidth);
-
- return CAST_S32(mTextRows.size())
+ mWidth = dataWidth + mPadding;
+ mHeight = CAST_S32(mTextRows.size())
* fontHeight + moreHeight + 2 * mPadding;
-}
-
-void StaticBrowserBox::updateHeight()
-{
- mWidth = mDimension.width;
- mHeight = calcHeight();
- setHeight(mHeight);
+ setSize(mWidth,
+ mHeight);
}
std::string StaticBrowserBox::getTextAtPos(const int x,
@@ -776,8 +762,3 @@ void StaticBrowserBox::selectSelection()
mLinkHandler->handleLink(mLinks[CAST_SIZE(mSelectedLink)].link,
nullptr);
}
-
-void StaticBrowserBox::widgetResized(const Event &event A_UNUSED)
-{
- updateHeight();
-}
diff --git a/src/gui/widgets/staticbrowserbox.h b/src/gui/widgets/staticbrowserbox.h
index e2d2bc1f9..7698ec5ff 100644
--- a/src/gui/widgets/staticbrowserbox.h
+++ b/src/gui/widgets/staticbrowserbox.h
@@ -34,7 +34,6 @@
#include "gui/widgets/widget.h"
#include "listeners/mouselistener.h"
-#include "listeners/widgetlistener.h"
#include "localconsts.h"
@@ -45,8 +44,7 @@ class LinkHandler;
* parent conteiner.
*/
class StaticBrowserBox final : public Widget,
- public MouseListener,
- public WidgetListener
+ public MouseListener
{
public:
/**
@@ -142,20 +140,13 @@ class StaticBrowserBox final : public Widget,
void setForegroundColorAll(const Color &color1,
const Color &color2);
- unsigned int getDataWidth() const noexcept2 A_WARN_UNUSED
- { return mDataWidth; }
-
void moveSelectionUp();
void moveSelectionDown();
void selectSelection();
- void widgetResized(const Event &event) override final;
-
private:
- int calcHeight() A_WARN_UNUSED;
-
typedef TextRows::iterator TextRowIterator;
typedef TextRows::const_iterator TextRowCIter;
TextRows mTextRows;
@@ -180,7 +171,6 @@ class StaticBrowserBox final : public Widget,
int mPadding;
unsigned int mNewLinePadding;
int mItemPadding;
- unsigned int mDataWidth;
Color mHighlightColor;
Color mHyperLinkColor;