[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/KDE/4.2/kdepim/akregator/src
From:       Frank Osterfeld <frank.osterfeld () kdemail ! net>
Date:       2009-05-14 17:33:22
Message-ID: 1242322402.342900.3540.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 967999 by osterfeld:

Make "copy link location" copy the URL to the primary selection buffer (what gets \
pasted on middle-click) again. Some unspecified spec allegedly says it's wrong, but \
                nobody seems to care, so reenable it for consistency's sake
.
Following a patch by Shlomi Fish

BUG:183022


 M  +3 -1      articleviewer.cpp  


--- branches/KDE/4.2/kdepim/akregator/src/articleviewer.cpp #967998:967999
@@ -304,7 +304,9 @@
     QClipboard *cb = QApplication::clipboard();
     cb->setText(m_url.prettyUrl(), QClipboard::Clipboard);
     // don't set url to selection as it's a no-no according to a fd.o spec
-    //cb->setText(m_url.prettyUrl(), QClipboard::Selection);
+    // which spec? Nobody seems to care (tested Firefox (3.5.10) Konqueror,and KMail \
(4.2.3)), so I re-enable the following line unless someone gives  +    // a good \
reason to remove it again (bug 183022) --Frank +    cb->setText(m_url.prettyUrl(), \
QClipboard::Selection);  }
 
 void ArticleViewer::slotSelectionChanged()


[prev in list] [next in list] [prev in thread] [next in thread]