mihir_Good Morning Aall :)06:31
Hargardfor those ubuntu developers out there am in college and ave decided to give it a go06:54
Hargardhow do i go about it  ??06:55
NimbleHargard, about what06:57
Nimbledeveloping for ubuntu?06:57
mihir_Nimble: Hi, Thank you for interest07:02
mihir_Nimble: Are you interested in developing?07:02
Nimblemihir_, yes, a bit07:03
Nimbleoriginally I came here because the instructions on using the messaging menu in the docs didn't seem to work07:03
mihir_Nimble: Okay, you can start doing resolving bugs..07:03
Nimblewell, why not07:04
NimbleI take it I should probably sign up for launchpad07:04
mihir_you can look visit this link http://developer.ubuntu.com/2013/07/join-the-ubuntu-touch-core-apps-development-team/07:04
mihir_Yup, you should do that first07:04
Nimblealright, will do.07:04
mihir_Nimble:  :)07:07
dholbachgood morning07:10
zsombioSoMoN: ping07:19
Nimblemihir_, will this work with my ubuntu single sign on?07:21
Nimbleand if I have one, will it not allow me to use my email address registered to that?07:21
mihir_Nimble: yes , you should use one email across the ubuntu :)07:24
Nimblealright, my single sign on worked07:24
Nimbleso that's good07:24
mihir_Nimble:  That's great it worked :)07:27
oSoMoNzsombi: pojng07:35
zsombioSoMoN: jenkins did the job now, you have the deb packages there :)07:36
oSoMoNzsombi: awesome, I’ll test right away07:36
mihir_dpm: Good Morning :)07:37
dpmmorning mihir_ and all07:44
mihir_dpm: Hi , I have proposed two MRs could you please review if you have time07:46
oSoMoNzsombi: do you know if the uitk’s autopilot tests can be run on devices?08:06
zsombibzoltan: ^08:07
bzoltanoSoMoN: It should be OK08:07
oSoMoNbzoltan, zsombi: I’m getting the following error when trying to run them on maguro: http://pastebin.ubuntu.com/5954209/08:09
zsombioSoMoN: bzoltan: no idea what can this be...08:10
oSoMoNalso getting an error on stdout about gcc not found…08:10
oSoMoNinstalled gcc and this error went away, but not the runtime error08:11
=== mpt_ is now known as mpt
nerochiarogusch: hi, i think i asked you already, but forgot the answer: can we find out if the currently active camera has light (torch/flash) support ?09:20
nerochiarogusch: also, how to find out the number of cameras on the current device09:20
guschnerochiaro: for the number of camera, you need to use the same class, that you use to switch cameras09:21
guschnerochiaro: http://qt-project.org/doc/qt-5.0/qtmultimedia/qvideodeviceselectorcontrol.html#deviceCount09:21
mihir_dpm: Ping !!09:22
guschnerochiaro: need to check the best way to get the flash information09:24
nerochiarogusch: ok. do you only have a galaxy nexus, right ?09:25
guschnerochiaro: and the N1009:25
nerochiarogusch: ok, but no n7, which apparently has only one camera09:25
nerochiarooSoMoN: timp: to store configuration info, are we currently using dconf or something else ?09:27
guschnerochiaro: nope09:27
oSoMoNnerochiaro: dunno09:28
zsombinerochiaro: we have an AP in the SDK to provide API for gsettings/dconf, so that will be the one you should use... when it will be ready09:36
guschnerochiaro: for the focus you can use this one http://qt-project.org/doc/qt-5.0/qtmultimedia/qml-qtmultimedia5-camerafocus.html#isFocusModeSupported-method09:37
nerochiarozsombi: so what should i do now ?09:37
nerochiarozsombi: i mean, in the meantime before it's ready ?09:37
zsombinerochiaro: yes, I got it :)09:37
zsombinerochiaro: well, do something :)09:38
nerochiarogusch: don't need it for focus ATM, only num of cameras and flash modes09:38
nerochiarogusch: but thanks09:38
nerochiarozsombi: ok, use whatever for now then when your stuff is ready use that. fair enough09:38
guschnerochiaro: for flash you'll need to use C++, and I'll have to update the plugin to do this: "Some camera devices may not have flash hardware, or may not be configurable. In that case, there will be no  QCameraFlashControl available."09:38
nerochiarogusch: that will work fine for me09:39
zsombinerochiaro: :) yep... We were trying to find out whether we could put under QSettings and provide QML binding to QSettings, but it ended up that Qt guys are planning something new and don't really want to see any backend added to QSettings anymore :/09:40
zsombinerochiaro: does the notes patch work? were you able to test it?09:43
nerochiarozsombi: what patch ? missing something here09:46
zsombi(12:32:26 PM) zsombi: nerochiaro: set propagateComposedEvents: true to the MA which activates the note card, also mouse.accepted = false in onClicked09:46
nerochiarozsombi: ah, let me try that09:47
nerochiarozsombi: in addition also propagateComposedEvents: true on the IMA, right ?09:48
zsombinerochiaro: yes, this is needed as the "inactive" card MA may be higher in the paint order, so that also needs to propagate the composed events09:49
nerochiarozsombi: ok, giving it a shot09:49
nerochiarozsombi: ok, seems to work. i'll submit an MR09:52
zsombinerochiaro: awesome!!! :)09:52
nerochiarozsombi: :909:52
nerochiarozsombi: btw, is it normal that on the latest image when i drag the launcher all the way across to reveal the apps lens, the dash is completely transparent and shows the running app underneath ?09:53
nerochiarozsombi: it's new in today's image09:53
zsombinerochiaro: no idea, don't have my device woken up yet...09:54
nerochiarozsombi: zzzzz09:56
zsombinerochiaro: I've set it on charger, but it is still dead09:57
nerochiarozsombi: shut it down before going to bed ;)09:59
zsombinerochiaro: I thought that it was, as when I pressed the power button it aint woke up... :)09:59
nerochiarozsombi: happens to me all the time10:02
nerochiarozsombi: i think we need a more reliable shutdown mechanism10:02
zsombinerochiaro: and a safety power-off, which turns off the device when the battery dries down to 3%, so a possible wake-up c an be done afterwards safely10:03
nerochiarozsombi: yeah10:06
AskUbuntuI want to create .dmg file from my source file on ubuntu so that I can distribute it on mac? | http://askubuntu.com/q/32924910:09
* zsombi lunch10:13
Mirvbzoltan: re: http://pad.lv/1202207 dpm e-mailed about... I remember some past hud discussion with Wellark, what was the end results? I thought it would be compiled in SDK PPA for older Ubuntus or something like that.10:43
ubot2`Launchpad bug 1202207 in Dropping Letters "12.04 install problems" [Undecided,New]10:43
nerochiarozsombi: https://code.launchpad.net/~amanzi-team/notes-app/notes-app-event-propagation/+merge/17871410:46
zsombinerochiaro: checking...10:46
nerochiarozsombi: haven't run the tests yet, i'll try that now on the device10:46
zsombinerochiaro: ok10:47
=== MacSlow is now known as MacSlow|lunch
=== gusch is now known as gusch|lunch
nik90|Officemhall119: ping11:05
nik90|Officeseb128: ping11:06
=== nik90|Office is now known as nik90|Lunch
seb128nik90|Lunch, context less ping = no pong usually (just saying)11:15
nik90|Lunchseb128: oh11:16
nik90|Lunchseb128: so I saw that in the system settings app, you guys show the timezone info "Europe/Amsterdam" etc..Is that just a mockup string? Or do you guys actually read the system timezone info?11:17
seb128nik90|Lunch, having "ping" without context is a trap, sometime you don't have time for a long discussion so you just default to ignore the ping to not be trapped11:17
seb128nik90|Lunch, we read the tz11:17
nik90|Lunchseb128: How do you do that? Did you create a custom C++ plugin to read the tzdata file?11:17
nik90|Lunchseb128: I need that same info for the clock app. However since the clock app is all javascript and qml, I am unable to get the timezone ID info. So before I start writing my own custom C++ plugin to do that, I wanted to know if other apps like the system app did that already to prevent code duplication unnecessarily.11:20
seb128nik90|Lunch, yes, http://bazaar.launchpad.net/~system-settings-touch/ubuntu-system-settings/trunk/view/head:/plugins/time-date/timedate.cpp11:20
mihir_dpm:  ping !!11:20
seb128nik90|Lunch, that seems like something useful enough, that we should have a sdk api for it imho11:21
nik90|Lunchseb128: I agree. However I am not really sure when that would be implemented since all sdk devs are quite busy. And I need this info asap :(11:22
seb128nik90|Lunch, right, you probably better copy what we are doing meanwhile11:22
nik90|Lunchseb128: yeah. thnx for getting the info for me11:23
mihir_mhall119: Ping !!11:23
seb128nik90|Lunch, hum, maybe https://github.com/nemomobile/nemo-qml-plugin-time/tree/master/src would be an option as well11:24
seb128nik90|Lunch, their current backend is for meego though11:27
kalikianaseb128: be sure to file a bug with the sdk that points at your branch, it might speed up development once it happens ;-)11:32
seb128kalikiana, what project is the right one for those sort of apis?11:33
nerochiarozsombi: tests are also successful on device (on top of jenkins being happy about them). maybe i should add another ap test for this specific case11:33
zsombinerochiaro: excellent! an extra test is always welcome11:34
zsombinerochiaro alos, the code looks right, so I'll approve from my side, but let's wait till timp approves it11:34
nerochiarozsombi: ok11:35
oSoMoNhey gusch|lunch, when you get back from lunch, could you please review https://code.launchpad.net/~osomon/webbrowser-app/simplify-historydomainlistmodel/+merge/178687 ? I already ran the autopilot tests on maguro, they all pass11:39
=== gusch|lunch is now known as gusch
guschoSoMoN: what a timing ;) - I review ...11:41
oSoMoNthanks :)11:41
kalikianaseb128: just file it in the sdk, it's easy to move, and it'd depend on what other api it might relate to. just timezone data won't be its own component11:50
kalikianait might also coincide with zsombi's alarm stuff11:51
guschoSoMoN: approved11:57
oSoMoNgusch: awesome, thanks, I’ve got a couple more MRs in the pipe, waiting to run autopilot tests on device before requesting a review11:58
guschoSoMoN: keep them going ;)11:59
zsombikalikiana: seb128: hehe, the github branch you've pointed actually connects to timed, the one we were suggesting to port to Ubuntu, as that handles all the time related stuff (including proper NITZ support with all its exceptions) + alarms...12:00
zsombibut seems we will do a different adaptation and have a different approach for that...12:00
=== nik90|Lunch is now known as nik90|Office
nik90|Officezsombi: Should I go about writing my own C++ plugin to get system timezone ID or will this come as part of the alarm package?12:02
nik90|Officezsombi: I need to make a decision depending on when a suitable API might land in the sdk for it12:02
=== dholbach_ is now known as dholbach
zsombinik90|Office this may not come as part of the alarm API as this is more related to time management, but we may create a separate API for Time management. I say may, as currently we don't have any API planned for that. But you could always contribute to the SDK by offering to do such an API ;)12:04
oSoMoNgusch: next one: https://code.launchpad.net/~osomon/webbrowser-app/blank-thumbnails/+merge/17871612:04
guschoSoMoN: ok12:04
nik90|Officezsombi: I would love to help but I am learning qml/c++ as I go. So what I implement may not have sdk standards and would rather be suited for an app instead.12:06
nik90|OfficeThat said I will give it a shot and see where it leads me12:06
zsombinik90|Office don't be afraid, we can help on that! :)12:06
nik90|Officezsombi: I will let you know when I have a working branch12:07
zsombinik90|Office cool!! thx!12:08
=== MacSlow|lunch is now known as MacSlow
nerochiaroom26er: is there any way in autopilot tests to wait for an animation to finish ?12:18
nerochiarooSoMoN: maybe you know that too ? ^12:18
om26ernerochiaro, most of the times, object.animating is what you need to assert12:19
nerochiaroom26er: oSoMoN: the reason i ask is because i need the animation to finish to get the new geometry of the object so that i can click in the right place for my test12:19
nik90|Officeom26er: I reviewed your autopilot branch fix for clock app.12:19
om26erthere are also cases where object.moving is used as well, but that's mostly when the page stack is being moved etc.12:20
nerochiaroom26er: object.animating does not exist12:20
oSoMoNnerochiaro: you may need to add an animating property to the object, and wait for it to become false12:20
oSoMoNnerochiaro: look for example at the SDK’s Panel implementation, it has an 'animating' property12:21
nik90|Officeom26er: I see that we continue to use our own custom toolbar emulator function. Should we convert that to use elopio's emulator function now or in a later commit?12:21
nerochiarooSoMoN: adding properties only for the sake of tests does not seem very clean12:21
om26ernik90|Office, I think doing that in an other branch will be better, also I will fix the problems you highlighted today12:21
oSoMoNnerochiaro: you asked for a solution, I’m sharing the only solution I know12:22
nik90|Officeom26er: okay12:22
om26ernerochiaro, which type of object is that ?12:22
nerochiarooSoMoN: sure, i wasn't complaining for the sake of complaining, i was trying to get a discussion running. it's an ok workaround if nothing else is available.12:22
oSoMoNgusch: next MR: https://code.launchpad.net/~osomon/webbrowser-app/cosmetics/+merge/17872512:23
om26erIn cases I have used .animating or .moving and also 'swipeState' depending on the type of object I am interacting with12:23
om26ernerochiaro, ^12:23
nerochiaroom26er: it's one of the notes in notes-app. i need to wait for the expanding animation to finish before it settles on a stable geometry12:23
nerochiaroom26er: so i can continue with the rest of the test12:23
nerochiaroom26er: oh, but wait, i think i have a property that represents the final height, and i animate towards that. so i can just check if height eventually equals final_height and that should work12:24
oSoMoNnerochiaro: you could probably write a custom matcher that waits for the height to be stable, i.e. not changing during a certain interval of time, but that sounds way more hackish than the other solution12:24
oSoMoNnerochiaro: yep, height == final_height looks like the cleanest solution so far12:24
nerochiarooSoMoN: ok let's try that12:24
om26ernerochiaro, you could tests the state of the NoteItem, it has collapsed and expanded12:26
nerochiaroom26er: was my first idea, but it doesn't work because the state changes immediately and then the animation is triggered, as far as I can see12:27
om26ernerochiaro, there is also 'isExpanded'12:28
nerochiaroom26er: same problem with that12:28
nerochiarooSoMoN: om26er: the height test works, i'll use that. thanks for the input12:28
guschoSoMoN: cosmetics approved12:34
guschoSoMoN: thumbnails approved as well12:40
=== dholbach_ is now known as dholbach
oSoMoNgusch: thanks!12:47
nerochiarozsombi: added more autopilot tests to https://code.launchpad.net/~amanzi-team/notes-app/notes-app-event-propagation/+merge/17871412:48
om26ernerochiaro, gusch if I turn on autopilot tests for camera-app for each merge proposal do you mind ?12:53
om26erI'll keep fixing if we face any blockers along the way12:53
=== _salem is now known as salem_
om26er*On Maguro and Mako I meant12:54
nerochiaroom26er: well, what's the current status ? will it fail if you do turn it on ?12:56
om26ernerochiaro, currently they are passing as seen on daily image testing12:56
guschom26er: is it possible to enable the camera-app autopilot tests for the qtubuntu-camera as well?12:56
om26ergusch, it should be, I can look into that. never tried qtubuntu-camera suite12:57
guschom26er: would be cool if you could try to do that at one point (no hurry)12:58
om26ergusch, sure12:59
seb128zsombi, great, is there a bug/blueprint/somewhere I can see what's going on about this timed?13:03
nerochiaroom26er: then go ahead with that ,i have no problem13:12
om26ernerochiaro, ok, thanks13:12
mihir_boiko: could help me regarding this bug ?https://bugs.launchpad.net/ubuntu-calculator-app/+bug/120502013:48
ubot2`Ubuntu bug 1205020 in Ubuntu UI Toolkit "functionality for copying result in clipboard" [Undecided,Confirmed]13:48
mihir_Still design team needs to approve it13:48
boikomihir_: I think for this one what you need to do right now is to wait for something to be provided by the UI Toolkit team13:49
mihir_boiko: Okay thank you :)13:50
=== _salem is now known as salem_
nerochiarozsombi: can we top approve https://code.launchpad.net/~amanzi-team/notes-app/notes-app-event-propagation/+merge/178714 as jenkins likes my new tests and they pass on the device too ?14:36
iBelievemhall119, ping14:56
mhall119iBelieve: pong15:00
iBelievemhall119, I've got a question about merge requests for File Manager15:00
iBelievemhall119, I was reading the backlog for the last meeting (I wasn't their), and Arto  said he doesn't have much time for development except for responding to emails. I've had a merge request that's been waiting for a long time. Should I have somebody else, like you or popey, review it? Or should I just approve it myself?15:02
mhall119iBelieve: I'll take a look, can you link me to the MP?15:05
iBelievemhall119, I'm going to make a few changes to the old merge request (https://code.launchpad.net/~mdspencer/ubuntu-filemanager-app/advanced-options), but you could take a look at the new one (https://code.launchpad.net/~mdspencer/ubuntu-filemanager-app/sidebar/+merge/178774)15:05
mhall119iBelieve: I'm in a meeting atm, but I'll look at them after15:06
iBelievemhall119, thanks. Also, I just checked on the first merge request that I linked (advanced-options), and that's ready to be reviewed, I'm not going to make any changes.15:07
guschnerochiaro: ping15:09
nerochiarogusch: hi15:10
guschnerochiaro: about detecting flash support15:10
guschnerochiaro: you should check it a bit different15:10
guschnerochiaro: it's not possible to dynamically add/remove the C++ flashControl object (so that QML detects it)15:11
guschnerochiaro: so get in C++ the QCameraFlashControl15:12
guschnerochiaro: and there use bool QCameraFlashControl::isFlashModeSupported(QCameraExposure::FlashModes mode)15:12
guschnerochiaro: as mode use QCameraExposure::FlashOn - if that is supported, than a flash exists ...15:12
guschnerochiaro: is that ok?15:13
nerochiarogusch: ok, i guess that doesn't tell me if a torch mode exists, i have to check that separately. other than that it works ok for me15:14
guschnerochiaro: yep - that should be checked separately15:14
guschnerochiaro: you need to have a running camera to get the correct values btw.15:17
nerochiarogusch: running as in initialized properly15:20
guschnerochiaro: yep15:20
zsombiseb128: we don't have blueprint for timed as we didn't do anything from SDK side. All we have is a BP for Alarm API, but that's not really related to the time functionality you have pasted from github15:26
seb128zsombi, I looked at timed, it doesn't seem to do stuff like system tz handling (or I looked at the wrong place for it)15:27
zsombinerochiaro: I'm affraid to top-approve that as may introduce regressions without my MR approved.15:27
zsombiseb128 it does, it handles alarms and also has NITZ support.15:28
iBelievetimp, ping15:28
guschnerochiaro: can you review this? https://code.launchpad.net/~schwann/qtubuntu-camera/qtcamera-vp-flash/+merge/17878415:29
seb128zsombi, NITZ is not system timezone though, it wouldn't work on a tablet or desktop15:29
zsombi seb128 it does if the tablet has GSM module in it. NITZ however helps you keep the time zone in sync when traveling or when the daylight saving starts/ends. So that one we also need to check, perhaps not in our time backend in SDK, but somewhere in the system. So far I know there is some work done around NITZ somewhere else in the stack...15:32
zsombiseb128: so whatever is needed right now for the clock, we don't have any BP/API planned. But we definitely need one15:32
seb128zsombi, right, I'm not saying NITZ it's not useful, it's only a piece of the puzzle though15:32
timpiBelieve: pong15:32
zsombiseb128: +115:32
iBelievetimp, could you explain the reasoning for marking LP #1202469 as Invalid?15:33
ubot2`Launchpad bug 1202469 in Ubuntu UI Toolkit "Actions should be hidable " [Undecided,Invalid] https://launchpad.net/bugs/120246915:33
iBelievetimp, I'm working on advanced options for File Manager and need a way to make actions hidden.15:34
timpiBelieve: hold on, let me see15:36
zsombiseb128: timed is mostly there to handle and schedule alarms, keep those in sync with the system time changes + NITZ changes, and also handles when to play alarm sound (there are quite many exceptions, like when in a call, when multimedia plays, etc)15:36
seb128zsombi, ok, it seems a bit orthogonal to the set manually time/date/timezone, use ntp features (which are more desktopish)15:37
nerochiarozsombi: ping15:44
zsombinerochiaro: pong15:44
nerochiarozsombi: can we top approve https://code.launchpad.net/~amanzi-team/notes-app/notes-app-event-propagation/+merge/178714 as jenkins likes my new tests and they pass on the device too ?15:44
zsombinerochiaro: so it doesn't bother it with the MA event propagation?15:45
nerochiarozsombi: what do you mean ?15:46
timpiBelieve: where do you use the Action? In the toolbar?15:47
zsombinerochiaro: these changes you did were because the IMA changes. Do these changes work without any problems with the SDK from archive?15:47
iBelievetimp, I was planning on having the Actions in an ActionSelectionPopover. For example, I was going to have an "Open in Terminal" action that is only shown when the user has set a setting that shows advanced features.15:48
nerochiarozsombi: ah, i don't know. i tested them with the sdk from your branch15:48
nerochiarozsombi: but it seems jenkins thinks they are ok with the sdk from archive as well15:48
zsombinerochiaro: ok, so let's approve this, then timp can approve IMA MR afterwards15:49
timpiBelieve: for the toolbar we replaced the Actions by a visual representation of actions (ToolbarButton) that has the visible property now15:50
timpiBelieve: I don't think we have a visual representation for the actions in the ActionSelectionPopover yet.15:51
timpzsombi: you wrote the ActionSelectionPopover? Is that still a recommended component to use?15:51
timpiBelieve: I'm reading the docs for ActionSelectionPopover now, I'm not familiar with it15:52
zsombitimp: why not? it gets a list of actions and displays them. However can be modified to work better with the new Action stuff15:52
nerochiarozsombi: ok cool15:53
nerochiarozsombi: thanks15:53
iBelievetimp, why are you saying that the visible property should be part of the visual representation of an Action (such as a ToolBarButton)? To me, it seems like it should be part of the Action itself.15:53
timpiBelieve: we don't have our "own" Actions in the SDK any more, but we use them from Unity.Actions. Those actions are integrated with HUD and will be with the launcher15:56
timpiBelieve: we had some discussions about the "visible" property, and I used to share your opinion.15:57
timpiBelieve: probably we'll need a visual representation for actions in an actionselectionpopover15:58
iBelievetimp, so why isn't there a visible property anymore? I'm just curious :)15:59
timpiBelieve: well, for the HUD, the idea is that it can show the user all the possible available actions16:00
timpiBelieve: and for other uses (toolbar), we thought that the ActionItem/ToolbarButton would take care of it16:01
timpiBelieve: your case still needs some thoughts16:02
timpWellark: ^ do you have any suggestions for the visibility of actions in an ActionSelectionPopover?16:02
iBelievetimp, that makes sense, somewhat, though it seems like Actions could still have a visible property and be shown no matter what in the HUD.16:02
iBelievetimp, anyway, what should I do for an ActionSelectionPopover? Is there a way to do what I want now, or do I need to wait for support in it?16:03
timpiBelieve: you can specify the delegate of the ActionSelectionPopover, of that you can set the visible property.16:05
timpiBelieve: if you use action.enabled, you can say visible: action.enabled in the delegate16:05
timpzsombi: in the docs for ActionSelectionPopover, it says:           text: action.text16:06
timpzsombi: shouldn't that be text: item.text ?16:06
iBelievetimp, that might work, I'll try it. Thanks for your explanation of the reasoning behind the bug and for help with a solution16:06
timpiBelieve: thanks for making me doubt our previous decision to remove visible ;)16:07
iBelievetimp, :)16:07
zsombitimp: no, because that delegate is the component which is used to show the action, and the "action" itself is the property from the Standard16:08
timpzsombi: ah yes, ListItem.Standard has an action property. Then setting the text here does nothing (it is action.text by default)16:09
zsombitimp: huhh??16:09
Wellarktimp, iBelieve: so, the rationale here is that Action is separated from it's visualization16:10
timpzsombi: what huh?16:10
zsombitimp: I was looking after this action property :)16:10
zsombitimp: but that has been moved to AbstractButton16:10
timpzsombi: no16:11
timpzsombi: to ActionItem, the parent of AbstractButton :)16:11
zsombitimp: actually AbstractButton is derived from ActionItem :)16:11
zsombitimp: right16:11
timpzsombi: many of our components are now derived from ActionItem16:11
zsombitimp: and that's right!16:11
WellarkAction is an abstract entity describing the functionality the application can do and when it's visualized you always assosiate the action with some form of delegate16:12
Wellarkthis allows a single action to be shared or contributed between different UI and shell elements and each delegate has a separate visibility property16:15
Wellarkor optional visibility propertyy if it makes sense16:15
iBelieveWellark, timp, so is there an easy way to control the visibility of an Action in an ActionSelectionPopover without a custom delegate? timp suggested visible: action.enabled, but I think that would require a custom delegate?16:19
WellarkiBelieve: I might be using the term delegate ambiguously16:20
WellarkI haven't looked in to ActionSelectionPopover yet16:20
timpiBelieve: does it work if you add "delegate.visible: action.enabled" to set the visible property of the standard delegate?16:20
WellarkiBelieve: that ^  sounds reasonable16:20
timpWellark: ActionSelectionPopover has a property Component delegate which is used in the Repeater that visualizes the Actions16:21
Wellarkif your action is not enabled then it's invisible16:21
iBelieveWellark, timp: QML won't let me do sub-properties like that :(16:21
iBelieveWellark, timp: maybe it would, but it doesn't show that property since it is a Component, not an Empty16:23
=== greyback_ is now known as greyback|food
timpiBelieve: ah yes, I think you're right16:28
iBelievetimp, I tried using another property like property Empty delegate2: delegate, and then used delegate2.visible, but then QML said Invalid property nesting16:28
timpiBelieve: actually the delegate is very simple: http://pastebin.ubuntu.com/5955564/16:30
iBelievetimp, I suppose I'll just copy it if there is no better way, but I never like to duplicate code, especially if the original is in the SDK and subject to change.16:31
timpiBelieve: I agree, it is better not to copy code. But like this you can continue working on it now16:32
iBelievetimp, right, that's what I'll do. Thanks for all the help!16:33
=== greyback|food is now known as greyback
akakistI cannot enter payment information into profile while trying to submit app20:32
iBelieveballoons, ping21:45
balloonsiBelieve, pong.. I wanted to chat with you :-)21:46
iBelieveballoons, about LP #1208999?21:46
ubot2`Launchpad bug 1208999 in Ubuntu File Manager App "Cannot paste a file on desktop or device" [Undecided,New] https://launchpad.net/bugs/120899921:46
balloonsiBelieve, indeed. that and https://bugs.launchpad.net/autopilot-qt/+bug/1209004 are the only failing tests on the device21:46
ubot2`Ubuntu bug 1209004 in Autopilot Qt Support "Process Manager tries to call Xlib on phablet devices" [Undecided,New]21:46
iBelieveballoons, so does copying and pasting work on the desktop?21:47
balloonsiBelieve, no.. I can't get it to work at all21:47
balloonsno matter what device I use :-)21:47
iBelieveballoons, does the Actions popover popup in the center of the app, or does it popup from the Actions button?21:48
balloonscenter of app on desktop, causing issues21:48
balloonsfrom the popover on the device I think..21:48
balloonsit's odd.. it's not the same failure depending on how you do it21:48
iBelieveballoons, the reason it centers is because of how the popover gets opened in code. I fixed that, but I'm getting errors about the copy and cut functions not existing.21:49
iBelieveballoons, can you try lp:~mdspencer/ubuntu-filemanager-app/sidebar and see if that works?21:49
iBelieveballoons, though wait a sec until I push more changes21:50
balloonsiBelieve, no rush, as I'm working on the clock app now and probably won't be back to file manager until tomorrow21:50
balloonsI just wanted to make you aware and see if you knew anything about it. Sounds like you do which is good :-)21:51
iBelieveballoons, I was already working on fixing the action selection popovers among other things, so thanks for noticing the copying problem. I uploaded the branch if you wanted to take a look at it, but it still isn't working for me.21:53
iBelieveballoons, I fixed the copying/cutting of files in the UI, but now those tests and others are broken, so I need to fix those.22:08
balloonsiBelieve, heh, always something eh? Good though, I'm glad to hear22:11
balloonsthoughts on fixing the open_file test to not depend on processmanager?22:11
balloonsiBelieve, ^^22:11
iBelieveballoons, I haven't seen the new tests yet, I'll take a look though22:13
iBelieveballoons, that's cool how that test works. Is there a reason why it should not depend on ProcessManager?22:14
balloonsiBelieve, https://bugs.launchpad.net/autopilot-qt/+bug/120900422:15
ubot2`Ubuntu bug 1209004 in Autopilot Qt Support "Process Manager tries to call Xlib on phablet devices" [Undecided,New]22:15
balloonsit blows up on the device, but it's not your fault.. however, I think it can be done without processmanager..22:15
iBelieveballoons, ah, that was the other bug you mentioned before. I wasn't the one who wrote that test, elopio did, so I'm not familiar with how it works, but I'll try to think of some way to do it.22:17
iBelieveballoons, I've got a question on how I should write the autopilot tests for my new changes, if you have time.22:18
balloonssure, go for it.. and yea, I know you didn't write it. I would leave it for now till we get feedback on whether or not we should be using processmanager for it I'd suspect22:19
iBelieveballoons, I've added a sidebar (like in Nautilus) that is visible when the app is wide enough (here is the screenshot: http://i.imgur.com/GZOVaLf.jpg). This basically replaces the Places menu in the toolbar. I've also changed the default size to 100 gu, so it looks good on the desktop, assuming the phone will correctly size it to 50 gu.22:22
* balloons wonders what it will look like on my tablet22:22
iBelieveballoons, now my question is, how should I write the tests for the new layout? There needs to be tests for both the tablet/desktop mode and phone mode. Should these be in one test, or in separate tests?22:23
iBelieveballoons, and is it possible to change QML properties from autopilot so I can put the app into phone mode to test it?22:24
balloonsiBelieve, ahh yes.. Well you could put it into the same test and detect the proper size then act accordinly22:24
balloonsand yea, forcing a particular size is interesting too22:24
iBelieveballoons, so should only do one test depending on what device it is running on, or should I force both sizes to test?22:25
balloonsI'm concerned with how a device will respond if you force the size. On the desktop, sure should be fine. On the phone, well it can't fit a tablet size :-)22:25
iBelieveballoons, good point, I hadn't thought about that. So I guess I'll just write the tests to test for the particular configuration the app is run in?22:26
balloonsyes, I would do that. As to wanting to hack on the different size and test them, I suppose the desktop test could run both..22:27
iBelieveballoons, for now, I'll just run the tests based on the initial size. Thanks for the help!22:29
iBelieveballoons, ping23:04
balloonsiBelieve, pong23:05
iBelieveballoons, so one of the reasons the copy autopilot test fails is that there is no wait for the destiniation directory to be opened. I'd like to be able to incorporate that into the FolderListDelegate emulator, but it seems that assertThat isn't available outside of the test case. Any suggestions?23:07
iBelieveballoons, here is a snippet of what I'm trying to do: http://paste.ubuntu.com/5956870/. First I tried calling assertthat on self, then on AutopilotTestCase, but AutopilotTestCase requires an instance.23:08
iBelieveballoons, I need to go, I'll try and see if I can find some way to do it, otherwise I'll ask again later.23:32
balloonsiBelieve, yes, in short, don't use asserts outside of testcases. That said, I'm not sure I get the hangup here23:33
iBelieveballoons, what do you mean "I'm not sure I get the hangup here"23:33
balloonsI mean I'm knee deep in ubuntu clock, and losing my mind as it's getting late here :-)23:38
balloonsin other words, I can't be of much more help atm ;-p23:38

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!