Re: Plasma mediacenter - is it still alive?
On 10.02.2011 13:59, Marco Martin wrote: On Wednesday 09 February 2011, Onur-Hayri Bakici wrote: On 08.02.2011 23:40, Marco Martin wrote: On Tuesday 08 February 2011, Marco Martin wrote: I've found the techbase-page http://techbase.kde.org/Projects/Plasma/Plasma_Media_Center and a gitorious projecthttp://gitorious.org/plasma-media-center that hasn't been updated for quite sometime. for everybody that wants to do a gsoc: put it in the wiki NOW as lidya just wrote :p http://community.kde.org/GSoC/2011/Ideas#Plasma I added one on mediacenter: if someone has different ideas, please don't hesitate to add a different one or even modify a lot that one To your idea of PMC in GSoC: It would be also cool to integrate the online media provider services into the GUI as well. I am still planning to enhance my GSoC project from last year :) absolutely! this is another thing i would love seeing it get going. it kinda depends the underlying thing being finally a bit working and stable instead of continuously drifting away Yeah, you're right. It really has been neglected. I'll try to finish it up until the student starts his/her GSoC project. Cheers, Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma mediacenter - is it still alive?
Good news everybody!(tm) this is its new home: https://projects.kde.org/projects/playground/multimedia/plasma-mediacenter Cheers, Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: tokamak five
On Thursday, February 10, 2011 23:35:46 Artur de Souza wrote: Quoting Sebastian Kügler se...@kde.org: Alright. Week 17 works for me, if we keep it at 12 people max, logistics will be a bit easier (we can fit 12 people in the living room hacking, more than that will be slightly uncomfortable, though). It seems I have a must go wedding on the 30th of April, but that shouldn't be a problem if we can start on Monday, instead of Wednesday..what do you think? No problem from my POV. I make a fine easterbunny! -- sebas http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9 ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: tokamak five
Quoting Sebastian Kügler se...@kde.org: No problem from my POV. I make a fine easterbunny! That's for sure :D -- --- http://claimid.com/morpheuz Blog: http://blog.morpheuz.cc PGP: 0xDBEEAAC3 @ wwwkeys.pgp.net --- ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Enabling Apply button in news
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100625/ --- (Updated Feb. 11, 2011, 2:41 p.m.) Review request for Plasma. Changes --- Changes done according to suggestion. Summary --- Enabled Apply button in News configuration dialog. Diffs (updated) - applets/news/news.cpp 731b8b6 Diff: http://git.reviewboard.kde.org/r/100625/diff Testing --- works fine in trunk. Thanks, Sinny ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Enabling Apply button in rssnow
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100624/#review1353 --- Ship it! All god, thanks Sinny! - Anne-Marie On Feb. 10, 2011, 6:21 p.m., Sinny Kumari wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100624/ --- Review request for Plasma. Summary --- Enabled Apply button in rssnow configuration dialog. Diffs - applets/rssnow/news.cpp 84b05cf Diff: http://git.reviewboard.kde.org/r/100624/diff Testing --- Working fine in trunk. Thanks, Sinny ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Enabling Apply button in rssnow
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100624/#review1354 --- Ship it! All good, thanks Sinny! - Anne-Marie On Feb. 10, 2011, 6:21 p.m., Sinny Kumari wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100624/ --- Review request for Plasma. Summary --- Enabled Apply button in rssnow configuration dialog. Diffs - applets/rssnow/news.cpp 84b05cf Diff: http://git.reviewboard.kde.org/r/100624/diff Testing --- Working fine in trunk. Thanks, Sinny ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request: Enabling Apply button in SystemLoadViewer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100631/ --- Review request for Plasma. Summary --- Enabled Apply button in SystemLoadViewer configuration dialog. Diffs - applets/systemloadviewer/systemloadviewer.cpp 685fee7 Diff: http://git.reviewboard.kde.org/r/100631/diff Testing --- works fine in trunk. Thanks, Sinny ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Enabling Apply button in SystemLoadViewer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100631/#review1365 --- Ship it! Thanks for the patch! Please remove the whitespace then commit. applets/systemloadviewer/systemloadviewer.cpp http://git.reviewboard.kde.org/r/100631/#comment1114 Whitespace - Matthew John On Feb. 11, 2011, 5:26 p.m., Sinny Kumari wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100631/ --- (Updated Feb. 11, 2011, 5:26 p.m.) Review request for Plasma. Summary --- Enabled Apply button in SystemLoadViewer configuration dialog. Diffs - applets/systemloadviewer/systemloadviewer.cpp 685fee7 Diff: http://git.reviewboard.kde.org/r/100631/diff Testing --- works fine in trunk. Thanks, Sinny ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Enabling Apply button in SystemLoadViewer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100631/#review1366 --- Ship it! OK from me, tested and it works as expected, good work! - Anne-Marie On Feb. 11, 2011, 5:26 p.m., Sinny Kumari wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100631/ --- (Updated Feb. 11, 2011, 5:26 p.m.) Review request for Plasma. Summary --- Enabled Apply button in SystemLoadViewer configuration dialog. Diffs - applets/systemloadviewer/systemloadviewer.cpp 685fee7 Diff: http://git.reviewboard.kde.org/r/100631/diff Testing --- works fine in trunk. Thanks, Sinny ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Enabling Apply button in SystemLoadViewer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100631/ --- (Updated Feb. 11, 2011, 6:24 p.m.) Review request for Plasma. Changes --- Removed extra whitespace. Summary --- Enabled Apply button in SystemLoadViewer configuration dialog. Diffs (updated) - applets/systemloadviewer/systemloadviewer.cpp 685fee7 Diff: http://git.reviewboard.kde.org/r/100631/diff Testing --- works fine in trunk. Thanks, Sinny ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Tips for commit messages
On Wednesday 09 February 2011 14:00:23 Artur de Souza wrote: Hello! :) Somebody just sent this two websites with short and useful tips for commit messages and I felt I should share with my new git friends on the block :P https://github.com/erlang/otp/wiki/Writing-good-commit-messages http://lbrandy.com/blog/2009/03/writing-better-commit-messages/ And in the same vein: http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html John. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: signal messageButtonPressed(const Plasma::MessageButton button);
On Tuesday, February 8, 2011, Marco Martin wrote: Hi all, I see in Plasma::Applet that void messageButtonPressed(const MessageButton button); was changed in void messageButtonPressed(const Plasma::MessageButton button); that sucks; specifying Plasma:: in the signature is absolutely required for it to be expected to work consistently. the only way for non-namespaced params to work in signals like this is if plasmoid writers include a using Plasma statement in their code which does a lot to invalidate the point of having a namespace in the first place. i suppose we can move back in this case, but it's really, really, really something we need to make consistent in libplasma2 as it is a source of ongoing annoyances. -- 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 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
SNI Additions Summary
Okay, I think we're done discussing, so let me try and summarize to make sure I understand (and everyone can correct me if not :) * Adding a field X-KDE-StatusNotifierItems to track the IDs of items in the desktop files was agreed upon. Other solutions were talked about, but this one seemed as good and relatively simple. * Adding a property for the accessible labels for the icons is agreed upon but the spec should specifically state that if it is not available the title of the tooltip should be used in its place. Also, the narrative text in the spec should talk about the usage and differentiation between the two values. * KDE is not interested in having the *ShowIn fields to allow for app developers to target specific visualizations. We should implement this in the XCanonical namespace and mention that it won't work on all implementations. If that's good I'll propose a diff to the spec for those fields. Thanks everyone for the discussion and ideas! --Ted 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
Featurlets for 4.7
Hi all, As i can see the fixing of apply buttons is proceeding very, very well, that's awesome. props to everyone involved :) another idea aamong those lines, could be, think about a little feature you would see in 4.7 could be a new option, could be a very simple plamoid, a service or two in the paste widget, a new feature or two that are missing in the microblog plasmoid in its protocol handling (hint, i would really love to see that) in this thread talk about ideas, anything comes to your mind will do, there is already a wiki page for it http://community.kde.org/Plasma/Tasks with a table in the end to say who is doing what, to better help coordinate efforts Let ideas begin :) Cheers, Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: KHoliday::HolidayRegionSelector: emit signal when modified
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100619/ --- (Updated Feb. 12, 2011, 4:40 a.m.) Review request for KDEPIM-Libraries and Plasma. Changes --- Redefining signal according to suggestion. Summary --- Makes HolidayRegionSelector emit a signal when something is modified in the widget. Needed for enabling Apply button on plasmaclock config [http://git.reviewboard.kde.org/r/100620/]. Diffs (updated) - kholidays/holidayregionselector.h a48823e kholidays/holidayregionselector.cpp 6115b2f Diff: http://git.reviewboard.kde.org/r/100619/diff Testing --- Works fine so far, although I'm not sure if the naming is OK. Thanks, Romário ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: KHoliday::HolidayRegionSelector: emit signal when modified
On Feb. 10, 2011, 4:53 p.m., John Layt wrote: Good idea, but not quite the right approach. I want to keep the internal implementation of the widget completely hidden from any clients, allowing it to be completely changed in the future if needs be, so exposing the QTreeWidgetItem signal is wrong for that, as would be exposing teh int version too (after all the client has no idea what those mean). Instead, simply add a new public slot for itemChanged() without any parms, then anyone connecting to it can use the proper api to find out what the new selection is. John Layt wrote: Actually, make that selectionChanged() instead, more generic name. John Layt wrote: And by slot I mean signal :-) Need coffee... Done. Check it out please. - Romário --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100619/#review1342 --- On Feb. 12, 2011, 4:40 a.m., Romário Rios wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100619/ --- (Updated Feb. 12, 2011, 4:40 a.m.) Review request for KDEPIM-Libraries and Plasma. Summary --- Makes HolidayRegionSelector emit a signal when something is modified in the widget. Needed for enabling Apply button on plasmaclock config [http://git.reviewboard.kde.org/r/100620/]. Diffs - kholidays/holidayregionselector.h a48823e kholidays/holidayregionselector.cpp 6115b2f Diff: http://git.reviewboard.kde.org/r/100619/diff Testing --- Works fine so far, although I'm not sure if the naming is OK. Thanks, Romário ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Enable Apply button for plasma clock
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100620/ --- (Updated Feb. 12, 2011, 4:42 a.m.) Review request for Plasma. Changes --- Just updating to new signal name. Summary --- Enables apply button for plasma clock. Dependens on patch http://git.reviewboard.kde.org/r/100619/. Needed for http://git.reviewboard.kde.org/r/100614/. Diffs (updated) - libs/plasmaclock/calendartable.cpp f15f1ff libs/plasmaclock/clockapplet.cpp b806792 Diff: http://git.reviewboard.kde.org/r/100620/diff Testing --- Seems to be fine. Thanks, Romário ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
