Re: Review Request: icon plasmoid - http url naming

2012-04-20 Thread Commit Hook

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103435/#review12735
---


This review has been submitted with commit 
246d9bcc81878977d68c3cc030f1b6fd2e28cb87 by Gregor Tätzner to branch KDE/4.8.

- Commit Hook


On Jan. 10, 2012, 1:41 p.m., Greg T wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/103435/
 ---
 
 (Updated Jan. 10, 2012, 1:41 p.m.)
 
 
 Review request for Plasma.
 
 
 Description
 ---
 
 I added a special case so the desktop icon gets a pretty name when you drag 
 and drop a link onto your desktop. Additional, a regex strips out the 
 protocol (like in firefox) 
 
 I'm allowed to commit it?
 
 
 This addresses bug 199811.
 http://bugs.kde.org/show_bug.cgi?id=199811
 
 
 Diffs
 -
 
   plasma/generic/applets/icon/icon.cpp b25e5d6 
 
 Diff: http://git.reviewboard.kde.org/r/103435/diff/
 
 
 Testing
 ---
 
 looks good
 
 
 Thanks,
 
 Greg T
 


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request: icon plasmoid - http url naming

2012-01-11 Thread Aaron J. Seigo

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103435/#review9741
---

Ship it!


Ship It!

- Aaron J. Seigo


On Jan. 10, 2012, 1:41 p.m., Greg T wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/103435/
 ---
 
 (Updated Jan. 10, 2012, 1:41 p.m.)
 
 
 Review request for Plasma.
 
 
 Description
 ---
 
 I added a special case so the desktop icon gets a pretty name when you drag 
 and drop a link onto your desktop. Additional, a regex strips out the 
 protocol (like in firefox) 
 
 I'm allowed to commit it?
 
 
 This addresses bug 199811.
 http://bugs.kde.org/show_bug.cgi?id=199811
 
 
 Diffs
 -
 
   plasma/generic/applets/icon/icon.cpp b25e5d6 
 
 Diff: http://git.reviewboard.kde.org/r/103435/diff/diff
 
 
 Testing
 ---
 
 looks good
 
 
 Thanks,
 
 Greg T
 


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request: icon plasmoid - http url naming

2012-01-11 Thread Commit Hook

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103435/#review9754
---


This review has been submitted with commit 
a0fd4056bdb115bd74e4e50a0c5b12d2462fa92f by Gregor Tätzner to branch master.

- Commit Hook


On Jan. 10, 2012, 1:41 p.m., Greg T wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/103435/
 ---
 
 (Updated Jan. 10, 2012, 1:41 p.m.)
 
 
 Review request for Plasma.
 
 
 Description
 ---
 
 I added a special case so the desktop icon gets a pretty name when you drag 
 and drop a link onto your desktop. Additional, a regex strips out the 
 protocol (like in firefox) 
 
 I'm allowed to commit it?
 
 
 This addresses bug 199811.
 http://bugs.kde.org/show_bug.cgi?id=199811
 
 
 Diffs
 -
 
   plasma/generic/applets/icon/icon.cpp b25e5d6 
 
 Diff: http://git.reviewboard.kde.org/r/103435/diff/diff
 
 
 Testing
 ---
 
 looks good
 
 
 Thanks,
 
 Greg T
 


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request: icon plasmoid - http url naming

2012-01-10 Thread Greg T

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103435/
---

(Updated Jan. 10, 2012, 1:41 p.m.)


Review request for Plasma.


Changes
---

bump


Description (updated)
---

I added a special case so the desktop icon gets a pretty name when you drag and 
drop a link onto your desktop. Additional, a regex strips out the protocol 
(like in firefox) 

I'm allowed to commit it?


This addresses bug 199811.
http://bugs.kde.org/show_bug.cgi?id=199811


Diffs
-

  plasma/generic/applets/icon/icon.cpp b25e5d6 

Diff: http://git.reviewboard.kde.org/r/103435/diff/diff


Testing
---

looks good


Thanks,

Greg T

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel