| === chriadam is now known as chriadam|away | ||
| === mpt_ is now known as mpt | ||
| DanChapman | Good Morning all o/ | 08:01 |
|---|---|---|
| mihir | DanChapman: morning :) | 08:05 |
| rpadovani | Hey all! In a UbuntuListView, in the PullToRefresh component, how can I disable the ActivityIndicator? I want to see only the text (Pull to refresh) | 08:05 |
| rpadovani | zsombi, ^^ | 08:05 |
| zsombi | rpadovani: I'm affraid you cannot | 08:10 |
| zsombi | rpadovani: why would you need that? | 08:11 |
| rpadovani | zsombi, because I already have a different activy indicator: I implemented a bouncing bar like in scopes when the function I call to refresh is running | 08:12 |
| rpadovani | *activity | 08:12 |
| zsombi | rpadovani: well, this behaviour of PullToRefresh is the default. the ActivityIndicator is the default, and will thus be used as is. If you want a different one, then you create a theme, and then do the style for the ActivityIndicator, then you will have consistency there. | 08:15 |
| zsombi | rpadovani: beside, the indicator is shown for a pretty short time, so if your model is fast enough, teh indicator shouldn't be for too long there :) | 08:15 |
| rpadovani | zsombi, that is how it works now in reminders, but dpm asked to use default components :-) | 08:15 |
| zsombi | rpadovani: well, then PullToRefresh works as dpm requested, uses default components | 08:16 |
| rpadovani | zsombi, sorry, I mean that now we have a custom implementation of PullToRefresh and our activityindicator, but dpm asked to switch to default pulltorefresh | 08:17 |
| rpadovani | dpm, this is a ping, when you have time please :-) | 08:17 |
| zsombi | rpadovani: so then you have to switch your activity indicator as well to default :) | 08:18 |
| rpadovani | zsombi, this is a good idea, indeed :D | 08:18 |
| zsombi | rpadovani: the only way to get your indicator to be shown in PullToRefresh is the theme. | 08:18 |
| rpadovani | mzanetti, ^^ | 08:19 |
| rpadovani | zsombi, ok, thanks :-) | 08:19 |
| zsombi | rpadovani: unless we expose a property to configure the animation there... but that takes time | 08:19 |
| mzanetti | rpadovani: dunno... not sure if its worth to go through the theming hazzle | 08:27 |
| mzanetti | rpadovani: but yeah, our pulldown component needs some love :) | 08:27 |
| mzanetti | it triggers too easily | 08:28 |
| rpadovani | mzanetti, no, definitely not. My question is, do we switch both pulldown and activityIndicator to default one? I love our activiy indicator :D | 08:28 |
| mzanetti | yeah, me too | 08:28 |
| mzanetti | there was a reason after all not to use the default one | 08:29 |
| rpadovani | mzanetti, ok, then maybe I can try to improve our pulldown component looking to the sdk one | 08:37 |
| rpadovani | I'll talk also with dpm to hear what he thinks | 08:37 |
| mzanetti | yep | 08:37 |
| mzanetti | zsombi: what does the activityindicator in the pulldown actually wait for? | 08:38 |
| zsombi | zbenjamin: waits for the refresh to be completed, rt()m | 08:39 |
| zsombi | mzanetti: ^ that was 4 U , sorry zbenjamin :) | 08:39 |
| zbenjamin | zsombi: np :) | 08:39 |
| zsombi | mzanetti: in order to get the animation vanish, you need to inform teh component about completion | 08:39 |
| vitimiti | Hi | 08:40 |
| rpadovani | mzanetti, in our case refreshing: notes.loading | 08:41 |
| mzanetti | rpadovani: did you try to keep "refreshing" property to false all the time? | 08:41 |
| mzanetti | rpadovani: well, you don't have to wire it up I guess, no? | 08:42 |
| rpadovani | mzanetti, let my try | 08:42 |
| rpadovani | mzanetti, doesn't work, when you pull down it is set to true, so it will be visible forever | 08:43 |
| mzanetti | rpadovani: try this: Binding { target: pullToRefresh; property: "refreshing"; value: false} | 08:44 |
| rpadovani | mzanetti, nope | 08:46 |
| mzanetti | ok then... I guess we should report a bug to the pulldown thing then... | 08:47 |
| rpadovani | mzanetti, do you want I push in a ~reminders-app branch so you can do some tries? | 08:47 |
| mzanetti | the fact that it sets the property to true itself has nasty side effects in other ways too... will break bindings | 08:48 |
| mzanetti | zsombi: ^ | 08:48 |
| mzanetti | would you agree? | 08:48 |
| zsombi | mzanetti: the property is set to true with a binding afaik | 08:49 |
| mzanetti | zsombi: but then you need to set it to false using java script | 08:49 |
| zsombi | mzanetti: well, if you do it like in the documentation, it won't require JS | 08:50 |
| zsombi | mzanetti: rpadovani: like refreshing: rssFeed.status === XmlListModel.Loading | 08:50 |
| mzanetti | zsombi: but how come it sets refreshing to true even if you do: refreshing: false then? (at least according to rpadovani) | 08:51 |
| zsombi | mzanetti: well, dunno that tbh... | 08:52 |
| zsombi | mzanetti: the sample we have in UITK gallery drives the refreshing, and works like charm... | 08:52 |
| JamesTait | Good morning all; happy Monday, and happy Yorkshire Pudding Day! :-D | 08:53 |
| zsombi | mzanetti: rpadovani: I've dbl checked: the refreshing property is not altered by the PullToRefresh component... | 08:54 |
| rpadovani | mzanetti, zsombi wait, I did a mistake: I thought that the animation was managed by refreshing property, but instead refresh() signal waits refreshing to be set to false | 08:55 |
| rpadovani | zsombi, indeed | 08:55 |
| mzanetti | aha :) | 08:55 |
| zsombi | rpadovani: happens ;) | 08:56 |
| rpadovani | mzanetti, zsombi so I can minimize the animation doing onRefresh: {refreshing=true;refreshing=false} | 08:57 |
| zsombi | rpadovani: well, you can, but why would you? I mean if the model refresh takes time, it would be wise to show sthing while teh new data arrives, right? | 08:57 |
| rpadovani | zsombi, because we want our activity indicator and default pulldown because works better than our one. It's a very low priority task, so don't wast time on this :-) | 08:59 |
| mzanetti | zsombi: the thing is that the refresh can take a long time in our case. but while its refreshing you can still use the list | 08:59 |
| mzanetti | zsombi: that's why we decided to go with a activitybar like the browser or the dash | 08:59 |
| nik90 | dpm: hey, can you take a look at https://bugs.launchpad.net/ubuntu-clock-app/+bug/1380248 and suggest some solutions to it? If it really requires UI String change, I can get them approved by the designer then. | 08:59 |
| ubot5 | Ubuntu bug 1380248 in Ubuntu Clock App "Alarm Settings: "%1m" can't be translatable in the right way" [High,Confirmed] | 08:59 |
| zsombi | mzanetti: well, PullToRefresh does not forbid you to use the list... if the list has data, you can still scroll it... | 09:00 |
| mzanetti | rpadovani: but I still don't understand... why do you even set refreshing to true? | 09:00 |
| mzanetti | rpadovani: can't you just leave it to false? | 09:00 |
| rpadovani | mzanetti, no, because as I understood (maybe I'm wrong) refresh() waits refreshing changes to false, if is already false it doesn't see it | 09:01 |
| rpadovani | I'm sure refreshing: false doesn't work | 09:01 |
| mzanetti | doesn't make any sense to me.. push your code please | 09:02 |
| rpadovani | mzanetti, lp:~rpadovani/reminders-app/PullToRefresh I modified only NotesPage.qml, hope I'm not doing some embarassing error :/ | 09:05 |
| mzanetti | lets see :) | 09:06 |
| rpadovani | brb | 09:10 |
| mzanetti | rpadovani: yeah, please file a bug. the activityindicator becomes visible even if refreshing is still set to false | 09:15 |
| rpadovani | mzanetti, I'm happy I didn't unlearn to code :D Bug #1380541 | 09:30 |
| ubot5 | bug 1380541 in Ubuntu UI Toolkit "[PullToRefresh] the ActivityIndicator becomes visible even if refreshing is set to false" [Undecided,New] https://launchpad.net/bugs/1380541 | 09:30 |
| tvoss | Orphis, ping | 09:30 |
| Orphis | tvoss: Pong | 09:30 |
| mihir | popey: can you confirm this bug on Calnedar, create an event with personal calendar, edit event and then try to change calendar to like your google calendar, it won't let you change the calendar. | 09:32 |
| mihir | popey: https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1380539 | 09:36 |
| ubot5 | Ubuntu bug 1380539 in Ubuntu Calendar App "Changing an event's calendar fails without user visible error" [High,Confirmed] | 09:36 |
| popey | mihir: will do | 09:38 |
| mihir | though i have checked , but want double check :) | 09:38 |
| rpadovani | mihir, popey confirmed | 09:39 |
| mihir | rpadovani: thanks :) | 09:39 |
| popey | that was easy | 09:39 |
| popey | thanks rpadovani | 09:39 |
| zsombi | nik90: did you have bugs for alarm model updates for teh toolkit? I cannot find those... | 09:48 |
| nik90 | zsombi: Regarding the model refresh slot? | 09:49 |
| zsombi | nik90: no, the one you needed(?) when you add a new alarm, which refreshes teh whoel alarm list | 09:50 |
| nik90 | zsombi: I think that marked as fix released when you fixed the whole alarm list from being refreshed after updating an alarm. but let me try to find it. | 09:51 |
| zsombi | nik90: how about when one is removed? | 09:51 |
| zsombi | nik90: I think we had some need for those as well, right? | 09:51 |
| nik90 | zsombi: I don't think that was removed...tbh I haven't checked that case | 09:52 |
| nik90 | s/removed/reported | 09:52 |
| zsombi | nik90: not removed, but when an alarm is removed, that refreshes teh whole model, right? | 09:52 |
| zsombi | ok, we agreed you will report one :) | 09:52 |
| zsombi | nik90: so pls, remort all alarm bugs so I can work on those... | 09:53 |
| zsombi | nik90: or at least justify why the work on alarms is proceeding... | 09:53 |
| nik90 | zsombi: I am a bit slightly confused :)...so are we talking about the use case that we discussed few days back? | 09:53 |
| nik90 | or th eonle really really long time ago? | 09:53 |
| zsombi | nik90: few weeks back | 09:53 |
| zsombi | nik90: when we fixed the alarm update | 09:53 |
| nik90 | ok I recollect now | 09:53 |
| zsombi | nik90: there you realized that removing an alarm also refreshes all teh alarms | 09:53 |
| nik90 | let me find the bug, 1 sec | 09:53 |
| nik90 | zsombi: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1359112 | 09:58 |
| ubot5 | Ubuntu bug 1359112 in Ubuntu UI Toolkit "Alarm model is updated after every change causing all the listview items to be repainted" [High,Fix released] | 09:58 |
| nik90 | zsombi: I haven't mentioned the delete alarm case although I can reproduce it | 09:59 |
| zsombi | nik90: perhaps then I did.... never the less, it is a valid one | 09:59 |
| nik90 | zsombi: give me a sec to report a new bug | 10:00 |
| zsombi | nik90: and if you have that reset() request as well, push that too | 10:00 |
| nik90 | zsombi: yup doing it atm | 10:00 |
| nik90 | zsombi: bug 1380553 | 10:04 |
| ubot5 | bug 1380553 in Ubuntu UI Toolkit "Alarm Model is refreshed after creating/deleting an alarm" [Undecided,New] https://launchpad.net/bugs/1380553 | 10:04 |
| justCarakas | help, I'm starting to type json when I wanne type jason | 10:07 |
| justCarakas | is there a cure ? :p | 10:07 |
| zsombi | nik90: the bug you correlated it to is tagged... shouldn't this one be as well? | 10:14 |
| nik90 | zsombi: I thought its you who tags bugs as rtm priority or not :) | 10:15 |
| nik90 | well your team | 10:15 |
| zsombi | nik90: not really... none of us should | 10:15 |
| zsombi | nik90: at least not with that tag... | 10:16 |
| nik90 | zsombi: next bug 1380559 | 10:23 |
| ubot5 | bug 1380559 in Ubuntu UI Toolkit "[API Request] Alarms API should expose a slot Update() to allow app devs to manually refresh the model" [Undecided,New] https://launchpad.net/bugs/1380559 | 10:24 |
| zsombi | nik90: I guess this is pretty critical, right? | 10:25 |
| nik90 | yes | 10:26 |
| zsombi | does it affect any rtm functionality 4 u? | 10:26 |
| nik90 | zsombi: yes since when a repeating alarm is triggered, the clock app bottom edge reads "No Active Alarms" since the alarm date has passed. | 10:27 |
| zsombi | nik90: ok, dpm U there? | 10:28 |
| zsombi | nik90: eventually you could tag it as well... | 10:28 |
| nik90 | zsombi: should I tag both of those bugs now ? | 10:29 |
| zsombi | nik90: yes | 10:29 |
| dpm | zsombi, I will be in about 30 mins. Feel free to ping and I'll try to answer any questions | 10:31 |
| dpm | nik90, would you mind pinging me again with the link of what you wanted me to look at earlier on today? | 10:31 |
| nik90 | dpm: sure, https://bugs.launchpad.net/ubuntu-clock-app/+bug/1380248 | 10:32 |
| ubot5 | Ubuntu bug 1380248 in Ubuntu Clock App "Alarm Settings: "%1m" can't be translatable in the right way" [High,Confirmed] | 10:32 |
| snakeleg | \\nick | 10:32 |
| snakeleg | \quit | 10:32 |
| zsombi | dpm: so, we have two more alarm bugs which are tagged | 11:07 |
| zsombi | 13:27 nik90: zsombi: yes since when a repeating alarm is triggered, the clock app bottom edge reads "No Active Alarms" since the alarm date has passed. | 11:10 |
| zsombi | nik90: maybe you should add this to teh bug | 11:10 |
| nik90 | zsombi: sure | 11:30 |
| zsombi | nik90: I've already added | 11:30 |
| nik90 | zsombi: ah thnx | 11:31 |
| zsombi | nik90: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/alarm-data-adaptation/+merge/237611 | 12:07 |
| zsombi | nik90: you can test it on your desktop | 12:07 |
| zsombi | nik90: however remember that you need the qtorganizer5-eds change from renato! | 12:07 |
| nik90 | zsombi: just got home, sure will test it out | 12:36 |
| nik90 | zsombi: I don't think renato's patch has landed yet..so it wouldn't be in utopic desktop yet. And his MP's doesn't output amd64 packages. So I will have to wait for jenkins to build armhf packages for your MP and then test on the phone | 12:38 |
| zsombi | nik90: thn you can test with memory backend. What I did was that I branched renato's MR, and make install-ed it | 12:39 |
| zsombi | nik90: I saw the MR not passing yet, but this is a way to brute force the qtorganizer-eds to be testable | 12:40 |
| nik90 | ok | 12:41 |
| === davidcalle_ is now known as davidcalle | ||
| rpadovani | renatu, hi :-) Do you have 5 minutes? I have a question about multipleselectionlistview :-) | 12:50 |
| renatu | rpadovani, yes | 12:50 |
| rpadovani | renatu, thanks! So, I implemented the multiple selection in browser, works all very well but delete of multiple things. It works for 1 element, but if I select n+1 arguments, it deletes only n elements | 12:51 |
| rpadovani | renatu, this is the code for the delete, I used dialer-app as base | 12:51 |
| rpadovani | https://bazaar.launchpad.net/~rpadovani/webbrowser-app/1351167/view/head:/src/app/webbrowser/HistoryView.qml#L142 | 12:51 |
| renatu | let me see | 12:52 |
| renatu | rpadovani, could you try replace "historyView.historyDomainRemoved(domain)" by "console.debug(domanin) to check if it is working correct | 12:54 |
| renatu | I believe the problem is that you are removing the items while interacting over it | 12:54 |
| renatu | you should store the domanins in a extra list | 12:54 |
| rpadovani | renatu, items.count is right (like 2) but it prints only 1 domain | 12:55 |
| renatu | ok | 12:55 |
| rpadovani | renatu, I can try to it, thanks | 12:55 |
| === _salem is now known as salem_ | ||
| === rmescandon is now known as rmescandon-lunch | ||
| rpadovani | renatu, aha! Saving domains list and removing from there worked well, thanks :-) | 13:09 |
| rpadovani | oSoMoN, ^^ | 13:09 |
| oSoMoN | excellent! | 13:10 |
| renatu | rpadovani, yes as I suspected you are removing from the list while interacting over it | 13:10 |
| rpadovani | oSoMoN, I did the MR, I'll start to work on other things in history :-) | 13:16 |
| zsombi | renatu: your qtorganizer branch fails on CI | 13:24 |
| dpm | mzanetti, o/ quick question: what do you think of https://code.launchpad.net/~rpadovani/reminders-app/multipleAccounts/+merge/237682 - do you see an easy way to use an OptionSelector instead of a ListView? | 13:24 |
| mzanetti | dpm: but we're opening a new page | 13:26 |
| mzanetti | dpm: would be odd to squeeze accounts into an optionselector while keeping the rest of the page empty | 13:26 |
| mzanetti | imo listview is better suited | 13:26 |
| mzanetti | other then that, if you want the optionselect, I don't see where the problem is | 13:27 |
| renatu | zsombi, we are aware of that, there is a intermittent test caused by EDS start and stop process during the tests | 13:28 |
| renatu | zsombi, but it is a unit test then if the silo build it ok, we are good to go | 13:29 |
| rpadovani | mzanetti, that I cannot add to an optionselector a new delegate as footer | 13:29 |
| dpm | mzanetti, but it's not a matter of space, is it? I.e. the optionselector can also be left expanded. The thing with listitem is that it does not show you clearly which account you've got currently selected. As per the OptionSelector issue, rpadovani can expand, but he was mentioning that he wanted to add a new delegate for the unknown account type, which seems not to be possible with optionselector | 13:29 |
| rpadovani | mzanetti, about the bug we talked yesterday, the user on g+ replied: http://pastebin.com/KdazQwnq | 13:29 |
| dpm | rpadovani, ah actually, why do you need a different delegate? | 13:30 |
| mzanetti | rpadovani: why would you want a footer | 13:30 |
| mzanetti | I think the current button in the footer is quite horrible anyways | 13:30 |
| mzanetti | should be a tick in the toolbar | 13:30 |
| rpadovani | dpm, because the unknown account has to execute setup.exec, but others delegate needs accountname.login() | 13:31 |
| rpadovani | mzanetti, because dpm wants in the list a delegate with "Unknown delegate - tap to authorize" that on click launches oa | 13:32 |
| dpm | rpadovani, ah, gotcha. And the call cannot be set dynamically? | 13:32 |
| rpadovani | dpm, mhhh, maybe yes, I can set a flag in the delegate that for default is false, but then add the unknown delegate with this flag set to true. But could I add something only to optionselector also if the model is set by c++ backend? | 13:33 |
| * mzanetti can't follow the discussion | 13:34 | |
| dpm | rpadovani, I'm not sure I understand the part about the model being set by the backend | 13:34 |
| rpadovani | dpm, if model: accounts, and I cannot modify accounts, can I add something to the model? | 13:35 |
| rpadovani | mzanetti, maybe I'm overcomplicating something very easy, but we have a list of accounts in an option selector, and all they delegates are equals, then I need a different delegate. How can I add the different delegate to the optionselector? In listview I use footer property | 13:37 |
| mzanetti | for example: | 13:38 |
| mzanetti | Component { id: a Foo {} } | 13:38 |
| mzanetti | Component { id: b Bar{} } | 13:38 |
| mzanetti | doelegate: ifSomething ? a : b | 13:38 |
| mzanetti | else | 13:38 |
| mzanetti | delegate: Loader { source: ifSomething ? a : b } | 13:39 |
| mzanetti | or, if the delegate still look mostly the same | 13:39 |
| mzanetti | just keep one delegate and change some properties inside | 13:39 |
| rpadovani | mzanetti, mhh, not sure I understood. We want to do something like this: https://i.imgur.com/ujy3RQ7.jpg you have n accounts in the model accounts, then ifFlag add also the "Unknown account ..." | 13:41 |
| mzanetti | I like how "dpm" is pixelated in the screenshot and then written in the sketch | 13:42 |
| rpadovani | lol | 13:42 |
| === rmescandon-lunch is now known as rmescandon | ||
| mzanetti | rpadovani: well, I'm sorry, I still don't see where the problem is tbh | 13:43 |
| mzanetti | isn't this just like this? | 13:43 |
| mzanetti | delegate: Label { text: model.authenticated ? model.name : "UNKNOWN - Do stuff by clicking" } | 13:44 |
| rpadovani | mzanetti, the uknown isn't in the model | 13:45 |
| rpadovani | *unknown - there is no way I spell it right | 13:45 |
| oSoMoN | rpadovani, excellent, thanks | 13:47 |
| mzanetti | rpadovani: so the problem is still that you can't know if an account is authenticated or now? | 13:49 |
| mzanetti | err, authorized | 13:49 |
| rpadovani | mzanetti, no, the problem is I have a model with all authorized accounts, then I have a flag that says if there is at least one unauthorized account. If this flag is true, I want to add another voice at the end of the listview | 13:50 |
| dpm | mzanetti, well, it's not actually dpm in the screenshot ;) | 13:50 |
| rpadovani | mzanetti, at the end of the optionSelector | 13:50 |
| rpadovani | because in the listview I use footer and it's easy | 13:50 |
| mzanetti | rpadovani: but that doesn't match with dpm's sketch, does it? | 13:51 |
| rpadovani | mzanetti, this is the dpm's sketch, he has one authorized account and at least one unauthorized. So he has a model with one account | 13:52 |
| rpadovani | and the unAuthorizedFlag set to true | 13:52 |
| mzanetti | so if you have 3 unauthenticated accounts, would there still only be one UNKNOWN shown in reminders? | 13:53 |
| dpm | rpadovani, would that make it easier to implement? http://i.imgur.com/zT8A3lo.jpg | 13:53 |
| rpadovani | mzanetti, yes | 13:53 |
| rpadovani | dpm, you have a working implementation right now, there is only that height set on component on completed that is a bit strange. Anyway, yes, this is easy to implement, I think. It's similar to the first one I implemented | 13:56 |
| dpm | rpadovani, cool, my point is to separate authorized accounts from those which require authorization. Question: does it make sense to have a list of unauthorized accounts, though? I.e. if I've got 2, is it worth having an entry for each one of them? Can they be authorized individually? | 13:59 |
| rpadovani | dpm, I'm not sure and I cannot verify right now, I have to see what AccountService gives to us | 14:00 |
| dpm | popey, all set? | 14:01 |
| dpm | on my way to the hangout | 14:01 |
| popey | am there ☻ | 14:02 |
| akiva-thinkpad | ah | 14:02 |
| akiva-thinkpad | hangout? | 14:02 |
| popey | team one. | 14:03 |
| akiva-thinkpad | so not on-air? | 14:03 |
| popey | no | 14:03 |
| rpadovani | mzanetti, anyway, I don't find any simple way to create n+1 delegates in an option selector if model has n delegates | 14:03 |
| akiva-thinkpad | ;_; I have to wait 1 whole day to ask more questions? | 14:03 |
| mzanetti | rpadovani: I thought you want n delegates only | 14:04 |
| mzanetti | rpadovani: like if there's 5 unauthenthicated accounts you'd show 5 of them | 14:05 |
| rpadovani | mzanetti, ok, you know, I'm not very good to explain what I want. Anyway, did you read my message about the bug user reported on G+? | 14:05 |
| rpadovani | mzanetti, I think has been lost in the conversation. He replied with that paste: http://pastebin.com/KdazQwnq | 14:06 |
| mzanetti | rpadovani: dpm: why don't we show the actual number of accounts? | 14:06 |
| mzanetti | if you press the "UNKNOWN" one, which would be selected then? | 14:06 |
| mzanetti | rpadovani: yeah, seen that... sadly not totally helpful the log | 14:07 |
| rpadovani | mzanetti, gmail webapp shows a windows, I think it's created by OA, with a list of your unauthorized accounts for that service | 14:08 |
| rpadovani | mzanetti, I dunno if is the only way to implement that | 14:08 |
| === boiko__ is now known as boiko | ||
| === artmello_ is now known as artmello | ||
| om26er | oSoMoN, Hi! | 14:43 |
| oSoMoN | om26er, hey | 14:43 |
| oSoMoN | what’s up? | 14:44 |
| akiva-thinkpad | akiva-thinkpad, Hi! | 14:44 |
| om26er | oSoMoN, twitter webapps seems broken, is that related to the browser string change ? | 14:44 |
| akiva-thinkpad | akiva-thinkpad, hey | 14:44 |
| om26er | broken as in old html version | 14:44 |
| akiva-thinkpad | whats up? | 14:44 |
| * akiva-thinkpad craws back to whenst he came | 14:44 | |
| oSoMoN | om26er, there hasn’t been a UA string change recently | 14:46 |
| oSoMoN | om26er, and I’m not seeing that on flo, the version of the twitter UX I’m getting in the browser seems rather modern | 14:48 |
| oSoMoN | om26er, can you reliably reproduce? and if so, can you please file a bug? | 14:48 |
| om26er | oSoMoN, always happening for me. Let me share the screenshot with you | 14:48 |
| om26er | oSoMoN, http://imgur.com/tPPKGsj | 14:50 |
| oSoMoN | om26er, I can’t even log in to the twitter webapp, it consistently crashes while authenticating me | 14:55 |
| om26er | oSoMoN, i'll report the bug | 14:57 |
| oSoMoN | om26er, thanks | 14:57 |
| om26er | oSoMoN, something that have annoyed me for a while is this bug https://bugs.launchpad.net/webapps-core/+bug/1371075 | 14:57 |
| ubot5 | Ubuntu bug 1371075 in The Webapps-core project "[Facebook] shows very small images" [Medium,New] | 14:57 |
| om26er | I had reported it for the wrong package it seems, its a problem of the browser | 14:57 |
| oSoMoN | om26er, it’s probably an umpteenth UA override bug | 14:58 |
| oSoMoN | facebook making wrong assumptions about the kind of device it’s running on, and serving content in consequence | 14:58 |
| akiva-thinkpad | popey, are you running the q&a tomorrow? Last time I noticed that there was not a ton of questions, and so I was thinking of making a reddit thread beforehand for people to ask. | 14:58 |
| akiva-thinkpad | and have people upvote the questions they think are good | 14:59 |
| om26er | oSoMoN, only the images that are uploaded by someone are small. Video thumbnails are shown the right size. | 14:59 |
| mihir | this looks amazing ==> http://goo.gl/kR0Hyt | 15:08 |
| akiva-thinkpad | mihir, what is this from? | 15:10 |
| popey | akiva-thinkpad: yes, I'm doing it tomorrow, dunno who with | 15:21 |
| nerochiaro | artmello: can you also look at https://code.launchpad.net/~phablet-team/gallery-app/popover-position/+merge/238181 when you have time ? | 15:28 |
| nerochiaro | artmello: and if you have bugs to pass to me please let me know, as I'm running out of high priority bugs | 15:28 |
| PaoloRotolo | Hi all! | 15:32 |
| PaoloRotolo | I'm trying to run an HTML5 app on Ubuntu Touch. Everything works fine on Desktop (with Ubuntu SDK) but on mobile the app is stuck at loading screen. | 15:33 |
| PaoloRotolo | I think this is the bug: https://lists.launchpad.net/ubuntu-phone/msg10189.html | 15:33 |
| PaoloRotolo | And this is my console output: http://pastebin.com/mULaCtqs | 15:33 |
| PaoloRotolo | Is there any fix :)? | 15:34 |
| dpm | hi PaoloRotolo o/ - it seems there aren't any HTML experts around, perhaps you can ask dbarth or alex-abreu on#ubuntu-webapps | 15:47 |
| dpm | PaoloRotolo, oh, daker is on this channel too, and he can perhaps help you ^ | 15:48 |
| nik90 | mzanetti: ping | 15:49 |
| dpm | akiva-thinkpad, you're my favourite person of the day! https://bugs.launchpad.net/ubuntu-filemanager-app/+bug/1380575 | 15:49 |
| ubot5 | Ubuntu bug 1380575 in Ubuntu File Manager App "File manager should use back in toolbar" [Undecided,Confirmed] | 15:49 |
| PaoloRotolo | dpm: great, thanks :) BTW, It seems that all HTML5 apps are not working in latest uTouch build | 15:50 |
| dpm | PaoloRotolo, oh, really? that's a major bug, then. Do you have an example of an app or apps that is/are not working? | 15:51 |
| popey | PaoloRotolo: wfm | 15:53 |
| popey | my calculator app works... | 15:53 |
| popey | hmm, seems my nexus 4 isn't fully up to date... | 15:54 |
| popey | PaoloRotolo: which image you on? | 15:55 |
| PaoloRotolo | popey: Ubuntu 14.10 r243 | 15:57 |
| PaoloRotolo | on Nexus 7 2013 | 15:57 |
| PaoloRotolo | Last updated 10/10/14 | 15:57 |
| popey | hm. I am just updating to #279 here | 15:57 |
| popey | on the devel-proposed channel | 15:58 |
| === gatox is now known as gatox_lunch | ||
| mzanetti | nik90: hey | 16:08 |
| mzanetti | what up? | 16:09 |
| nik90 | mzanetti: Hi :) I got 2 question about qml test pratices , I will be quick | 16:09 |
| PaoloRotolo | popey: nice, let me know :) | 16:09 |
| nik90 | mzanetti: first, is there a way to use compare() or tryCompare() to check if a value is greater than some value? | 16:09 |
| nik90 | mzanetti: or perhaps I am approaching it incorrectly | 16:10 |
| mzanetti | nik90: for compare, you can do compare(foo > bar, true) | 16:10 |
| nik90 | like for instance, I need to check if the lisitem.count is greater than 0 | 16:10 |
| nik90 | ah | 16:11 |
| mzanetti | nik90: for tryCompare it's not that easy, in UnityTestCase (not sure about UbuntuTestCase) we have one called tryCompareFunction() | 16:11 |
| nik90 | so compare(list.count>0, true, "message") | 16:11 |
| mzanetti | you can use it like: tryCompareFunction(function() { return foo > bar;}, true) | 16:11 |
| mzanetti | yeah | 16:11 |
| nik90 | mzanetti: ah ubuntu test case has it as well :D | 16:12 |
| nik90 | I just checked the ubuntu sdk api docs :) | 16:12 |
| nik90 | mzanetti: ok, so second question..I am creating a file with common test functions like pressing a header button etc as shown in http://paste.ubuntu.com/8553488/ | 16:13 |
| nik90 | mzanetti: am I doing it correctly? | 16:13 |
| nik90 | I am already using it in my tests and has helped reduce code quite a bit. However I don't want to be using a wrong method here | 16:14 |
| mzanetti | nik90: yeah, looks good. some of them even look like being worth being upstreamed into UbuntuTestCase | 16:14 |
| nik90 | mzanetti: ooh cool. As I use them more and more, I am checking if they have defined in the most general way possible. | 16:15 |
| nik90 | once they stabilise a bit, I will propose them upstream | 16:15 |
| nik90 | mzanetti: thnx for answering the questions, I think I am set | 16:16 |
| mzanetti | kk | 16:16 |
| dpm | popey, does the music scope show your music in its front page? For me it's stopped doing that and only shows me results from 7digital | 16:29 |
| popey | dpm: yes, it shows my music | 16:31 |
| ahayzen_ | dpm, trying pulling to refresh the scope :) | 16:31 |
| dpm | ahayzen_, yeah, it shows my music for a split second after refresh, then it disappears | 16:32 |
| ahayzen_ | dpm, does the music-app show the music? | 16:32 |
| dpm | ahayzen_, yeah, it does, and also when I click on the scope's "my music" header. It will just refuse to show it on the front page | 16:33 |
| ahayzen_ | dpm, strange | 16:33 |
| dpm | we might have built some kind of scopes AI that refuses to show music if it doesn't accord to its tastes :) | 16:33 |
| ahayzen_ | haha +1 | 16:34 |
| ahayzen_ | although popey won't be able to listen to any music then :/ | 16:34 |
| * ahayzen_ hides his ABBA quickly | 16:34 | |
| dpm | nothing is quick enough for... scopes! mwahaha | 16:35 |
| === gatox_lunch is now known as gatox | ||
| akiva-thinkpad | ahayzen_, or mihir ; got a qml question | 17:08 |
| ahayzen_ | akiva-thinkpad, o/ | 17:08 |
| akiva-thinkpad | ahayzen_, :D | 17:08 |
| akiva-thinkpad | okay looking to have a flickable of labels in a row, but I want to constrain the view of said labels | 17:09 |
| akiva-thinkpad | because I don't want the labels which are out of the scope, to be sitting on top of other components. | 17:09 |
| akiva-thinkpad | i'll show you what I mean by this: | 17:09 |
| === karni is now known as karni-afk | ||
| akiva-thinkpad | ahayzen_, http://i.imgur.com/M1DG40J.png | 17:11 |
| akiva-thinkpad | so the highlighted part is the part of the flickable I do not want viewable | 17:11 |
| akiva-thinkpad | Can I do this by placing the flickable inside a rectangle? | 17:11 |
| ahayzen_ | akiva-thinkpad, so it is going outside of the specified width of the flickable? ... try clip: true ? | 17:11 |
| akiva-thinkpad | ah clip | 17:11 |
| akiva-thinkpad | never used that | 17:12 |
| akiva-thinkpad | sec | 17:12 |
| ahayzen_ | akiva-thinkpad, http://qt-project.org/doc/qt-5/qml-qtquick-item.html#clip-prop | 17:12 |
| akiva-thinkpad | ahayzen_, worked perfectly | 17:12 |
| akiva-thinkpad | thanks | 17:12 |
| ahayzen_ | akiva-thinkpad, no problem | 17:12 |
| akiva-thinkpad | that was... omg wonderful :) | 17:12 |
| ahayzen_ | heh | 17:12 |
| akiva-thinkpad | !cookie | ahayzen_ | 17:13 |
| ubot5 | ahayzen_: Wow! You're such a great helper, you deserve a cookie! | 17:13 |
| akiva-thinkpad | *100 | 17:13 |
| * ahayzen_ eats cookies nom nom nom | 17:13 | |
| akiva-thinkpad | :) | 17:13 |
| === karni-afk is now known as karni | ||
| mihir_ | akiva-thinkpad, ping | 17:19 |
| mihir_ | akiva-thinkpad, i there are some trivial issues on your MR | 17:19 |
| akiva-thinkpad | mihir_, pong | 17:19 |
| akiva-thinkpad | mihir_, ah I'll go do that right now | 17:19 |
| akiva-thinkpad | this is the calendar app right? | 17:20 |
| akiva-thinkpad | I'm in a good mood; making good progress with the file manager app | 17:21 |
| mihir_ | akiva-thinkpad, http://i.imgur.com/M7lWwia.png | 17:21 |
| mihir_ | akiva-thinkpad, great :) | 17:21 |
| akiva-thinkpad | mihir_, you want the dividers to be full length? | 17:22 |
| akiva-thinkpad | or to have a margin? | 17:22 |
| mihir_ | akiva-thinkpad, margin will do i guess , just take a call yourself. | 17:23 |
| mihir_ | akiva-thinkpad, now also if you could get what i meant on From time :) | 17:23 |
| akiva-thinkpad | mihir_, it almost fits on your screen there. I am terribly tempted to take out the calendar label | 17:24 |
| akiva-thinkpad | no I don't see it | 17:24 |
| akiva-thinkpad | :S | 17:24 |
| akiva-thinkpad | ohhhhhhhhhhhhhhhhh | 17:24 |
| akiva-thinkpad | oh | 17:24 |
| akiva-thinkpad | oh I'm such a doof | 17:24 |
| akiva-thinkpad | *Facepalm* | 17:24 |
| mihir_ | akiva-thinkpad, also you need merge from trunk :) | 17:25 |
| akiva-thinkpad | yah will do | 17:25 |
| vitimiti | I am creating an app that requires creating a folder in the predefined datalocation (ie: ~/.local/share/appfolder for Ubuntu desktop) and downloading a database to it. Is it possible to do so as many times as it's required with just the network permissions or do I need to add other permissions to the .apparmor file? | 17:25 |
| mihir_ | akiva-thinkpad, let it be as of now. | 17:25 |
| mihir_ | akiva-thinkpad, we'll take call afterwards :) | 17:25 |
| akiva-thinkpad | heh | 17:26 |
| mihir_ | nik90, you around ? | 17:30 |
| nik90 | mihir_: sort of | 17:30 |
| mihir_ | nik90, got few mins to review small MR https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/blankBubbleInWeekView/+merge/237069 | 17:31 |
| mihir_ | i am planning to push store updates , if we can get this done would be great :) | 17:31 |
| mihir_ | nik90, only if you get some time :) | 17:32 |
| nik90 | mihir_: I will do it by end of tonight, would that be okay? I am kinda in the middle of some work | 17:32 |
| mihir_ | nik90, no issues :) take your time. | 17:32 |
| mihir_ | nik90, thanks. | 17:33 |
| mihir_ | renatu, is that EDS change will be available in latest image? | 17:33 |
| renatu | mihir, not yet, bzoltan is working on that | 17:33 |
| mihir_ | renatu, okay thanks for the update. | 17:34 |
| akiva-thinkpad | Oh my gosh person in the coffee shop, turn off your annoying phone's game music. | 17:36 |
| akiva-thinkpad | mihir_, I really like what you did with Repeats and Reminder; I am curious whether we can do something similar with Location, where we make it a subtitled list item rather than a manual entry | 17:42 |
| mihir_ | akiva-thinkpad, that was basically nik90 idea :) | 17:43 |
| akiva-thinkpad | nik90, oh! taking credit for my idea >:[[[[[[[[[[[[[[[[[[[[[[ | 17:43 |
| akiva-thinkpad | jk | 17:43 |
| mihir_ | akiva-thinkpad, i meant the idea of what we have done with repeat and reminders :) | 17:43 |
| akiva-thinkpad | ah | 17:44 |
| mihir_ | not the one with Location. | 17:44 |
| mihir_ | akiva-thinkpad, hmm location is just to enter text , and moving screen for that too would be annoying for user. | 17:44 |
| akiva-thinkpad | mihir_, I was thinking of moving the screen to two growing quick lists | 17:45 |
| akiva-thinkpad | 1 quick list would be from your contacts | 17:45 |
| akiva-thinkpad | the other would be from previously entered locations | 17:45 |
| akiva-thinkpad | and perhaps a third option would be to pin point a location on a map | 17:45 |
| akiva-thinkpad | writing "vancouver 124 main street, v7r 3x3 apt suite 50" manually seems way too... inefficient. | 17:46 |
| PaoloRotolo | popey: any news? :) | 17:46 |
| akiva-thinkpad | i'll leave it as is for now | 17:46 |
| * akiva-thinkpad is working on the filemanager app, and I want to get that done today | 17:46 | |
| mihir_ | akiva-thinkpad, yeah | 17:48 |
| mihir_ | lets get this merged :) | 17:48 |
| akiva-thinkpad | :) | 17:48 |
| akiva-thinkpad | well that is slightly confusing... why are the margins different for the desktop and the emulator...? | 18:04 |
| mihir | akiva-thinkpad: use units.gu | 18:08 |
| akiva-thinkpad | mihir, I am | 18:08 |
| akiva-thinkpad | thats why its so confusing... | 18:08 |
| mihir | akiva-thinkpad: then it shoudl take care based on device. | 18:08 |
| mihir | akiva-thinkpad: it shouldn't required more than 0.5 and maximum would be 1 | 18:09 |
| nerochiaro | artmello: anything i can take care of from you tomorrow ? | 18:12 |
| artmello | nerochiaro: I will finish the review of those MR today | 18:12 |
| nerochiaro | artmello: thanks. any bugs in your list you want to pass to me ? | 18:13 |
| akiva-thinkpad | mihir http://i.imgur.com/nWpMmax.png that is inconsistent; i'm at a loss as to how to deal with this. | 18:13 |
| artmello | nerochiaro: tomorrow I will take a look on the buglist for gallery, so far all the ones I was working on are handled | 18:13 |
| mihir_ | akiva-thinkpad, you gave 0.5 or 1? | 18:13 |
| mihir_ | akiva-thinkpad, also let it be the calendar it is i guess. | 18:14 |
| nerochiaro | if you could do that today it would help, so i can start working on them on the morning | 18:14 |
| nerochiaro | artmello: ^ | 18:14 |
| artmello | nerochiaro: sure, I drop you an email eod | 18:14 |
| nerochiaro | artmello: excellent. thanks and have a good rest of the day. eod'ing too | 18:14 |
| artmello | nerochiaro: thx | 18:15 |
| nik90 | akiva-thinkpad: in that screenshot the one on the left is better | 18:24 |
| nik90 | akiva-thinkpad: what's the issue | 18:24 |
| === rpadovani is now known as rpadovani|Dota | ||
| mihir_ | akiva-thinkpad, are you done with the changes ? | 19:16 |
| mihir_ | akiva-thinkpad, sorry i am after you , lol | 19:16 |
| mihir_ | popey, only thing worried me on calendar is , it still takes time to open a new event page :| | 19:53 |
| === randomcpp is now known as gcollura | ||
| nik90 | mihir_: use the qtc profiler tool to figure out where it is taking time to load stuff | 20:20 |
| nik90 | mihir_: I just reduced the world city page loading time by loading models dynamically by analysing it in qtc profiler tool | 20:20 |
| mihir_ | nik90, we need to get this externally or it is there inside QTC ? | 20:21 |
| nik90 | it is inside qtc | 20:21 |
| nik90 | analyse tab | 20:21 |
| === salem_ is now known as _salem | ||
| mihir_ | nik90, yeah got it. | 20:21 |
| mihir_ | thanks i'll do that. | 20:21 |
| akiva-thinkpad | mihir, http://i.imgur.com/bvbyW7C.png | 20:22 |
| akiva-thinkpad | exact same list header specifications, yet the operate differently in the emulator | 20:23 |
| mihir_ | akiva-thinkpad, but i guess you changed position of calendar right ? | 20:24 |
| mihir_ | nik90, is there any way to enable it ? i see Record button is on in Analyser | 20:25 |
| nik90 | mihir_: http://developer.ubuntu.com/apps/platform/guides/lets-talk-about-performance/ | 20:25 |
| nik90 | you press the green button near the record button | 20:26 |
| mihir_ | nik90, got it , my eyes couldn't recognize that small green play button :P | 20:28 |
| nik90 | :p | 20:28 |
| * nik90 returns to writing more qml tests | 20:29 | |
| === A7med is now known as Neo31 | ||
| nik90 | elopio: Hey I added one more qml test MP to your review todo :-) | 21:12 |
| sergiusens | nik90: hey, wondering how your testplan as plainbox provider thing turned out | 21:19 |
| nik90 | sergiusens: hey, at the moment we have the plainbox clock app provider which I run before every push to the store. | 21:20 |
| akiva-thinkpad | Are we getting rid of the Loco? | 21:20 |
| nik90 | sergiusens: although still from the terminal | 21:20 |
| elopio | nik90: cool, can I get the lin, please? | 21:20 |
| akiva-thinkpad | I just received an email from the vancouver loco saying that loco is being dropped from the name | 21:20 |
| sergiusens | nik90: were does the provider live? | 21:20 |
| akiva-thinkpad | because it is derogatory in spanish | 21:20 |
| sergiusens | nik90: I want to get rid of the testplan in a wiki thing :-) | 21:21 |
| nik90 | elopio: https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/+activereviews (2 MPs named qml-tests) | 21:21 |
| nik90 | sergiusens: at the moment the provider lives in the source tree of the clock app | 21:21 |
| sergiusens | akiva-thinkpad: we consider it friendly in Argentina :-) | 21:21 |
| nik90 | sergiusens: so whoever is testing will need to install it and then run the manual test suite | 21:21 |
| nik90 | sergiusens: however zyga is working on allowing app devs create a click package which has checkbox-touch + app tests which can then be run on the phone easily...essentially a nice gui | 21:22 |
| nik90 | I believe it is a WIP at the moment. I can check with him tomorrow to see close it is to completion | 21:22 |
| === marcoceppi_ is now known as marcoceppi | ||
| sergiusens | nik90: I prefer off phone anyways | 21:24 |
| sergiusens | so not an issue | 21:25 |
| nik90 | :) | 21:25 |
| akiva-thinkpad | sergiusens, oh neat; :P | 21:37 |
| akiva-thinkpad | where can I find a list of the icon names | 22:16 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
