GSoC aspirant
Hi, I recently joined the plasma mailing list. I'm relatively new to contributing KDE (well, FOSS actually). I'm interested in taking up the GSoC project QMLify the Plasma Widgets. I'm an undergraduate student from India. I have been a KDE fan since the KDE 4.4 (mostly because of the plasma workspace) and have been playing around with it since. I got in touch with Qt a few months back and developed a number of (little) standalone apps (using PyQt). I've been going through QML since a month now, and hoping to get over with most of it after the conf.kde.in. I got the trunk up and running to get myself acquainted with development process with KDE, and submitted a few patches to KDE games and KIPI plugins. QMLifying the plasma widgets is really a great idea that I would love to implement. I plan to get my proposal ready by this time next week. But first, I would love to hear from the plasma team about their views on the idea. I really look forward to get involved with KDE (particularly, plasma) through this GSoC project. Happy hacking, Viranch PS: Apologies for such a long email :) ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC aspirant
Hi Artur, I would recommend that in your proposal you try to be as much clear as possible regarding the objectives like: which plasmoids you plan to QMLify and how you plan that. Sure, I plan to QMLify the widgets that come packaged with KDE by default (the plasmoids). I'll list them specifically. I'm saying this because some plasmoids make use of plasma stuff that (still) are not exported to QML and your work would probably include this (with our help of course). You mean I would also have to change relevant stuff in plasma that I need for QMLifying the plasmoids? Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
GSoC: QMLify Plasmoids
Hi, I know it has been a long while I haven't turned up with any work. I'm kind of caught up with my semester very badly (not an excuse, I know). Anyhow, I had some very basic questions regarding the project implementation. 1. I have learned that QML support will be/is being written in libplasma2. So, does THIS project include doing that work? If no (because I saw it listed as another GSoC project), will I have to wait for the QML support to be implemented until I can start with my project? Also, it would SEEM to be a too small scale GSoC project if it only has transforming 10-15 plasmoids (of course, it won't be easy). If yes, what percentage of the project time might approximately go into doing it? I was stuck at this thought while preparing the project proposal. Regards, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC: QMLify Plasmoids
On Mon, Feb 28, 2011 at 8:27 PM, Sebastian Kügler se...@kde.org wrote: Sure, but you'll have to be more specific. Which one exactly are you talking about? The ones residing in trunk/playground/base/plasma/declarative-applets/replacements/, say rssnow. A friend of mine suggested me to soft link the appropriate plasmoid path in kde-workspace/plasma with the relevant plasmoid directory in playground and complie. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC: QMLify Plasmoids
On Mon, Feb 28, 2011 at 8:32 PM, Marco Martin notm...@gmail.com wrote: Don't underestimate that. QML is powerful and makes you more efficient, but it still takes time to get things right (and not only barely working). and, there is still quite a bit of infrastructure missing, (example: drag and drop) Seems heavy now :P Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC: QMLify Plasmoids
Hi Shantanu, As Marco said, QML is pretty cool, but it takes very careful planning and getting used to. So it might not be as simple as it seems to be ;) I have experienced this while working on Calligra Mobile, so thought will share with you. Cheers, Thanks for the thought there. :) Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
GSoC proposal: First draft
Hi, Attached is the first draft of my GSoC proposal for the project: QMLify the KDE Plasma Widgets. Please review it and send me comments/suggestions. Regards, Viranch GSoC Project Proposal = Name: Viranch Mehta Email: viranch.me...@gmail.com IRC nick: viranch Location: Gujarat, India. GMT +0530 ::QMLify the KDE Plasmoids:: Motivation: == Plasma Workspace has been what defines the look and feel of KDE4. It is very well designed to accommodate various panels, widgets, etc. on the desktop and has seen many changes since KDE 4.1 in the plasma design and architecture. Its developer friendly code-base allows various programmers to easily create custom plasmoids which can be distributed as a small bundle and be easily integrated into the plasma workspace. With introduction of Qt Quick/QML in Qt 4.7, the gap between the developers and UI designers got narrowed down with its rich set of QML elements and intuitive declarative QML language. QML allows designers to create fluid like animated UIs very quickly. And most importantly, the animations happen without the slightest glitch. The Javascript/C++ bundle efficiently handles all the back-end implementation, leaving the designers to concentrate on the animations and UI simplicity. This projects aims at transforming all the current plasma widgets (Plasmoids) to QML/C++, opening the doors to lots of better animations with the plasmoids and probably faster loading at start up. Most of the plasmoids that come bundled with KDE and the default ones will be completely transformed to QML/C++. My enthusiasm towards KDE initially led me to learning Qt (which I grasped very quickly) and made a lot of small applications with it[1] for some experience. I immediately developed interest with the UI designing and developing and that's how I landed up writing a proposal for this project. I have already been a fan of the KDE Plasma amongst other KDE features, and this project is the bull's eye towards my interest. I see a perfect opportunity to develop some real life desktop user interface with the latest technology of QtQuick through this project. Implementation plan: === KDE already has support for QML plasmoids since KDE 4.6. So moving one step forward, the following plasmoids will be transformed to either QML/Javascript or QML/C++ in this project: - Kickoff Application Launcher - Battery Monitor - Device Notifier - Digital Clock - Folder View - Lock/Logout - Network Management - System Tray If time permits, some other plasmoids may also be worked upon. Tentative Timeline: == May beginning – Read plasma code and get familiar with the current plasma/plasmoids code and the plasma library. Probably write one or two practice plasmoids to get a first hand feel of how they're made. May 15 onwards – Start with the actual re-factoring of the plasmoids into QML/C++ or QML/Javascript (whichever is most suitable). I plan to invest one-two weeks with the first two-three plasmoids (May 15 – June 31) and then the subsequent plasmoids may be covered within a week each (July 1 – August 10), hopefully reducing the time to less than a week with each plasmoid that I finish up with. Each plasmoid will be tested as and when it undergoes the re-factoring. Later – Fix bugs with the re-factored plasmoids and re-factor as many more plasmoids as possible. About Me: I'm a third year engineering student, pursuing B.Tech. in Information and Communication Technology (ICT) from DA-IICT, Gandhinagar. I have been a Linux and KDE user since two years now and have developed a great interest in contributing to the FOSS since then. KDE is the project I've been most familiar to and interested in, as I see most of cutting edge technologies wrapped up in KDE, and of course, I love the awesome community commitment and dedication towards the same. I've been following few mailing lists and the development in trunk since half an year. I had submitted a patch to KDE games[3] and another to KIPI plugins[4] to get a hold of KDE code-base and the development process of KDE. Besides being proficient with C/C++/Python programming languages, I have worked upon several of my PyQt projects[1] like MP3 cutter/joiner, PDF splitter/joiner, etc. The project SAM[2] (Syberoam Account Manager) has been the most worked upon by me. It is a cyberoam account manager made using PyQt where users at colleges/workplaces that have cyberam firewall installed can use SAM *to automatically re-login after certain amount of time, *to manage multiple accounts and quota limits. I have also been extensively interested in python scripting, especially the ones relating to web crawling. Besides coding, I love eating, entertainment and exploring new open source technologies. I'll soon be adding experimental plasma codes to the Links section. Links: = [1] http://www.bitbucket.org/viranch
Re: GSoC proposal: First draft
On Mon, Mar 14, 2011 at 3:28 PM, Sebastian Kügler se...@kde.org wrote: On Friday, March 11, 2011 19:28:13 todd rme wrote: Might it be good to start with the ones that are already in-progress, that way you can directly compare the the existing plasmoid code and the QML code and thus, hopefully, get a feel for how to do the porting? But the ones which are in progress are possibly the ones which get done, anyway. I don't think it's a good idea to put a GSoC student onto something people are already working on. I think I can look through some on-going work and possibly write some patches for getting familiar. Another question: All the QML plasmoids are currently in playground, right? Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC proposal: First draft
On Tue, Mar 15, 2011 at 1:52 AM, Hayri Bakici theha...@gmail.com wrote: I think it's pretty good. You pointed out the reasons of your doings, which imo are really important. Last year, I wrote two use cases in my proposal, however I don't know if something like that really fits in yours. Well, the plasmoids are already existent, so use cases are pretty much defined and well-known by user. So I don't think that's required here. Thanks for the comments, btw :) Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC proposal: First draft
On Fri, Mar 11, 2011 at 1:26 PM, Aaron J. Seigo ase...@kde.org wrote: Marco: is there a list somewhere of the QML porting status of plasmoids? if not, let's start on on community.kde.org. This'd be a good idea! Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC proposal: First draft
On Wed, Mar 16, 2011 at 12:11 AM, Marco Martin notm...@gmail.com wrote:
On Tue, Mar 15, 2011 at 6:17 PM, Viranch Mehta viranch.me...@gmail.com
wrote:
On Fri, Mar 11, 2011 at 1:26 PM, Aaron J. Seigo ase...@kde.org wrote:
Marco: is there a list somewhere of the QML porting status of plasmoids?
if
not, let's start on on community.kde.org.
yes, it's a great idea and no, there still doesn is a page for it, if
you will do one.. free hugs :p
I scored one: http://techbase.kde.org/QML_Plasmoids ! :P
I've listed plasmoids that are in
kde-workspace/plasma/{desktop,generic}/applets. I guess those are the
primary ones.
Marco: Please update as soon as possible.
Cheers,
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC proposal: First draft
On Thu, Mar 17, 2011 at 6:56 PM, Marco Martin notm...@gmail.com wrote: i'll mark the ones that are started, and should really really start porting them over git, meh while people start to do the ports, i think the branches where they live should be marked here near the plasmoid name Please get the repository ready as soon as possible, so that I can start submitting few patches to the QML plasmoids to get familiar. Or can I go ahead with SVN playground for now? Thanks, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
GSoC QMLify plasmoids proposal
Hi, Please review my proposal for GSoC project QMLify Plasmoids: https://docs.google.com/document/d/19zZn2ObkGWFRVOjaaVu1qp54Olb7g50FN6RgyAQhFm0/edit?hl=enauthkey=CKPlh6oP I've specified the order of plasmoids that I think would be appropriate in increasing order of complexity. Thanks, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC QMLify plasmoids proposal
On Mon, Mar 28, 2011 at 5:03 PM, Marco Martin notm...@gmail.com wrote: Hi, the proposal is good, it should be a bit more elaborate tough. the qmlfy term was used just for fun, however the official proposal should have a bit more serious name, like QML port of workspace and addons plasmoids My primary target is the default plasmoids, and after that if there is time, I'll go ahead with the addons. So I have not mentioned those in the proposal. I hope that's fine. please also state that you will work together whoever will write the plasma qtcomponents set (hope there will be a separate gsoc for that, could end up be necessary you to work also on them...) If qtcomponents gsoc doesn't get in (and mine does), I'll not be able to port as many plasmoids as planned. Is that considered ok? Regards, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC QMLify plasmoids proposal
Hi Sebas, On Tue, Mar 29, 2011 at 3:20 PM, Sebastian Kügler se...@kde.org wrote: I thin the focus should be on: - Port Plasmoids visual appearance and interaction to QML and JavaScript This tempts me to mention about QGraphicsView (so that it is specifically clear that from what the plasmoids are going to be ported). Is it ok to mention it or is it too technical to mention in an application? I hope my feedback is useful for your application. It is :) Thanks. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC QMLify plasmoids proposal
Hi Marco, On Mon, Mar 28, 2011 at 5:03 PM, Marco Martin notm...@gmail.com wrote: please also state that you will work together whoever will write the plasma qtcomponents set (hope there will be a separate gsoc for that, could end up be necessary you to work also on them...) Could you please give me some idea of the project as to what exactly are widgets that are finally expected out of it? And if possible, some examples already under progress? Thanks, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC QMLify plasmoids proposal
Hi, I have modified my proposal according to the feedbacks. Please review the final version: https://docs.google.com/document/d/19zZn2ObkGWFRVOjaaVu1qp54Olb7g50FN6RgyAQhFm0/edit?hl=enauthkey=CKPlh6oP Thanks, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC QMLify plasmoids proposal
Hi, On Thu, Mar 31, 2011 at 6:05 PM, Artur de Souza aso...@kde.org wrote: Quoting Marco Martin notm...@gmail.com: to me the proposal seems quite good now +1. I would only change to make kickoff one of the last ones. I didn't touch kickoff's code, but maybe it's a little complex and it's better to finish others that are simpler first? Done. BTW, who is going to mentor the project? Anything decided? Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: GSoC QMLify plasmoids proposal
Hi, On Thu, Mar 31, 2011 at 11:51 PM, todd rme toddrme2...@gmail.com wrote: kickoff? I personally would like having a favorites or places widget. Oh that's a good idea. I've actually wanted to have places plasmoid since long. But I want to first complete the default plasmoids in this GSoC, and then move on to addons if time permits. Cheers, Viranch PS: I'm finally submitting the application :) ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
GSoC: QML Plasmoids Proposal
Hi, I've put up my proposal for GSoC Project QML Port of KDE Plasmoids at melange since few days now. Thought I'd just share the link :) http://www.google-melange.com/gsoc/proposal/review/google/gsoc2011/viranch/1 Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Got into GSoC!
Hi, As all of you might have known by now, my proposal (QML port of Plasmoids) just got accepted into GSoC. I want to thank all the community members of Plasma and KDE for giving me the opportunity :) Hope to make the plasma and KDE a better piece of software! Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Got into GSoC!
Hi, On Wed, May 4, 2011 at 1:26 AM, Marco Martin notm...@gmail.com wrote: we should meet on irc either later this week or next week for a kickstart discussion I'm sorry I was busy with college evaluations last week so could not meet up. Now I'll mostly be free from 1230 hrs to 1830 hrs UTC. Currently I'm in temporary trouble of constant internet access, so please let me know your time of convenience one day before. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Got into GSoC!
On Mon, May 9, 2011 at 5:06 AM, Marco Martin notm...@gmail.com wrote: what about tomorrow ~12:30 utc? Sure. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Power management dataengine: Odd compile error
Hi,
I ran into some odd compile problems while trying to compile the
powermanagement dataengine after I wrote a service in it:
CMakeFiles/plasma_engine_powermanagement.dir/powermanagementengine.o: In
function `factory':
*
$src/kde-workspace/plasma/generic/dataengines/powermanagement/powermanagementengine.h
*:71: multiple definition of `factory::factory(char const*, char const*,
QObject*)'
CMakeFiles/plasma_engine_powermanagement.dir/plasma_engine_powermanagement_automoc.o:
*
$build/kde-workspace/plasma/generic/dataengines/powermanagement/../../../../../../src/kde-workspace/plasma/generic/dataengines/powermanagement/powermanagementengine.h
*:71: first defined here
$src and $build are in ~/kde/{src,build}
The same error (multiple definition) occurs for all functions of factory
class, and for qt_plugin_query_verification_data, qt_plugin_instance,
kde_plugin_verification_data, kde_plugin_version.
And all these errors are reported only at *
kde-workspace/plasma/generic/dataengines/powermanagement/powermanagementengine.h:71
*, which is *K_EXPORT_PLASMA_DATAENGINE(powermanagement,
PowermanagementEngine)*. Also, the two files in the error refer to one same
file only.
Anyone aware of what this means?
Cheers,
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Power management dataengine: Odd compile error
Hey!
Aaron's suggestion worked :)
But I don't understand, it compiled fine earlier. After I added the service
and the job (not touching the engine.{h,cpp} at all), the compilation broke!
Cheers,
Viranch
On Mon, May 30, 2011 at 12:56 AM, Marco Martin notm...@gmail.com wrote:
On Sun, May 29, 2011 at 2:02 PM, Viranch Mehta viranch.me...@gmail.com
wrote:
Hi,
I ran into some odd compile problems while trying to compile the
powermanagement dataengine after I wrote a service in it:
CMakeFiles/plasma_engine_powermanagement.dir/powermanagementengine.o: In
function `factory':
$src/kde-workspace/plasma/generic/dataengines/powermanagement/powermanagementengine.h:71:
multiple definition of `factory::factory(char const*, char const*,
QObject*)'
CMakeFiles/plasma_engine_powermanagement.dir/plasma_engine_powermanagement_automoc.o:$build/kde-workspace/plasma/generic/dataengines/powermanagement/../../../../../../src/kde-workspace/plasma/generic/dataengines/powermanagement/powermanagementengine.h:71:
first defined here
$src and $build are in ~/kde/{src,build}
The same error (multiple definition) occurs for all functions of
factory
class, and for qt_plugin_query_verification_data, qt_plugin_instance,
kde_plugin_verification_data, kde_plugin_version.
And all these errors are reported only at
kde-workspace/plasma/generic/dataengines/powermanagement/powermanagementengine.h:71,
which is K_EXPORT_PLASMA_DATAENGINE(powermanagement,
PowermanagementEngine). Also, the two files in the error refer to one
same
file only.
Anyone aware of what this means?
Seeing how you did modify the headers could help ;)
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
Close button in task manager widget
Hey all, This though just popped up in my mind that I though is worth a post in the list. I was thinking it would be nice to have a close button (like the ones for plasmoids in the small popup-sidebar) for each entry in the task manager widget in the panel. So user could easily close a window directly without having to right-click. Comments :) Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Calling dataengine service operation from QML plasmoid
Hi there, I want to call a the service operation setBrightness available in powermanagement dataengine from my qml plasmoid. The service associated with all the sources. How can I go about doing that? dataSource.data[PowerDevil][setBrightness][70] ? Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
QMap in QML
Hi, I have a combobox to which I need to add the items. The dataengine gives me a QMapQString,QString, from which I want to add the keys to the combobox. I'm not sure how well QML recognizes a QMap and lets me use QMap::keys() method. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QMap in QML
On Sat, Jun 4, 2011 at 1:19 PM, Marco Martin notm...@gmail.com wrote:
try to do
for (item in map) {
print(item)
print(map[item])
}
Well actually, i'm getting the data
( dataSource.data[PowerDevil][Available profiles] ) inside
Component.onCompleted and then running the for loop on it. But the execution
stops saying:
TypeError: Result of expression 'dataSource.data[PowerDevil]' [undefined]
is not an object.
So it never goes into the for loop (I checked this by putting normal prints
in the for loop). The above error occurs at all places in the plasmoid
wherever I call the engine, but still gives the data. Here, it doesn't.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QMap in QML
Actually, I'm not sure if its a QMap-in-QML problem, or the [undefined] object problem. But the thing is the execution never enters the for loop. Also, the QMapQString,QString is typedef-ed to StringStringMap in the engine, if that matters. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QMap in QML
On Sat, Jun 4, 2011 at 7:54 PM, Aaron J. Seigo ase...@kde.org wrote: this is a problem in the JS bindings. i've fixed it in master, which will become 4.7.0 ... i woul dlike to find a better fix, however.. Is it only StringStringMap specific? Because, we'll be seeing new dataengines coming up and present ones changing in future, and there may be other datatypes begin used as well, like QHash, QPair, etc. So wouldn't we need a generic fix? Unless, we restrict the dataengines to use only the basic datatypes. Also, I could not build kde-runtime after the StringStringMap fix, says: ‘qt_metatype_id’ is not a member of ‘QMetaTypeIdPlasma::Service*’ kde-runtime/plasma/scriptengines/javascript/simplebindings/dataengine.cpp:66:104: instantiated from here and kde-runtime/plasma/scriptengines/javascript/simplebindings/dataengine.cpp:68:113: instantiated from here Thanks, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Still cannot access QMap in QML
Hi, I am still unable to access QMapQString,QString returned by dataengine in QML. The piece of JS goes like: print(hello); print(dataEngine.data[PowerDevil][Available profiles]); print(world); And it prints only hello, nothing else. I went through the code of JS bindings in runtime, tried playing around with it without luck. I'm totally clueless on this now. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Still cannot access QMap in QML
On Thu, Jun 9, 2011 at 2:58 PM, Aaron J. Seigo ase...@kde.org wrote: On Wednesday, June 8, 2011 23:20:02 Viranch Mehta wrote: I am still unable to access QMapQString,QString returned by dataengine in and you are using kde-runtime from master? Yes, I pulled and compiled it after you fixed this. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Still cannot access QMap in QML
On Thu, Jun 9, 2011 at 3:40 PM, Aaron J. Seigo ase...@kde.org wrote:
On Wednesday, June 8, 2011 23:20:02 Viranch Mehta wrote:
print(dataEngine.data[PowerDevil][Available profiles]);
wait .. what is dataEngine? is it the actual dataEngine? or? it's
entirely
impossible to tell what this code is actually doing without a bit more
context.
Here's the dataEngine:
PlasmaCore.DataSource {
id: dataEngine
engine: powermanagement
connectedSources: [AC Adapter, Battery, Battery 0, PowerDevil]
interval: 0
}
Everything else works fine. I'm able to get the battery charge info, whether
its plugged in to AC source or not, etc.
Cheers,
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Still cannot access QMap in QML
Here's the plasmoidviewer's output: http://paste.kde.org/80143/ (the
pmSource in the output=dataEngine)
http://paste.kde.org/80143/Though it says dataEngine.data[AC Adapter],
dataEngine.data[Battery0], dataEngine.data[PowerDevil] are [undefined],
I'm able to access the data associated with the sources.
On Thu, Jun 9, 2011 at 3:50 PM, Viranch Mehta viranch.me...@gmail.comwrote:
On Thu, Jun 9, 2011 at 3:40 PM, Aaron J. Seigo ase...@kde.org wrote:
On Wednesday, June 8, 2011 23:20:02 Viranch Mehta wrote:
print(dataEngine.data[PowerDevil][Available profiles]);
wait .. what is dataEngine? is it the actual dataEngine? or? it's
entirely
impossible to tell what this code is actually doing without a bit more
context.
Here's the dataEngine:
PlasmaCore.DataSource {
id: dataEngine
engine: powermanagement
connectedSources: [AC Adapter, Battery, Battery 0, PowerDevil]
interval: 0
}
Everything else works fine. I'm able to get the battery charge info,
whether its plugged in to AC source or not, etc.
Cheers,
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: DataEngine compilation problems
Remove line 18 and add datarequest.cpp to line 17. So it would look like this: set(obsdataengine_engine_SRCS obsdataengine.cpp datarequest.cpp) Hope this helps. Cheers, Viranch On Fri, Jun 10, 2011 at 11:11 PM, Saurabh Sood saurabhsoo...@gmail.comwrote: Hi all, I am trying to write a dataengine, that makes API calls to the Open Build Service (OBS) and displays the results in plasmoids. I have written part of the dataengine, but am facing problems in compiling it. I am getting undefined reference errors when I try to create an object of the datarequest subclass. Link to Code: https://gitorious.org/build-service-plasma-widgets Error List: http://pastebin.com/hdChx9zz Have I missed something in my implementation? Please guide me on tackling these errors. Hoping to hear soon, Saurabh Sood ___ 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
Notify of hovering over icon widget
Hi, I've reached a use case where I need a signal to be generated in QML when an IconWidget is hovered. I don't see one implemented. I was thinking of doing the same in kdelibs and was looking for feedback. Also, whether I should do it in master or elsewhere. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Notify of hovering over icon widget
On Wed, Jun 22, 2011 at 1:00 PM, Aaron J. Seigo ase...@kde.org wrote: can you use the onEnter/onExit properties with a MouseArea? Well, I want the background of the IconWidget to be painted when hovered (by doing drawBackgournd: true). But the problem with putting MouseArea on the IconWidget is, the IconWidget does not detect hovers and doesn't paint the background. While if the MouseArea is below the IconWidget, I can't get to do other stuff on hover. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Notify of hovering over icon widget
On Wed, Jun 22, 2011 at 2:57 PM, Marco Martin notm...@gmail.com wrote: avoid to use IconWidgets, use a QIconItem from extracomponents or even an Image with an icon://iconname source. Does that let me do the drawBackground stuff? then all the mouse input you do is with a MouseArea, that has all the events you need. 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: QML model with image and text per element
On Mon, Jun 27, 2011 at 12:16 AM, Shaun Reich predator...@gmail.com wrote:
Okay, so if the data is laid out as you said, in the last example..how
would I display that in the qml view? Since I need to display *both*
the icon and the user's name in the model. Give me something concrete,
please -- as you well know, the docs suck ;-)
With the method you prefer, you can have something like:
model: mainEngineSource.keysForSource(userList)
Component {
id: delegateItem
Item {
Image { source:
mainEngineSource.data[userList][modelData][image] }
Text { text: mainEngineSource.data[userList][modelData][name] }
}
and assign delegate: delegateItem in the view.
Hope this helps.
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Opinion on config dialog for QML plasmoids
Hi, Currently the only way QML applets can have customized config dialog is to put the config.ui file in the main.qml directory which is loaded when settings are opened. We have reached a use case where we need a number of settings that are categorized into tabs on left side of the config dialog (like the one in device notifier, which loads the KCM module from the system settings). This would have to be implemented in the bindings, but not too sure if it should be done due to the uncertainty of this particular kind of use case. It would be nice to have people's opinion on whether it is a desired use case to implement it. Also whether to support the KCM modules too inside the config dialog. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Switching to different widgets on each desktop resets the plasma-desktop
Hi, This is an issue I just came across. Here's the scenario: I have a set of plasmoids on my plasma desktop that is shown on every virtual desktop. Now I decide I want to use different set of plasmoids on each virtual desktop. So I go to system settings and change the relevant setting. This suddenly scraps away all my current plasmoids and gives me empty desktops. This is probably irritating to a user, he might have useful todo's and notes on his desktop, which he loses on changing this setting without any warning. This can be taken care of: ask the user on which desktop he wants the current set of plasmoids to be (or he wants to throw them away) before changing this setting. Until this is done, we can have a dialog, warning the user of losing all his current plasmoids. Comments? Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Switching to different widgets on each desktop resets the plasma-desktop
On Thu, Aug 11, 2011 at 1:13 PM, Aaron J. Seigo ase...@kde.org wrote: On Thursday, August 11, 2011 06:37:54 Viranch Mehta wrote: I go to system settings and change the relevant setting. This suddenly scraps away all my current plasmoids and gives me empty desktops. This is the widgets should still be there, but likely put into a different activity. what you describe is a one-time issue at configuration, and while not the perfect experience it's not something i plan to put any more effort into improving as we have far too many other things on our plate. Do we have any page on techbase (or somewhere else?) where such minor issues are listed? That would be helpful so that we can tackle these as and when possible. Cheers, Viranch patches are welcome, however. i will not accept patches that just add a warning dialog, however. any patches made should be real fixes (e.g. making sure the existing containment gets used in the set of desktop containments) -- 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 ___ 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: Opinion on config dialog for QML plasmoids
On Wed, Aug 17, 2011 at 3:30 PM, Marco Martin notm...@gmail.com wrote: at the moment the only thing stopping the battery and the device notifier to me merged in master is the system kcms missing. Let me point out here that the battery still somehow lacks the ability to recognize the QStringStringMap power profiles returned by the dataengine. I'm not sure how this can be fixed with the JS bindings, otherwise I would have done it. Aaron did commit some patch for this, but did not seem to work in the plasmoid. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Opinion on config dialog for QML plasmoids
On Thu, Aug 18, 2011 at 2:09 AM, Marco Martin notm...@gmail.com wrote:
i tought i explained it: a QMapQString, QString will never work.
you need to change it to QMapQString, QVariant, as easy as that
That doesn't work either (I had tried it when it was made QVariantMap in the
engine). It works if I access in the following manner:
Text {
text: dataSource.data[PowerDevil][Available profiles][Performance]
}
But not in the case:
Component.onCompleted: {
var profiles = dataSource.data[PowerDevil][Available profiles];
// do stuff with the map, eg iterate
}
The latter simply stops at the first line inside the onCompleted saying,
dataSource.data[PowerDevil] [undefined] is not an object, and just
wouldn't go any further. The former cannot be used because we wouldn't know
names of all the profiles beforehand. Hence we need to iterate over the keys
and get the values from the map, which, as I said, is not working even with
QVariantMap.
Viranch
--
Marco Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Opinion on config dialog for QML plasmoids
On Thu, Aug 18, 2011 at 5:13 PM, Viranch Mehta viranch.me...@gmail.comwrote:
Component.onCompleted: {
var profiles = dataSource.data[PowerDevil][Available profiles];
// do stuff with the map, eg iterate
}
Fixed. The above code (for getting the map and iterating) had to be put
inside MouseArea.onClicked (which is for showing the popup dialog of battery
applet). For some reason, it worked there, but wouldn't work inside any
Component.onCompleted. So everytime it is clicked, the map gets populated,
clear the combo box first and then add each item in the combo box. This
seems kind of a dirty work around, but works for now.
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: merging declarative plasmoids
Hi, First of all, I apologize for my prolonged absence. But I'd like to give some relevant updates from my side. I'm currently working on the device notifier implementing the mount/unmount messages (safe to remove now, mount failed, unmount failed, etc). It will be mostly done in a day or two. I think that is the main thing missing with the applet, so it can be merged after I'm done with this (i would merge it myself but I'm a little inexperienced with it). I'll post here after this is got over with. Also, I would like to make a note here that battery monitor and lock/logout BOTH are in plasma/viranch/powermanagementservices branch. So if it is merged, both the applets will come into the master (Sorry for this as I was unfamiliar with git branching earlier). The device notifier is in plasma/viranch/devicenotifier and task manager in plasma/viranch/tasks as would be expected. Cheers, Viranch On Tue, Sep 13, 2011 at 6:12 PM, Marco Martin notm...@gmail.com wrote: On Tue, Sep 13, 2011 at 2:23 PM, Giorgos Tsiapaliwkas terie...@gmail.com wrote: On 13 September 2011 02:19, Aaron J. Seigo ase...@kde.org wrote: On Sunday, September 11, 2011 13:54:07 Marco Martin wrote: the device notifier and the battery one. if, and only if, there are no functional regressions. Does someone has an answer for this one?Otherwise i will start testing the QMLified applets. modulo che config dialogs, the battery applet should be a 100% functional copy, has to be tested if the combobox was finally fixed and if all options even the hidden ones are available. the devicenotifier should also be near completition, not sure about the last item open by default, has to be tested in any case, in order to have 100% functional match, should be less than a day of work. 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: merging declarative plasmoids
On Wed, Sep 14, 2011 at 2:43 PM, Marco Martin notm...@gmail.com wrote: On Wednesday 14 September 2011, Aaron J. Seigo wrote: On Tuesday, September 13, 2011 20:39:20 Viranch Mehta wrote: Also, I would like to make a note here that battery monitor and lock/logout speaking of lock/logout and applets which are essentially rows of icons ... has someone worked out the issues around QML widgets not have size hints and appropriate sizing of applets like lock/logout in panels in plasma-desktop? yep, the slc applet should have correct sizing now. unfortunately it requires some manual calculation from the qml side since is not something offered stock, but if the root element offers minimumWidth and minimumHeight the applet will use them as minimum size hints. How about using plasmoid.resize(w,h)? in plasma2, in the case of a scene that doesn't have qgraphicswidgets in the middle, sizing of things should be a bit easier and a tad more automatic (just remains pretty hard do to expanding things like the taskbar) Viranch, that should be done for the lock/logout applet as well 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: merging declarative plasmoids
Hi, I have incorporated the device notifications in the device notifier applet now (though not fully functional). I have also worked on fixing sizing issues in lock/logout applet, without significant luck. I guess they can be merged now and further fixes can be done after the merge. Cheers, Viranch On Thu, Sep 15, 2011 at 3:45 PM, Viranch Mehta viranch.me...@gmail.comwrote: On Wed, Sep 14, 2011 at 2:43 PM, Marco Martin notm...@gmail.com wrote: On Wednesday 14 September 2011, Aaron J. Seigo wrote: On Tuesday, September 13, 2011 20:39:20 Viranch Mehta wrote: Also, I would like to make a note here that battery monitor and lock/logout speaking of lock/logout and applets which are essentially rows of icons ... has someone worked out the issues around QML widgets not have size hints and appropriate sizing of applets like lock/logout in panels in plasma-desktop? yep, the slc applet should have correct sizing now. unfortunately it requires some manual calculation from the qml side since is not something offered stock, but if the root element offers minimumWidth and minimumHeight the applet will use them as minimum size hints. How about using plasmoid.resize(w,h)? in plasma2, in the case of a scene that doesn't have qgraphicswidgets in the middle, sizing of things should be a bit easier and a tad more automatic (just remains pretty hard do to expanding things like the taskbar) Viranch, that should be done for the lock/logout applet as well 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: Order QML plasmoid context menu
On Sun, Sep 18, 2011 at 5:21 AM, Francesco Nwokeka francesco.nwok...@gmail.com wrote: Hi guys, wanted to notify about an issue. Currently it's possible to add actions to a QML plasmoid's context menu. But this menu is not ordered in any way. Nor by insertion and nor alphabetically. So I wanted to know, is there some hidden function to do this (order the list of actions) or does it still have to be implemented? Actions are added in the order of plasmoid.setAction function calls, followed by the standard entries. Eg: plasmoid.setAction(act1, myAction1, icon1); plasmoid.setAction(act2, myAction2, icon2); would look like: myAction1 myAction2 Plasmoid's settings Desktop settings -- Remove this plasmoid As far as re-ordering is concerned, I don't think there is any way to do such a thing, but I might be missing something too. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Re: Some simple ideas for desktop 4.8
About borders.. The blue window glow around the windows is also quite oldish, of course, personally :) I have turned it off, since windows look much better without the glow, also dragging windows is much faster when compositing is on, without the glow (at least on my 3-year old laptop). So it would be a good thought to have it disabled by default (of course, let there be the option to enable it) Just a thought :) Cheers, Viranch On Sun, Sep 25, 2011 at 9:11 PM, Alex Fiestas afies...@kde.org wrote: On Sunday, September 25, 2011 05:28:12 PM Martin Gräßlin wrote: On Sunday 25 September 2011 16:24:25 Alex Fiestas wrote: Hi there ! I've been using the default settings for a while now (like 1month) and I'd like to propose a couple changes for 4.8, let's see what are your opinions: 1-Show the same windows in Window switcher and Taskbar: Right now by default, Window Switcher is showing only the windows in the current desktop while the taskbar is showing the windows in all desktop. This has been very confusing for me since I'm not able to access to a window I'm seeing in the taskbar via Window Switcher. I don't care wether we switch both to Show only windows in this workspace or the other way, but both should show the same. With window switcher you mean Alt+Tab, right? Yes, the default Box one Having those two synced sounds reasonable and I would suggest to go for only current desktop as by default it's only one desktop and means less clutter. Well activating this we have the problem that libtaskmanager is quite broken right now, so we must fix it if we turn this option on by deafult. without borders you cannot resize the window. Yes you can, a small grabber? appears on the bottom-right, but I can see how that is less usable if you don't know it, though in the other hand that solution quite common these days, for example web-textareas. Nuno has been begging me to improve the situation and maybe I get to do something about it. Anyway this is something Nuno has to decide and IIRC he wants to have it on per app basis, e.g. Konsole without borders, others with borders. Apart from that we can hardly change anything in a general way for window decorations. This would have to be completely inside the Oxygen decoration. Oki, then I will wait to see what Nuno thinks about this :p ___ 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
QML DataSource
Hi all,
I am facing a very annoying problem in QML plasmoids with the dataengines.
The following does not seem to work ever for almost all dataengines:
PlasmaCore.DataSource {
id: dataSource
engine: someEngine
onSourceAdded: print (data[source][some_valid_key];
}
The print statement always throws data[source] is [undefined]. This happens
with
most engines, esp with those engines where sources are added/removed
dynamically
by the engine. This has been very annoying, any help will be awesome!
Thanks,
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: [Junior job] Re: translations of qml plasmoids
Hey, On Wed, Oct 19, 2011 at 5:39 PM, Marco Martin notm...@gmail.com wrote: so, the translation of the activity screen plasmoid worked, http://websvn.kde.org/trunk/l10n-kde4/templates/messages/playground- base/org.kde.active.activityscreen.pot?revision=1259592view=markup that means, a proper adapted Messages.sh file has to be added in every plasmoid file. that's a quite easy and boring job, also a quite easy junior job... anybody interested? :p I have never worked with translations. Could you summarize the steps for this? Thanks, Viranch 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: Some help for recreating the Plasma Tool Box in QML
On Thu, Oct 20, 2011 at 7:57 PM, Mark mark...@gmail.com wrote: * What is the current way to get the basic KDE SC running from Git? Build kdelibs, kde-runtime, libkactivities and kde-workspace. * What environment variables do i need to set to use KDE from Git (libdir, bindir...)? I use the same old bashrc from techbase. * Which branches do i need for QML + KDE? Master on all of the above repos will do. * Where do i need to be for recreating the plasma tool box in QML? kdebase-workspace? I guess, but I'm not sure. * How do i get a QML file running without the window decorations (the current plasma tool box doesn't have decorations so i guess i don't want them in QML either) Well, I'm not sure if it is possible to just change a part of the panel (the tool box). You'd need to re-do the panel in qml, afaik. But again, not sure. * Target: KDE 4.8? I prefer to use KDE from git since (as far as i understand it) the new KDE QML components made during this GSoC are in either a special git branch or in the branch that becomes KDE 4.8. And i'm kinda fearing odd problems if i use git for that but KDE 4.7 for the rest. Please do correct me if i'm wrong. I think that covers about all my questions. Last but not least. Above is just my idea of what i'd like to do. I've not spend any time yet on making anything and don't know if anyone else is already working on this. I also don't know if this is a change people want.. So some more information here would be nice. Kind regards and thanx in favor for any responses, Mark ___ 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: QML device notifier
On Sun, Oct 30, 2011 at 10:03 PM, Marco Martin notm...@gmail.com wrote: the system kcms is the only detail still missing :p after this is in, i'll merge it's in! commit: http://commits.kde.org/kdelibs/f53ef3872b3755350ca5eb99eea8ef3b96dd8c74 pushed to kdelibs KDE/4.7 branch -- 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
QML Analog Clock
Hi, I have recently started working on the QML analog clock plasmoid. I adapted basic code from declarative-plasmoids repo. I added and implemented the config dialog and options, and showing the timezone. What I'm stuck at: I saw the original analog clock uses clockapplet from libs/plasmaclock. This clock applet implements all the config dialog sections and the popup calendar widget UI and all kinds of other stuff. How can those be brought into this QML applet? I don't think its possible with only QML, so I think I'll have to plugin cpp code. But I'd like to listen to expert advice on how to do that or any other solution ;) Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QML Analog Clock
On Fri, Nov 11, 2011 at 1:00 PM, todd rme toddrme2...@gmail.com wrote: Is there a QML version of the calendar widget yet? Nope. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QML Analog Clock
On Sat, Nov 12, 2011 at 8:10 PM, Marco Martin notm...@gmail.com wrote: how does this sound? That's helpful! But what about loading of the multiple custom config uis? If that part will also be taken care of by the c++ part, I don't see the point of doing this applet in qml at all. All that qml part does is to show few svgs and make them move around. Everything else is being done by c++. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QML Analog Clock
On Mon, Nov 14, 2011 at 12:50 PM, Aaron J. Seigo ase...@kde.org wrote: the point is so that we can eventually one day move off of QGraphicsView and onto the QML-only, OpenGL-accelerated QML SceneGraph :) Right! I totally missed the point about moving away from qgraphicsview ;) makes sense. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QML Analog Clock
On Mon, Nov 14, 2011 at 1:35 PM, Marco Martin notm...@gmail.com wrote: On Friday 11 November 2011, Viranch Mehta wrote: On Fri, Nov 11, 2011 at 1:00 PM, todd rme toddrme2...@gmail.com wrote: Is there a QML version of the calendar widget yet? Nope. Viranch that can go for grades as well. replacing the internal clock leaving the calendar, then redoing the calendar for instance Yes, I actually thought of that too in the beginning. I think I'll start doing that now. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: bug killing
On Mon, Nov 21, 2011 at 2:07 AM, Aaron J. Seigo ase...@kde.org wrote: ** * hold a seminar on irc to help teach others effective bugs.kde.orgreport sqashing, plasma style I'm *very* interested in this! Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: bug killing
Hi, Could someone please post a log of the discussions as I couldn't make it. Cheers, Viranch On Dec 4, 2011 12:00 AM, Aaron J. Seigo ase...@kde.org wrote: On Saturday, December 3, 2011 17:37:01 Martin =?ISO-8859-1?Q?Gr=E4=DFlin?= wrote: On Saturday 03 December 2011 17:20:20 Aaron J. Seigo wrote: On Sunday, November 20, 2011 21:37:16 Aaron J. Seigo wrote: so .. 1700+ bugs. fun, huh? :) as an encouraging little update, thanks to everyone's combined efforts we are now down to ~1270 reports. that's ~500 fewer than when i sent the email 2 weeks ago. I want that, too. And it's only 410 bugs and kwin is down to 0. So please ehehe ;) seriously: we should do a bug days for kwin too.. maybe in january? assuming you need or could use the help for a good cleaning. Seriously: awesome job all of you. Keep that going. I'm pretty sure we can get plasma down to around 300 to 500 real bugs and that makes it managable. agreed... right now i'm looking at the 1000 mark as the next acheivable short-term goal. btw.. did a quick LOC count the other day and plasma is shipping nearly 300k LOC right now. that include Plasma Active, but not the applications, just the actual plasma bits. that doesn't include kwin, either, which itself is 92k+ LOC in kde-workspace. dr. konqi makes reporting bugs amazingly easy and pleasant now as well... given the feature-to-LOC ratio, it's quite evident why we have so many reports. but i'm quite enthused with how much progress was made in just 2 days (of admiteedly abnormal levels of effort ;) -- 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 ___ 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: QML widget for the Skrooge plasma dashboard
Hi, On Mon, Dec 5, 2011 at 12:50 AM, Guillaume DE BURE guillaume.deb...@gmail.com wrote: file:///.../main.qml:36: Unable to assign [undefined] to QString text That would indicate that there is no source connected, right ? So what did I do wrong ? The DataSource has no property called count. Hence [undefined]. You can have a look at the API here: http://techbase.kde.org/Development/Tutorials/Plasma/QML/API#DataSource Also, since you've already declared connectedSources in the DataSource, you don't need to redo it inside Component.onCompleted ;) Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QML Analog Clock
On Mon, Nov 14, 2011 at 4:41 PM, Viranch Mehta viranch.me...@gmail.comwrote: On Mon, Nov 14, 2011 at 1:35 PM, Marco Martin notm...@gmail.com wrote: On Friday 11 November 2011, Viranch Mehta wrote: On Fri, Nov 11, 2011 at 1:00 PM, todd rme toddrme2...@gmail.com wrote: Is there a QML version of the calendar widget yet? Nope. Viranch that can go for grades as well. replacing the internal clock leaving the calendar, then redoing the calendar for instance ok so i have been working on the qml calendar widget, and now there's another problem: the existing widget uses the complex KCalendarSystem to manage the various calendar systems (which have different no of months, days per month, names of months/weekdays, etc. also various logic like get weekday, week number from date, etc) so i was wondering if this would have to be taken care from c++ part (i have not yet plugged in any c++ part in the plasmoid). Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Skrooge plasma dashboard: stuck with QML
On Wed, Dec 21, 2011 at 4:55 AM, Guillaume DE BURE guillaume.deb...@gmail.com wrote: I'm doing this in 4.7.90 (unstable archlinux packages) You should probably move to KDE compiled from git source. 1) widget size: --- even though I specified the widget size, if the dashboard contains only QML widgets, they appear with a very small width: http://wstaw.org/m/2011/12/20/plasma-desktopVB1577.png http://wstaw.org/m/2011/12/20/plasma-desktopav1577.png As soon as I add a C++ widget, the size is correctly computed: http://wstaw.org/m/2011/12/20/plasma-desktopbb1577.png Try adding X-Plasma-DefaultSize=600,600 in metadata.desktop and instead of width/height: 600 in main.qml, do the following: property int minimumWidth: 600 property int minimumHeight: 600 This might produce expected behavior. 2) PlasmaComponents.Label font size: -- For some reason, the font size there is significantly larger than in basic QML Text component. Any idea why ? You can simply use Text component wherever you just want to display text. 3) ListView highlight: --- With the aforementionned code, I can't seem to get the expected behaviour on highlighting items in the list. Any idea why ? The highlight follows the currentItem. So you need to add a MouseArea in your delegate with hoverEnabled: true and onEntered: list.currentIndex = index This will make the hovered item your list's current item and the highlight will follow it. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QML plasmoid porting status needs updating?
On Fri, Dec 30, 2011 at 5:55 AM, Shaun Reich shaun.re...@kdemail.netwrote: This is woefully out of date, and I think it'd be a good idea for everyone to add what they're working on, where it is, etc, if everyone's okay with it. Because I want a good overview of what needs to be done and what is being worked on. http://community.kde.org/Plasma/PlasmoidScripting#Porting_Plasmoids_to_QML Thanks for pointing this out. I've updated what I'm working on. Additionally, is this pretty much the only QML plasma docs? http://techbase.kde.org/Development/Tutorials/Plasma/QML/API I have these bookmarked in addition to above: Development/Tutorials/Plasma/QML/GettingStarted - KDE TechBasehttp://techbase.kde.org/Development/Tutorials/Plasma/QML/GettingStarted Development/Tutorials/Plasma/JavaScript/API-PlasmoidObject - KDE TechBasehttp://techbase.kde.org/Development/Tutorials/Plasma/JavaScript/API-PlasmoidObject Development/Tutorials/Plasma/ThemeDetails - KDE TechBasehttp://techbase.kde.org/Projects/Plasma/Theme I don't really use the first one. The second one comes in handy when you want to play around with plasmoid object. The third is useful when you're using SVGs, it has the list of all available SVGs in 8th section. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: QML plasmoid porting status needs updating?
On Fri, Dec 30, 2011 at 10:03 PM, Mark mark...@gmail.com wrote: Aren't the Digital Clock and Calendar one big element? Yes, Calendar is used in Digital Clock and Analog Clock and that's where I'm somewhat stuck now. It all depends on KLocale and KCalendarSystem. I still have to see how to get those into QML. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Access D-BUS through QML/JS
On Thu, Jan 12, 2012 at 12:20 PM, Shaun Reich shaun.re...@kdemail.netwrote: was wondering if and how I could call a remote method (org.freedesktop.screensaver) through a pure QML/JS applet. Do I need to use a dataengine? Should I make that dataengine C++ or JS? one obvious approach would be to implement a service that would make the dbus calls in a c++ dataengine; and call the service from the qml/js applet. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: A new way of QML PopupApplets
On Fri, Mar 2, 2012 at 3:35 AM, Marco Martin notm...@gmail.com wrote: right now it wasn't possible to write in qml those applets that are always collapsed to a tiny element, but *not* an icon, like the notifications or the digital clock. Aren't we already doing this thing in battery monitor? Its not collapsed to just an icon, and it has a popup. The tiny element (battery status svg) is in the main qml item of the plasmoid, and the popup component (power profiles, screen brightness, etc.) is inside a PlasmaCore.Dialog's mainItem. And this is without making it a PopupApplet. exporting a Component property compactRepresentation, will use that whatever widgets and fancy graphics contains as icon and will be always collapsed, just the same behavior of the c++ ones. This method does sound like a more convenient way of doing it though. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: A new way of QML PopupApplets
On Fri, Mar 2, 2012 at 2:47 PM, Marco Martin notm...@gmail.com wrote: On Friday 02 March 2012, Marco Martin wrote: yep, what i want to avoid is having to manage a dialog by hand more specifically what i want in palsma2 is every applet being able to specify that is collapsable and optionally specify a component for that representation. Cool! Is this stable enough now? If so, I'd like to update the battery monitor. Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/battery/CMakeLists.txt 7794f88 plasma/generic/applets/battery/Messages.sh 8b06e2d plasma/generic/applets/battery/README.txt 5b352e8 plasma/generic/applets/battery/battery-oxygen-inkscape.svgz b68ba66 plasma/generic/applets/battery/battery-oxygen.svgz a037e60 plasma/generic/applets/battery/battery.h ebc1a3d plasma/generic/applets/battery/battery.cpp 3a5cda3 plasma/generic/applets/battery/batteryConfig.ui 5595ca2 plasma/generic/applets/battery/plasma-battery-default.desktop e254028 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 13, 2012, 9:51 a.m.) Review request for Plasma. Changes --- Temporarily remove i18n() for complex strings Alias plasmoid.rootItem.pmSource to pmSource Implement powermanagement enabled/disabled Make setScreenBrightness service job return void Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs (updated) - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 81 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52425#file52425line81 This should not be in there, basically only if it has been enabled by config showRemainingTime (look at the C++ version when it's shown). We explicitely excluded this feature by default since the remaining time cannot be accurately computed. I don't see a showRemainingTime config in C++ version (I'm looking at master). Should I add this in the QML version? On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/dataengines/powermanagement/powermanagementjob.cpp, line 87 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52432#file52432line87 This line always sets result to false, no matter what happened earlier. I think the code you added here is correct, could you check why it worked earlier? The control reaches here only if there were no 'return's earlier. It works because there is a return inside each if case. So if the operation name was invalid, it would setResult(false), otherwise setResult(whatever the operation returns). On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 66 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52425#file52425line66 The word puzzle here is not translatable. You'll need to enclose a full string into i18n(), with the current code, translators can't figure out what the message is. Also, appending strings to each other doesn't work, as the word order might be different. So you have to identify the cases, and then return a completely translated string. For now, I'm dropping i18n() completely on such computed strings. On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml, line 65 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52426#file52426line65 instead of plasmoid.rootItem.pmSource, try using just pmSource. If necessary, that means moving pmSource somewhere visible. Should make porting to QML2 easier. Just pmSource doesn't work because compactRepresentation is a Component, and hence pmSource cannot be moved to any visible location. This is the only way to do it AFAICT. Use of plasmoid.rootItem.pmSource can be reduced by assigning it to another variable inside the component. (Look at updated diff). On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml, line 89 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52426#file52426line89 This guy is unnecessary, as the exact same info is already shown in the dialog. I'd prefer getting rid of this overlay altogether (including the config option). Getting this info from dialog requires two clicks (opening and closing the dialog). While this overlay simply displays it. We can still drop it if desired. On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml, line 144 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52426#file52426line144 What's missing here? Either ditch // TODO, or add a note what's missing I figured out what should be done here, and implemented (see updated diff). - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11311 --- On March 13, 2012, 9:51 a.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 13, 2012, 9:51 a.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 81 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52425#file52425line81 This should not be in there, basically only if it has been enabled by config showRemainingTime (look at the C++ version when it's shown). We explicitely excluded this feature by default since the remaining time cannot be accurately computed. Viranch Mehta wrote: I don't see a showRemainingTime config in C++ version (I'm looking at master). Should I add this in the QML version? Done. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11311 --- On March 13, 2012, 9:51 a.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 13, 2012, 9:51 a.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 13, 2012, 10:26 a.m.) Review request for Plasma. Changes --- Add a config option to show/hide remaining battery time. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs (updated) - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 13, 2012, 11:14 a.m.) Review request for Plasma. Changes --- Make the battery charge info string translatable as done in C++ version. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs (updated) - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 12, 2012, 10:31 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 66 http://git.reviewboard.kde.org/r/104226/diff/1/?file=52425#file52425line66 The word puzzle here is not translatable. You'll need to enclose a full string into i18n(), with the current code, translators can't figure out what the message is. Also, appending strings to each other doesn't work, as the word order might be different. So you have to identify the cases, and then return a completely translated string. Viranch Mehta wrote: For now, I'm dropping i18n() completely on such computed strings. Fixed it to work in a manner similar to the C++ one. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11311 --- On March 13, 2012, 11:14 a.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 13, 2012, 11:14 a.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 13, 2012, 11:59 a.m.) Review request for Plasma. Changes --- Take results from service operation calls (beginSuppressing...() and stopSuppressing...()) and store them in cookies. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs (updated) - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 14, 2012, 5:48 p.m.) Review request for Plasma. Changes --- Properly translates all displayed text with comments to translators wherever necessary (all of the i18n/i18nc/i18np are taken from C++ version). Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs (updated) - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 13, 2012, 5:32 p.m., David Edmundson wrote:
plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 66
http://git.reviewboard.kde.org/r/104226/diff/5/?file=53015#file53015line66
This i18n string doesn't really work.
1) This string doesn't really contain any words, so it's not really
suitable for translation. At least use i18nc() so translators have context
of what it is.
2) the state (i.e charging, charged, discharging) is never translated.
I have adopted the translation straight from the C++ applet now (except the
if-else logic):
if (pluggedIn) {
if (percent100) return i18n(%1 (charging), percent);
else return i18n(%1 (charged), percent);
} else {
return i18n(%1 (discharging), percent);
}
On March 13, 2012, 5:32 p.m., David Edmundson wrote:
plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 101
http://git.reviewboard.kde.org/r/104226/diff/5/?file=53015#file53015line101
This isn't translated.
Also this is a word puzzle.
http://techbase.kde.org/Development/Tutorials/Localization/i18n_Mistakes#Pitfall_.232:_Word_Puzzles
You also can't do
if (hrs==1) {
hour
} else {
hours
}
for some languages plurals come after 1st, 11th 111th.. it's not as
simple as you just wrote.
use i18np.
We can achieve completely formatted and translated string only by
KLocale::prettyFormatDuration() as far as I know. But since we don't yet have
KLocale QML bindings, I have a temporary work around:
var time = new Date(remainingMsec);
var hrs = i18np(1 hour, %1 hours, time.getUTCHours());
var mins = i18np(1 minute, %1 minutes, time.getUTCMinutes());
return hrs+, +mins;
On March 13, 2012, 5:32 p.m., David Edmundson wrote:
plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 108
http://git.reviewboard.kde.org/r/104226/diff/5/?file=53015#file53015line108
Don't do this to determine how wide something should be.
What if the japanese for power management enabled is only 3
characters long and the time remaining is larger?
Even if you could garauntee it's the longest string right now, what if
someone changes this in the future?
set the Grid to be
width:childRect.width.
and remove the call to width on all these labels, and that /should/
work. (I've not tested that and could be wrong.)
Discarded.
This was done to achieve right alignment to the labels (on the left side). For
now, they are all left-aligned due to this change.
- Viranch
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104226/#review11360
---
On March 14, 2012, 5:48 p.m., Viranch Mehta wrote:
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104226/
---
(Updated March 14, 2012, 5:48 p.m.)
Review request for Plasma.
Description
---
I fixed the QML battery monitor to be fairly usable and this diff merges it
to master.
Diffs
-
plasma/generic/applets/CMakeLists.txt 2dedcb2
plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION
plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION
plasma/generic/applets/batterymonitor/README.txt PRE-CREATION
plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz
PRE-CREATION
plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION
plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION
plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml
PRE-CREATION
plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml
PRE-CREATION
plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml
PRE-CREATION
plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION
plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION
plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2
plasma/generic/dataengines/powermanagement/powermanagementengine.cpp
5572fcb
plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015
plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0
plasma/generic/dataengines/powermanagement/powermanagementservice.operations
ad1301f
Diff: http://git.reviewboard.kde.org/r/104226/diff/
Testing
---
Applet and dataengine both tested and work fine.
Thanks,
Viranch Mehta
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 14, 2012, 6:39 p.m.) Review request for Plasma. Changes --- Add screenshot. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots (updated) --- http://git.reviewboard.kde.org/r/104226/s/464/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 13, 2012, 5:32 p.m., David Edmundson wrote: Read up on i18n, ideally most of http://techbase.kde.org/Development/Tutorials/Localization/ and double check everything again. Also personally I like to submit a screenshot with any very visual change. Thanks for the link. Screenshot uploaded. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11360 --- On March 14, 2012, 6:39 p.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 14, 2012, 6:39 p.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- http://git.reviewboard.kde.org/r/104226/s/464/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 14, 2012, 6:46 p.m.) Review request for Plasma. Changes --- Fix the typo i18n(%1...) to i18n(%1%...) to show the '%' sign in Battery: field of the popup. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs (updated) - plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- http://git.reviewboard.kde.org/r/104226/s/464/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 20, 2012, 9:44 p.m.) Review request for Plasma. Changes --- Updated screenshot and the patch. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs (updated) - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots (updated) --- http://git.reviewboard.kde.org/r/104226/s/482/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 19, 2012, 7:23 p.m., Marco Martin wrote: as for comments on the screenshot, labels on the left should be right-aligned, the code seems fine now Labels now right-aligned. Screenshot updated. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11632 --- On March 20, 2012, 9:44 p.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 20, 2012, 9:44 p.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- http://git.reviewboard.kde.org/r/104226/s/482/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 20, 2012, 10:27 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 97 http://git.reviewboard.kde.org/r/104226/diff/7/?file=53144#file53144line97 horizontalAlignment: Text.AlignRight This doesn't work for some reason (the width of the Label item is same as that required by the painted text, not the width of the cell it is in). Instead of that, anchors.right: parent.right is used. On March 20, 2012, 10:27 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 81 http://git.reviewboard.kde.org/r/104226/diff/7/?file=53144#file53144line81 showRemainingTime? There is added condition: text!= showRemainingTime elsewhere. Hence here is simply connected to it. On March 20, 2012, 10:27 a.m., Sebastian Kügler wrote: plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml, line 130 http://git.reviewboard.kde.org/r/104226/diff/7/?file=53144#file53144line130 sleepButton and hibernateButton should be hidden when the system does not support it. Looks like this was forgotten here. Did you make sure it works? It's exposed in the dataengine, in Sleep States, keys: Suspend, Hibernate. Could be as simple as visible: powermanagmentEngine.data[Sleep States][Suspend] etc... Yes it was forgotten, thanks for pointing out. Added the visible condition. On March 20, 2012, 10:27 a.m., Viranch Mehta wrote: When installing from the plasma/viranch/powermanagementservices branch, I'm getting this error during installation: CMake Error at applets/batterymonitor/cmake_install.cmake:56 (FILE): file INSTALL cannot find /home/sebas/kdesvn/src/kde-workspace/plasma/generic/applets/batterymonitor/battery-oxygen.svgz. Can you have a look? Fixed. I'd recommend not to merge it from that branch though, because it also has lock_logout QML applet and few other things that might cause a lot of merge conflicts. Use the attached diff for merging. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11650 --- On March 20, 2012, 9:44 p.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 20, 2012, 9:44 p.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- http://git.reviewboard.kde.org/r/104226/s/482/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 20, 2012, 10:02 p.m., Vishesh Handa wrote: Screenshot: http://git.reviewboard.kde.org This looks kind bad. There should be a certain amount of padding. There is already some default padding in the popup. It is stripped off while viewing in plasmoidviewer for some reason, notice the absence of top left and bottom left rounded corners. Should look fine when added to an actual workspace. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11667 --- On March 20, 2012, 9:44 p.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 20, 2012, 9:44 p.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- http://git.reviewboard.kde.org/r/104226/s/482/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 21, 2012, 9:20 a.m., Marco Martin wrote: let's merge then fix all the things that may remain ok? (if you are rady to quickly fix issues as they come in, because many will report ;) Yep, I'm ready. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11683 --- On March 20, 2012, 9:44 p.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 20, 2012, 9:44 p.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- http://git.reviewboard.kde.org/r/104226/s/482/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On March 21, 2012, 9:20 a.m., Marco Martin wrote: let's merge then fix all the things that may remain ok? (if you are rady to quickly fix issues as they come in, because many will report ;) Viranch Mehta wrote: Yep, I'm ready. Sebastian Kügler wrote: I'd prefer if we wait with merging until all issues are fixed, there are still quite a few regressions here, from my testing: - discharge percentage isn't shown at all - the top right graphic in the popup is missing - information on the RHS doesn't match labels on the LHS - the Hibernate button is shown here, while the machine only supports sleep (works fine with old applet) - s/Powermanagement/Power Management - Lots of runtime warnings, especially about dereferencing missing datasources Sebastian Kügler wrote: I'm testing from the plasma/viranch/powermanagementservices branch, does that one contain the latest state of the art? Yes, the plasma/viranch/powermanagementservices branch contains the latest QML applet (including the following fixes): (I'm not updating the diff here anymore. Will do it when applet is ready.) - discharge percentage isn't shown at all - information on the RHS doesn't match labels on the LHS Fixed both. These were due to some serious mistakes in merge conflicts in last commit (my apologies) - the Hibernate button is shown here, while the machine only supports sleep (works fine with old applet) - Lots of runtime warnings, especially about dereferencing missing datasources Fixed both. Had forgotten to add Sleep States in connectedSources. - s/Powermanagement/Power Management Fixed this in the i18nc comment of the label Power management enabled:. Did I miss elsewhere? - the top right graphic in the popup is missing Added, and working. - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review11683 --- On March 20, 2012, 9:44 p.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 20, 2012, 9:44 p.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- http://git.reviewboard.kde.org/r/104226/s/482/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 23, 2012, 6:20 p.m.) Review request for Plasma. Changes --- Screenshot update, showing the top right battery icon in the popup Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots (updated) --- with top right graphic in the popup http://git.reviewboard.kde.org/r/104226/s/487/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request: Couple of fixes to QML ScrollBar component
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104402/ --- Review request for Plasma. Description --- This patch fixes following issues with QML Scrollbar component: - The hover and non-hover arrow images in vertical scrollbars are the same. - The down/right arrows are connected to events of mousearea of up/left arrows. - The mouseareas of the arrows do not have hoverEnabled property set to true. This is required to use their containsMouse property. Diffs - plasma/declarativeimports/plasmacomponents/qml/private/ScrollBarDelegate.qml 15ee7aa Diff: http://git.reviewboard.kde.org/r/104402/diff/ Testing --- I didn't know how to test these from Gallery.qml (what command?). Tested, however, by putting in a test plasmoid. Works well now. Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On April 2, 2012, 12:25 p.m., Sebastian Kügler wrote: Looking good now, thanks a lot for your patience! I'm getting this TypeError still, haven't looked into the reason behind it: file:///home/sebas/kdesvn/install/share/apps/plasma/plasmoids/battery/contents/ui/batterymonitor.qml:77: TypeError: Result of expression 'pmSource.data[PowerDevil]' [undefined] is not an object. I propose to merge it into master now. There's still a bit of polishing possible (margins, for example), but we can easily do those when it's in master. I'm not getting the TypeError here. About the merge, I have worked on QML lock/logout applet in the same branch. So should I just copy-paste QML battery monitor into master, or cherry-pick from the branch to preserve the history? - Viranch --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/#review12114 --- On March 23, 2012, 6:20 p.m., Viranch Mehta wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated March 23, 2012, 6:20 p.m.) Review request for Plasma. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots --- with top right graphic in the popup http://git.reviewboard.kde.org/r/104226/s/487/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On Mon, Apr 2, 2012 at 7:52 PM, Marco Martin notm...@gmail.com wrote: hmm, not too happy about both, let's make lock/logout work as well then merge everything? That sounds fine. I'll soon open a review request. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104226/ --- (Updated April 3, 2012, 5:12 a.m.) Review request for Plasma. Changes --- Add a screenshot of the applet when its in a panel. Description --- I fixed the QML battery monitor to be fairly usable and this diff merges it to master. Diffs - plasma/generic/applets/CMakeLists.txt 2dedcb2 plasma/generic/applets/batterymonitor/CMakeLists.txt PRE-CREATION plasma/generic/applets/batterymonitor/Messages.sh PRE-CREATION plasma/generic/applets/batterymonitor/README.txt PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz PRE-CREATION plasma/generic/applets/batterymonitor/battery-oxygen.svgz PRE-CREATION plasma/generic/applets/batterymonitor/contents/config/main.xml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml PRE-CREATION plasma/generic/applets/batterymonitor/contents/ui/config.ui PRE-CREATION plasma/generic/applets/batterymonitor/metadata.desktop PRE-CREATION plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2 plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015 plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0 plasma/generic/dataengines/powermanagement/powermanagementservice.operations ad1301f Diff: http://git.reviewboard.kde.org/r/104226/diff/ Testing --- Applet and dataengine both tested and work fine. Screenshots (updated) --- with top right graphic in the popup http://git.reviewboard.kde.org/r/104226/s/487/ applet on the workspace http://git.reviewboard.kde.org/r/104226/s/503/ Thanks, Viranch Mehta ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On Tue, Apr 3, 2012 at 1:45 PM, Marco Martin notm...@gmail.com wrote:
here the biggest challenge i guess is making all resize correctly in
horizontal/vertical panels.
Yes, just spent 5 hours on this. Finally resizing works almost fine inside
and outside panels, except:
* it does not resize/change when the orientation of a panel changes (eg, top
panel is moved to right). i tried addEventListener('FormFactorChanged',
formFactorChanged); but the callback is never called.
* if the applet is on the desktop, the last visible icon in the list
flickers _a lot_
when the applet is resized. this happens for more than one icons visible,
and
happens to only the last icon.
* the c++ applet shows a confirmation box for suspend hibernate,
how can this be done in QML? how about showing our regular applet-popup
like in the battery monitor? with this option, the user also wouldn't have
to
move the mouse too much around the screen.
Cheers,
Viranch
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On Tue, Apr 3, 2012 at 7:24 PM, Daniel Nicoletti dantt...@gmail.com wrote: doesn't the kworskspace API handle the confirmation already? (I know it does for Shutdown, logoff and restart) kworkspace API is only used for leave (a combined action for logout, turn off, restart). for suspend actions, dbus calls are made. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Merge the final and fixed QML battery monitor to master.
On Tue, Apr 3, 2012 at 7:25 PM, Marco Martin notm...@gmail.com wrote: * the c++ applet shows a confirmation box for suspend hibernate, how can this be done in QML? how about showing our regular applet-popup like in the battery monitor? with this option, the user also wouldn't have to move the mouse too much around the screen. hmm, it should call just the ksmserver dialog, should be added to the service? (power? another one?) we have to simply show a KMessageBox as does the c++ applet. the qml applet uses power management engine for all the actions. putting it in power management services would also affect the suspend actions of battery monitor. Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: Couple of fixes to QML ScrollBar component
On Wed, Apr 4, 2012 at 3:51 PM, Sebastian Kügler se...@kde.org wrote: On Sunday, March 25, 2012 02:59:35 Viranch Mehta wrote: I didn't know how to test these from Gallery.qml (what command?). Tested, however, by putting in a test plasmoid. Works well now. I'm using: KDE_PLASMA_COMPONENTS_PLATFORM=touch qmlviewer -I \ /home/sebas/kdesvn/install/lib/kde4/imports/ That did the job! Thanks :) Cheers, Viranch ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request: QML Lock/logout applet
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104592/
---
Review request for Plasma and Marco Martin.
Description
---
This patch essentially contains the new QML lock logout applet, along with
required changes to powermanagement engine.
The patch contains *some* irrelevant code too (in powermanagement data engine)
since the same branch has QML battery applet. So it'd be best to checkout
plasma/viranch/powermanagementservices branch and compile
plasma/generic/{applets/lockout,dataengines/powermanagement} instead of using
the attached patch.
Two pending issues with the applet:
- There is no shine on the icons on hover.
- Sleep/Hibernate actions do not ask for confirmation, as does the current C++
version
Comments welcome on this thread :)
Diffs
-
plasma/generic/applets/lock_logout/CMakeLists.txt a19615b
plasma/generic/applets/lock_logout/contents/config/main.xml PRE-CREATION
plasma/generic/applets/lock_logout/contents/ui/config.ui PRE-CREATION
plasma/generic/applets/lock_logout/contents/ui/data.js PRE-CREATION
plasma/generic/applets/lock_logout/contents/ui/lockout.qml PRE-CREATION
plasma/generic/applets/lock_logout/lockout.h 94f33cf
plasma/generic/applets/lock_logout/lockout.cpp 830e64b
plasma/generic/applets/lock_logout/lockoutConfig.ui d9285c2
plasma/generic/applets/lock_logout/metadata.desktop PRE-CREATION
plasma/generic/applets/lock_logout/plasma-applet-lockout.desktop d8a4337
plasma/generic/dataengines/powermanagement/CMakeLists.txt 9f2a528
plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2
plasma/generic/dataengines/powermanagement/powermanagementengine.cpp 5572fcb
plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015
plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0
plasma/generic/dataengines/powermanagement/powermanagementservice.operations
ad1301f
Diff: http://git.reviewboard.kde.org/r/104592/diff/
Testing
---
- Tested inside horizontal/vertical panels and floating on desktop, sizing
works as expected.
- There may be some unexpected behavior if the panel containing the applet is
changed from being horizontal to vertical or vice-versa.
Screenshots
---
Screenshot of the applet on desktop
http://git.reviewboard.kde.org/r/104592/s/526/
Thanks,
Viranch Mehta
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request: QML Lock/logout applet
On April 14, 2012, 10:11 p.m., Kai Uwe Broulik wrote:
Couldn’t we add a Hibernate dialog, similar to the confirmation on
shutdown, i.e that dialog that counts down Shutting down in 30 seconds
with the ability to cancel it? You press Hibernate, and a KSMserver dialog
(that we don’t have yet?) pops up and after 30 seconds, it will hibernate
anway. This way, if you miss the dialog, e.g. you are in a hurry, it will
proceed after the countdown runs out.
That seems like a good idea. Comments?
- Viranch
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104592/#review12448
---
On April 13, 2012, 6:18 p.m., Viranch Mehta wrote:
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104592/
---
(Updated April 13, 2012, 6:18 p.m.)
Review request for Plasma and Marco Martin.
Description
---
This patch essentially contains the new QML lock logout applet, along with
required changes to powermanagement engine.
The patch contains *some* irrelevant code too (in powermanagement data
engine) since the same branch has QML battery applet. So it'd be best to
checkout plasma/viranch/powermanagementservices branch and compile
plasma/generic/{applets/lockout,dataengines/powermanagement} instead of using
the attached patch.
Two pending issues with the applet:
- There is no shine on the icons on hover.
- Sleep/Hibernate actions do not ask for confirmation, as does the current
C++ version
Comments welcome on this thread :)
Diffs
-
plasma/generic/applets/lock_logout/CMakeLists.txt a19615b
plasma/generic/applets/lock_logout/contents/config/main.xml PRE-CREATION
plasma/generic/applets/lock_logout/contents/ui/config.ui PRE-CREATION
plasma/generic/applets/lock_logout/contents/ui/data.js PRE-CREATION
plasma/generic/applets/lock_logout/contents/ui/lockout.qml PRE-CREATION
plasma/generic/applets/lock_logout/lockout.h 94f33cf
plasma/generic/applets/lock_logout/lockout.cpp 830e64b
plasma/generic/applets/lock_logout/lockoutConfig.ui d9285c2
plasma/generic/applets/lock_logout/metadata.desktop PRE-CREATION
plasma/generic/applets/lock_logout/plasma-applet-lockout.desktop d8a4337
plasma/generic/dataengines/powermanagement/CMakeLists.txt 9f2a528
plasma/generic/dataengines/powermanagement/powermanagementengine.h 20642c2
plasma/generic/dataengines/powermanagement/powermanagementengine.cpp
5572fcb
plasma/generic/dataengines/powermanagement/powermanagementjob.h 2c32015
plasma/generic/dataengines/powermanagement/powermanagementjob.cpp e205bb0
plasma/generic/dataengines/powermanagement/powermanagementservice.operations
ad1301f
Diff: http://git.reviewboard.kde.org/r/104592/diff/
Testing
---
- Tested inside horizontal/vertical panels and floating on desktop, sizing
works as expected.
- There may be some unexpected behavior if the panel containing the applet is
changed from being horizontal to vertical or vice-versa.
Screenshots
---
Screenshot of the applet on desktop
http://git.reviewboard.kde.org/r/104592/s/526/
Thanks,
Viranch Mehta
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
