On May 22, 2010, Alessandro Diaferia wrote:
> > hmm. no comment on whether it's a feature..  code looks sensible,
> > although wouldn't it be more future-proof to iterate over data->urls()
> > instead of only
> > taking the first?
> 
> you're just right.. so here comes the updated patch :)

looks good with those changes, and a nice usability improvement. please 
commit. 

btw, one additional nice feature to have there would be to make it possible to 
just drag it onto the favourites tab.. but that can wait :)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks

Attachment: signature.asc
Description: This is a digitally signed message part.

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

Reply via email to