summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTimothy Pearson <kb9vqf@pearsoncomputing.net>2015-09-21 21:47:11 -0500
committerTimothy Pearson <kb9vqf@pearsoncomputing.net>2015-09-21 21:47:11 -0500
commit70526c5a3e76b42b242499cb57c051696bcee5eb (patch)
treec6edf9baa36d8291c9b6d81b21ab2be0c6297ba6
parent6a0fb5c342285314621a127d307d35ca8057ab5f (diff)
downloadtdelibs-70526c5a.tar.gz
tdelibs-70526c5a.zip
Limit size of notification popups
This works around badly designed applications that abuse the notification subsystem to display large chunks of text
-rw-r--r--tdeui/kpassivepopup.cpp175
1 files changed, 119 insertions, 56 deletions
diff --git a/tdeui/kpassivepopup.cpp b/tdeui/kpassivepopup.cpp
index ba3459b45..3f0cac58e 100644
--- a/tdeui/kpassivepopup.cpp
+++ b/tdeui/kpassivepopup.cpp
@@ -123,40 +123,96 @@ void KPassivePopup::setView( const TQString &caption, const TQString &text,
setView( standardView( caption, text, icon, this ) );
}
-TQVBox * KPassivePopup::standardView( const TQString& caption,
+static void truncateStringToFit(TQString &string, TQFont font, int max_width) {
+ bool truncated = false;
+ TQFontMetrics fm(font);
+ while (fm.width(string) > max_width) {
+ string.truncate(string.length() - 1);
+ truncated = true;
+ }
+ if (truncated) {
+ string += " ...";
+ }
+}
+
+TQVBox * KPassivePopup::standardView(const TQString& caption,
const TQString& text,
const TQPixmap& icon,
- TQWidget *parent )
+ TQWidget *parent)
{
- TQVBox *vb = new TQVBox( parent ? parent : this );
- vb->setSpacing( KDialog::spacingHint() );
-
- TQHBox *hb=0;
- if ( !icon.isNull() ) {
- hb = new TQHBox( vb );
- hb->setMargin( 0 );
- hb->setSpacing( KDialog::spacingHint() );
- ttlIcon = new TQLabel( hb, "title_icon" );
- ttlIcon->setPixmap( icon );
- ttlIcon->setAlignment( AlignLeft );
- }
+ TQString sizedCaption = caption;
+ TQString sizedText = text;
- if ( !caption.isEmpty() ) {
- ttl = new TQLabel( caption, hb ? hb : vb, "title_label" );
- TQFont fnt = ttl->font();
- fnt.setBold( true );
- ttl->setFont( fnt );
- ttl->setAlignment( Qt::AlignHCenter );
- if ( hb )
- hb->setStretchFactor( ttl, 10 ); // enforce centering
- }
+#ifdef Q_WS_X11
+ int max_width;
+
+ NETRootInfo info( tqt_xdisplay(),
+ NET::NumberOfDesktops |
+ NET::CurrentDesktop |
+ NET::WorkArea,
+ -1, false );
+ info.activate();
+ NETRect workArea = info.workArea(info.currentDesktop());
+ max_width = workArea.size.width / 3;
+#endif
- if ( !text.isEmpty() ) {
- msg = new TQLabel( text, vb, "msg_label" );
- msg->setAlignment( AlignLeft );
- }
+ TQVBox *vb = new TQVBox( parent ? parent : this );
+ vb->setSpacing( KDialog::spacingHint() );
+
+ TQHBox *hb=0;
+ if ( !icon.isNull() ) {
+ hb = new TQHBox( vb );
+ hb->setMargin( 0 );
+ hb->setSpacing( KDialog::spacingHint() );
+ ttlIcon = new TQLabel( hb, "title_icon" );
+ ttlIcon->setPixmap( icon );
+ ttlIcon->setAlignment( AlignLeft );
+ }
+
+ if ( !sizedCaption.isEmpty() ) {
+ ttl = new TQLabel( sizedCaption, hb ? hb : vb, "title_label" );
+ TQFont fnt = ttl->font();
+#ifdef Q_WS_X11
+ truncateStringToFit(sizedCaption, fnt, max_width);
+ ttl->setText(sizedCaption);
+#endif
+ fnt.setBold( true );
+ ttl->setFont( fnt );
+ ttl->setAlignment( Qt::AlignHCenter );
+ if ( hb ) {
+ hb->setStretchFactor( ttl, 10 ); // enforce centering
+ }
+ }
+
+ if ( !sizedText.isEmpty() ) {
+ msg = new TQLabel( sizedText, vb, "msg_label" );
+#ifdef Q_WS_X11
+ TQStringList textLines = TQStringList::split("\n", sizedText, true);
+ for (TQStringList::Iterator it = textLines.begin(); it != textLines.end(); ++it) {
+ truncateStringToFit(*it, msg->font(), max_width);
+ }
+
+ // Limit message to 5 lines of text
+ if (textLines.count() > 5) {
+ int count = 3;
+ TQStringList truncatedLines;
+ for (TQStringList::Iterator it = textLines.begin(); it != textLines.end(); ++it) {
+ truncatedLines.append(*it);
+ if (count > 5) {
+ truncatedLines.append("...");
+ break;
+ }
+ count++;
+ }
+ textLines = truncatedLines;
+ }
+ sizedText = textLines.join("\n");
+ msg->setText(sizedText);
+#endif
+ msg->setAlignment( AlignLeft );
+ }
- return vb;
+ return vb;
}
void KPassivePopup::setView( const TQString &caption, const TQString &text )
@@ -194,27 +250,33 @@ void KPassivePopup::mouseReleaseEvent( TQMouseEvent *e )
void KPassivePopup::show()
{
- if ( size() != sizeHint() )
- resize( sizeHint() );
-
- if ( d->fixedPosition.isNull() )
- positionSelf();
- else {
- if( d->popupStyle == Balloon )
- setAnchor( d->fixedPosition );
- else
- move( d->fixedPosition );
- }
- TQFrame::show();
-
- int delay = hideDelay;
- if ( delay < 0 ) {
- delay = DEFAULT_POPUP_TIME;
- }
-
- if ( delay > 0 ) {
- hideTimer->start( delay );
- }
+ TQSize desiredSize = sizeHint();
+
+ if (size() != desiredSize) {
+ resize(desiredSize);
+ }
+
+ if (d->fixedPosition.isNull()) {
+ positionSelf();
+ }
+ else {
+ if( d->popupStyle == Balloon ) {
+ setAnchor(d->fixedPosition);
+ }
+ else {
+ move(d->fixedPosition);
+ }
+ }
+ TQFrame::show();
+
+ int delay = hideDelay;
+ if ( delay < 0 ) {
+ delay = DEFAULT_POPUP_TIME;
+ }
+
+ if ( delay > 0 ) {
+ hideTimer->start( delay );
+ }
}
void KPassivePopup::show(const TQPoint &p)
@@ -346,13 +408,14 @@ void KPassivePopup::setAnchor(const TQPoint &anchor)
void KPassivePopup::paintEvent( TQPaintEvent* pe )
{
- if( d->popupStyle == Balloon )
- {
- TQPainter p;
- p.begin( this );
- p.drawPolygon( d->surround );
- } else
- TQFrame::paintEvent( pe );
+ if( d->popupStyle == Balloon ) {
+ TQPainter p;
+ p.begin( this );
+ p.drawPolygon( d->surround );
+ }
+ else {
+ TQFrame::paintEvent( pe );
+ }
}
void KPassivePopup::updateMask()