From 73fc9c79cff979e9023df0b1a77848c67b590681 Mon Sep 17 00:00:00 2001 From: Jan Dalheimer Date: Mon, 10 Mar 2014 17:38:27 +0100 Subject: [PATCH] Instance badges. Some easter eggs and one for broken so far. --- gui/groupview/InstanceDelegate.cpp | 85 +++++++++++++++++++++++++++-- gui/groupview/InstanceDelegate.h | 6 ++ logic/BaseInstance.cpp | 9 ++- logic/BaseInstance.h | 8 +-- logic/BaseInstance_p.h | 7 ++- logic/LegacyFTBInstance.cpp | 2 +- logic/LegacyInstance.cpp | 4 +- logic/NostalgiaInstance.cpp | 2 +- logic/OneSixFTBInstance.cpp | 2 +- logic/OneSixInstance.cpp | 8 +-- resources/instances/broken.png | Bin 0 -> 1589 bytes resources/instances/instances.qrc | 7 +++ resources/instances/kitten.png | Bin 0 -> 9107 bytes 13 files changed, 115 insertions(+), 25 deletions(-) create mode 100644 resources/instances/broken.png create mode 100644 resources/instances/kitten.png diff --git a/gui/groupview/InstanceDelegate.cpp b/gui/groupview/InstanceDelegate.cpp index 8a273758..7ca4d7b4 100644 --- a/gui/groupview/InstanceDelegate.cpp +++ b/gui/groupview/InstanceDelegate.cpp @@ -18,9 +18,13 @@ #include #include #include -#include +#include #include "GroupView.h" +#include "logic/BaseInstance.h" +#include "logic/lists/InstanceList.h" + +QCache ListViewDelegate::m_pixmapCache; // Origin: Qt static void viewItemTextLayout(QTextLayout &textLayout, int lineWidth, qreal &height, @@ -45,8 +49,6 @@ static void viewItemTextLayout(QTextLayout &textLayout, int lineWidth, qreal &he textLayout.endLayout(); } -#define QFIXED_MAX (INT_MAX / 256) - ListViewDelegate::ListViewDelegate(QObject *parent) : QStyledItemDelegate(parent) { } @@ -108,6 +110,64 @@ void drawProgressOverlay(QPainter *painter, const QStyleOptionViewItemV4 &option painter->restore(); } +void drawBadges(QPainter *painter, const QStyleOptionViewItemV4 &option, BaseInstance *instance) +{ + QList pixmaps; + for (auto flag : instance->flags()) + { + switch (flag) + { + case BaseInstance::VersionBrokenFlag: + pixmaps.append("broken"); + break; + } + } + + // begin easter eggs + if (instance->name().contains("btw", Qt::CaseInsensitive) || + instance->name().contains("better then wolves", Qt::CaseInsensitive) || + instance->name().contains("better than wolves", Qt::CaseInsensitive)) + { + pixmaps.append("herobrine"); + } + if (instance->name().contains("direwolf", Qt::CaseInsensitive)) + { + pixmaps.append("enderman"); + } + if (instance->name().contains("kitten", Qt::CaseInsensitive)) + { + pixmaps.append("kitten"); + } + if (instance->name().contains("derp", Qt::CaseInsensitive)) + { + pixmaps.append("derp"); + } + // end easter eggs + + static const int itemSide = 24; + static const int spacing = 1; + const int itemsPerRow = qMax(1, qFloor(double(option.rect.width() + spacing) / double(itemSide + spacing))); + const int rows = qCeil((double)pixmaps.size() / (double)itemsPerRow); + QListIterator it(pixmaps); + painter->translate(option.rect.topLeft()); + for (int y = 0; y < rows; ++y) + { + for (int x = 0; x < itemsPerRow; ++x) + { + if (!it.hasNext()) + { + return; + } + const QPixmap pixmap = ListViewDelegate::requestPixmap(it.next()).scaled( + itemSide, itemSide, Qt::KeepAspectRatio, Qt::FastTransformation); + painter->drawPixmap(option.rect.width() - x * itemSide + qMax(x - 1, 0) * spacing - itemSide, + y * itemSide + qMax(y - 1, 0) * spacing, itemSide, itemSide, + pixmap); + } + } + painter->translate(-option.rect.topLeft()); +} + static QSize viewItemTextSize(const QStyleOptionViewItemV4 *option) { QStyle *style = option->widget ? option->widget->style() : QApplication::style(); @@ -257,8 +317,14 @@ void ListViewDelegate::paint(QPainter *painter, const QStyleOptionViewItem &opti line.draw(painter, position); } - drawProgressOverlay(painter, opt, - index.data(GroupViewRoles::ProgressValueRole).toInt(), + auto instance = (BaseInstance*)index.data(InstanceList::InstancePointerRole) + .value(); + if (instance) + { + drawBadges(painter, opt, instance); + } + + drawProgressOverlay(painter, opt, index.data(GroupViewRoles::ProgressValueRole).toInt(), index.data(GroupViewRoles::ProgressMaximumRole).toInt()); painter->restore(); @@ -284,3 +350,12 @@ QSize ListViewDelegate::sizeHint(const QStyleOptionViewItem &option, QSize sz(100, height); return sz; } + +QPixmap ListViewDelegate::requestPixmap(const QString &key) +{ + if (!m_pixmapCache.contains(key)) + { + m_pixmapCache.insert(key, new QPixmap(":/icons/badges/" + key + ".png")); + } + return *m_pixmapCache.object(key); +} diff --git a/gui/groupview/InstanceDelegate.h b/gui/groupview/InstanceDelegate.h index de2f429b..9ab44864 100644 --- a/gui/groupview/InstanceDelegate.h +++ b/gui/groupview/InstanceDelegate.h @@ -16,14 +16,20 @@ #pragma once #include +#include class ListViewDelegate : public QStyledItemDelegate { public: explicit ListViewDelegate(QObject *parent = 0); + static QPixmap requestPixmap(const QString &key); + protected: void paint(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const; QSize sizeHint(const QStyleOptionViewItem &option, const QModelIndex &index) const; + +private: + static QCache m_pixmapCache; }; diff --git a/logic/BaseInstance.cpp b/logic/BaseInstance.cpp index d78f1ea0..24af20cf 100644 --- a/logic/BaseInstance.cpp +++ b/logic/BaseInstance.cpp @@ -37,7 +37,6 @@ BaseInstance::BaseInstance(BaseInstancePrivate *d_in, const QString &rootDir, I_D(BaseInstance); d->m_settings = settings_obj; d->m_rootDir = rootDir; - d->m_flags = 0; settings().registerSetting("name", "Unnamed Instance"); settings().registerSetting("iconKey", "default"); @@ -147,13 +146,13 @@ SettingsObject &BaseInstance::settings() const return *d->m_settings; } -BaseInstance::InstanceFlags BaseInstance::flags() const +QSet BaseInstance::flags() const { I_D(const BaseInstance); - return InstanceFlags(d->m_flags); + return QSet(d->m_flags); } -void BaseInstance::setFlags(const BaseInstance::InstanceFlags flags) +void BaseInstance::setFlags(const QSet &flags) { I_D(BaseInstance); if (flags != d->m_flags) @@ -166,7 +165,7 @@ void BaseInstance::setFlags(const BaseInstance::InstanceFlags flags) bool BaseInstance::canLaunch() const { - return !(flags() & VersionBrokenFlag); + return !flags().contains(VersionBrokenFlag); } QString BaseInstance::baseJar() const diff --git a/logic/BaseInstance.h b/logic/BaseInstance.h index 27f939bc..443b0eaa 100644 --- a/logic/BaseInstance.h +++ b/logic/BaseInstance.h @@ -17,6 +17,7 @@ #include #include +#include #include @@ -184,9 +185,8 @@ public: NoFlags = 0x00, VersionBrokenFlag = 0x01 }; - Q_DECLARE_FLAGS(InstanceFlags, InstanceFlag) - InstanceFlags flags() const; - void setFlags(const BaseInstance::InstanceFlags flags); + QSet flags() const; + void setFlags(const QSet &flags); bool canLaunch() const; @@ -216,4 +216,4 @@ protected: // pointer for lazy people typedef std::shared_ptr InstancePtr; -Q_DECLARE_OPERATORS_FOR_FLAGS(BaseInstance::InstanceFlags) +Q_DECLARE_METATYPE(BaseInstance::InstanceFlag) diff --git a/logic/BaseInstance_p.h b/logic/BaseInstance_p.h index 73eebec7..8cf3b27d 100644 --- a/logic/BaseInstance_p.h +++ b/logic/BaseInstance_p.h @@ -14,10 +14,13 @@ */ #pragma once + #include +#include + #include -class BaseInstance; +#include "BaseInstance.h" #define I_D(Class) Class##Private *const d = (Class##Private * const)inst_d.get() @@ -26,5 +29,5 @@ struct BaseInstancePrivate QString m_rootDir; QString m_group; SettingsObject *m_settings; - int m_flags; + QSet m_flags; }; diff --git a/logic/LegacyFTBInstance.cpp b/logic/LegacyFTBInstance.cpp index 23cb259d..3c3356c9 100644 --- a/logic/LegacyFTBInstance.cpp +++ b/logic/LegacyFTBInstance.cpp @@ -7,7 +7,7 @@ LegacyFTBInstance::LegacyFTBInstance(const QString &rootDir, SettingsObject *set QString LegacyFTBInstance::getStatusbarDescription() { - if (flags() & VersionBrokenFlag) + if (flags().contains(VersionBrokenFlag)) { return "Legacy FTB: " + intendedVersionId() + " (broken)"; } diff --git a/logic/LegacyInstance.cpp b/logic/LegacyInstance.cpp index 3b9181e0..4f2dfd9b 100644 --- a/logic/LegacyInstance.cpp +++ b/logic/LegacyInstance.cpp @@ -268,7 +268,7 @@ QString LegacyInstance::defaultCustomBaseJar() const bool LegacyInstance::menuActionEnabled(QString action_name) const { - if (flags() & VersionBrokenFlag) + if (flags().contains(VersionBrokenFlag)) { return false; } @@ -281,7 +281,7 @@ bool LegacyInstance::menuActionEnabled(QString action_name) const QString LegacyInstance::getStatusbarDescription() { - if (flags() & VersionBrokenFlag) + if (flags().contains(VersionBrokenFlag)) { return "Legacy : " + intendedVersionId() + " (broken)"; } diff --git a/logic/NostalgiaInstance.cpp b/logic/NostalgiaInstance.cpp index 96ce4cc7..52820725 100644 --- a/logic/NostalgiaInstance.cpp +++ b/logic/NostalgiaInstance.cpp @@ -23,7 +23,7 @@ NostalgiaInstance::NostalgiaInstance(const QString &rootDir, SettingsObject *set QString NostalgiaInstance::getStatusbarDescription() { - if (flags() & VersionBrokenFlag) + if (flags().contains(VersionBrokenFlag)) { return "Nostalgia : " + intendedVersionId() + " (broken)"; } diff --git a/logic/OneSixFTBInstance.cpp b/logic/OneSixFTBInstance.cpp index 91efce8e..07b123b6 100644 --- a/logic/OneSixFTBInstance.cpp +++ b/logic/OneSixFTBInstance.cpp @@ -182,7 +182,7 @@ bool OneSixFTBInstance::providesVersionFile() const QString OneSixFTBInstance::getStatusbarDescription() { - if (flags() & VersionBrokenFlag) + if (flags().contains(VersionBrokenFlag)) { return "OneSix FTB: " + intendedVersionId() + " (broken)"; } diff --git a/logic/OneSixInstance.cpp b/logic/OneSixInstance.cpp index 10411c56..148fbc09 100644 --- a/logic/OneSixInstance.cpp +++ b/logic/OneSixInstance.cpp @@ -327,12 +327,12 @@ bool OneSixInstance::reloadVersion(QWidget *widgetParent) } if (ret) { - setFlags(flags() & ~VersionBrokenFlag); + d->m_flags.remove(VersionBrokenFlag); emit versionReloaded(); } else { - setFlags(flags() | VersionBrokenFlag); + d->m_flags.insert(VersionBrokenFlag); } return ret; } @@ -369,7 +369,7 @@ QString OneSixInstance::defaultCustomBaseJar() const bool OneSixInstance::menuActionEnabled(QString action_name) const { - if (flags() & VersionBrokenFlag) + if (flags().contains(VersionBrokenFlag)) { return false; } @@ -387,7 +387,7 @@ QString OneSixInstance::getStatusbarDescription() { descr += " (custom)"; } - if (flags() & VersionBrokenFlag) + if (flags().contains(VersionBrokenFlag)) { descr += " (broken)"; } diff --git a/resources/instances/broken.png b/resources/instances/broken.png new file mode 100644 index 0000000000000000000000000000000000000000..9be55691366ae61b828b814489bb7ec91f21ba9c GIT binary patch literal 1589 zcmaJ>drT8|9KZM=zy?yL4m2C*f$k-}N8dLJYkO3>SR`$om2SduJ!r+=mAk{%mc%D2 z_+-l_1lh|uA2G&ly0O7zyxhczB$AM!=x`zuV%#JVv@Yz5fc&v|$=&bu`Fy_5-&N#f zuZW3W6b%4Cj46w-h+|y%jf@b#Nh9N=I3x<0cEQRP3OD*Lu zkxm1E*>1PZF4)a0afXS)Xt_Vgo8ckRO!%{^=D!+*lNPjBBCr>F5G*5AEui$1GFsw)x zu%&_)5`( z`n+wEmxe7g&z5qOSjEER8K$0ZC=8s80XLIeF)dcnNN(Wii3774Cs?Y~LoU2ACoeD8(G%|%sBO_3hFsd;fW>jjFI#ivB5=vx>YhrkTWGH%y>wd0M#vLHgNBgFHARY#Ck-n)9dvH zqY+hMs6q+Bq8o6yKW6mH)Rx$U;pueui;L;|Lo;H!Iq`NZ>^YtZ0J92AgwE#g-D}Cq zvXb*NMjL)0*2h(C-TU$I$J}?J&2tkUy!JD1A(9S;d(NO1TU9z^W8p!=HMQf~& z9bY$mn6K+uf3UBwJTDmhynDD*qHFv{trMG%xSGoJkVZnCZWQAclp`2bDh~B z^;KbYTZQdJ-=!Sg>7-04m;VmxKDknP`n@kI zTYh{$R^N}PgMsupt^Fl?8zr-ztlT20iAo$Zv<(TB!}jG1$Kv-4)Gn|&-f~ICY9m+2 zuUb}mTmF$fX7)}~20GR@IKmvW;8E?8K8m_wTXrPRHb3j=)yECx<=~y2D?cr})V~FP z^GM$wSMGeY{?&b0psK5*BmPmIN>%fFLRqfE5xB#B(AWFNQ0w^3-LKs|du{8wqb1A!f(p4%^_-@l(R z*!0SxK*j4dY2^XtR8`mAb zDR0;H(0LeMWnC)|Ua!{cOEM|=FDl*J@NYGi-0MHvH180OcgDmojZSGommL3Xv8Q!h znA>~k+qL72X5Q|p8R@i;K7m-SNi4`5WoRc6n@#W|6%w?ZpzFiPV4ix F{Ri2LU2XsX literal 0 HcmV?d00001 diff --git a/resources/instances/instances.qrc b/resources/instances/instances.qrc index ec3cdf21..22e2936d 100644 --- a/resources/instances/instances.qrc +++ b/resources/instances/instances.qrc @@ -32,4 +32,11 @@ squarecreeper128.png steve128.png + + enderman.png + herobrine.png + kitten.png + broken.png + derp.png + diff --git a/resources/instances/kitten.png b/resources/instances/kitten.png new file mode 100644 index 0000000000000000000000000000000000000000..f9999b06138f1b8aa946ba4d23e32a287492962e GIT binary patch literal 9107 zcmeI1X*kq*0LRU^W|$d+Ovo{E=DtQF?XD3K@r>jwxuOhZGtr^7Ge{GXGn8wF9(OdT zcF9$YJ0?UHlX7;aRq2@ht9{y@&2D*~&A$21iGSCG zbn+~|xUY(LZE(#S;pSrZ-R?YUSZl*gOjJ@*8lq7M zdI})~9c#SS$c?DCisSU*!jQaX{6T%Xz;vGsrY} zS&$&S={keZ%F1UAbOf+(H1x8ed1lRR)q_QRc~EwDJ!icC?j&pK!TqAqCp}I55CNz( z(P=BE;n>4EuVvWgM51A=zC-96iS~7RIY{z=jR@Ia!6`n@C`qYJ-(-&H(YS5MG6`4F4~K6W z+8kA1;*8<+E&MJszD5yCYp9vu>CQSS!m?Ctih?mM%iZIQp{F^+f_waHH`I;|6i1#e z)N{vRr-%A0oy1MM7`mA)t%9LDta1kCPhOH2CL12gkPg64xhNnZ^9`P4-q}>y$fp*q zIbK$3uu&azm+3nvu+eJXQ3@b6AT=O0VAiZ$)&R%SUndkFctU|E)Ur<~kQ%VpEWg)) z)PU50)PNgmB{ozPP;&ld{sY?|FaRIt|A*}l41Zw$f6ev>9{ykSruY-@mwh7b!?G9o zUzdF*B}HOx71pCBDE&>_v#J2>HRyv;i6VLjr8yR@v8yWmfKBJNSY!gt_z>Xtci3M z#2C=T$J9y>vKXu++M9mJM2cibxaw;wGaXywkPKYpD({}|+8V_|nzsEW+cUnQO`4++ zDr>fnj6V>a6^6vXM6oaHvK$Y!;A9<+KzE(HxU=!)ESg7dA?WGV>_|gZRLcGw{caBx zRXhoKxc{=N61C9JUefN6asg{3n=Hj(C=SJpDM=bA>Kc$s0&`2uA$F&*#xkUSXj zC~`e1t~Jp^JOP5?;&rDz33{oT?o5MM0_W9IQNiuUG8_t+@dldgZMmM*-Yk(IIb}D5 z8iLNCSp!aKmXBXqgOO=nt$s5@jvmVIpPYMB2KVe#b#I7VmsxnD zwIs%X!aH_2?1j}?3!31IpM(GeS)r?MX%^VA^#P}3wdj<5+s$-68qHh)rxg0Vs&t@Y z#Bi6q@Q<^{j&ujcilWV7;;tpy!31p_C@l}|7p@clok!U4MI#Nxm&Q=%G-L zKEQ!TAdIFtk?80pY|;|+eJVo1B>c>Af=T$Tk`P<~-~w3w0ss?ixwkZ6f_~bc508SR!oB#j- literal 0 HcmV?d00001