| qwebirc231403 | I just start using autopilot. I encounter this error, "import mock\n ImportError: No module named mock". Any people know how to solve it? | 02:18 |
|---|---|---|
| ahayzen | qwebirc231403, u'll need to install the package python-mock | 02:31 |
| qwebirc231403 | ahayzen: fixed. Thanks. Now I have this error "from ubuntuuitoolkit import emulators as toolkit_emulators\n ImportError: No module named ubuntuuitoolkit" | 02:39 |
| ahayzen | qwebirc231403, try installing ubuntu-ui-toolkit-autopilot | 02:42 |
| qwebirc231403 | ahayzen: It works, thank you | 02:47 |
| ahayzen | qwebirc231403, no problem | 02:47 |
| dholbach | good morning | 08:23 |
| === oSoMoN_ is now known as oSoMoN | ||
| dpm | morning mzanetti o/ the reminders app is already in the store and looking better every day, thanks! :) I've tested the latest branches and filed a bunch of bugs | 09:00 |
| JamesTait | Good morning all; happy Bubble Wrap Appreciation Day! :-D http://www.virtual-bubblewrap.com/popnow.shtml | 09:20 |
| mzanetti | dpm: o/ | 10:10 |
| mzanetti | dpm: regarding this comment: https://code.launchpad.net/~mzanetti/reminders-app/change-edit-page/+merge/203139/comments/474181 | 10:10 |
| mzanetti | dpm: I've seen this too. haven't really investigated as I' quite sure it's not in my code. but it seems that the scrolled offset is not taken into account | 10:11 |
| mzanetti | dpm: like: if there isn't anything scrolled, everything is fine. but as soon as you scroll down a bit, setting the cursor position breaks | 10:12 |
| mzanetti | and I think exactly the offset for how much you scrolled | 10:12 |
| dpm | mzanetti, yeah, I figured out it wasn't in the app's code. Yes, that seems plausible, right now it's really not possible to do edits because of this, though. I shall report it as an SDK bug | 10:13 |
| dpm | a UITK bug, rather | 10:13 |
| mzanetti | dpm: cool, thanks. | 10:13 |
| === alblasch is now known as ablasche | ||
| === ablasche is now known as ablasche_ | ||
| === ablasche_ is now known as ablasche | ||
| === MacSlow is now known as MacSlow|lunch | ||
| === _salem is now known as salem_ | ||
| WebbyIT | Hey mzanetti :) | 12:00 |
| mzanetti | WebbyIT: o/ | 12:00 |
| WebbyIT | mzanetti, I'm here to help you with reminders-app, if you are ok :-) But I don't know C++... | 12:01 |
| mzanetti | WebbyIT: yeah, I've been talking to dpm. | 12:01 |
| mzanetti | WebbyIT: I was thinking if you could perhaps start with creating the RTF formatting panel | 12:02 |
| WebbyIT | mzanetti, ok, I'll start to work on this asap! | 12:02 |
| mzanetti | WebbyIT: that is, if you use latest trunk and edit a note, you'll see a panel at the bottom for importing pictures and do RTF formatting | 12:02 |
| mzanetti | WebbyIT: when pressing the rtf button, I think the panel should transform into another panel giving controls for bold, italic, color etc | 12:03 |
| dpm | WebbyIT, so if you're interested in participating, in the meantime I'll add you to the team if you're ok with that | 12:03 |
| WebbyIT | mzanetti, cool, yeah! | 12:03 |
| WebbyIT | dpm, yes, sure | 12:03 |
| mzanetti | WebbyIT: I'm not entirely sure yet how to do the actual editing. you could have a look if its possible to edit the stuff in the TextArea directly | 12:03 |
| WebbyIT | mzanetti, ok, I'll investigate then I'll update you | 12:04 |
| WebbyIT | mzanetti, meanwhile, I tried to fix a bug: https://code.launchpad.net/~rpadovani/reminders-app/1273102 | 12:04 |
| mzanetti | WebbyIT: we could do it in c++ easily, however, that would require to reload the content which would cause it to be repositioned etc | 12:04 |
| mzanetti | so if possible we should edit the content in qml directly | 12:04 |
| dpm | WebbyIT, nice! | 12:04 |
| mzanetti | approved | 12:05 |
| dpm | \o/ | 12:06 |
| dpm | WebbyIT, you'll also find links to the design wireframes in https://wiki.ubuntu.com/Touch/CoreApps/Reminders | 12:06 |
| WebbyIT | dpm, saved, thanks | 12:06 |
| WebbyIT | Launch time, I'll return this evening, thanks for indications guys! | 12:06 |
| mzanetti | WebbyIT: yeah, its the "Note composer/Format menu" picture in the wireframe | 12:07 |
| mzanetti | see you later then | 12:07 |
| === MacSlow|lunch is now known as MacSlow | ||
| === salem_ is now known as _salem | ||
| === _salem is now known as salem_ | ||
| ybon | Seems like both my desktop (Saucy) and my mobile (Trusty devel) are shipping Qt 5.0.2, any idea when/if SDK will go on Qt5.2? | 12:50 |
| === gatox is now known as gatox_bbl | ||
| popey | ybon: Mirv may be able to answer that | 12:58 |
| Mirv | ybon: as soon as it works perfectly. Unity8 already runs with it, and Qt 5.2 is being tested via a PPA https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2 | 13:00 |
| Mirv | the problem is that we want it to run as good as Qt 5.0.2 before switching, and there bugs to be fixed (https://bugs.launchpad.net/bugs/+bugs?field.tag=qt5.2) and yet to be filed. hopefully however we're on track to reach the goal soon, as it's now finally possible to test it both on desktop and device to an extent. | 13:01 |
| popey | thanks Mirv | 13:01 |
| ybon | thanks Mirv :) | 13:02 |
| ybon | so we are close to | 13:02 |
| * ybon tempted to upgrade manually | 13:02 | |
| ybon | but, well, no, if I use 5.2 things in my app, others testers will no be able to use the app | 13:03 |
| Mirv | ybon: it will make your phone almost unusable because the last I checked this morning was that app launching didn't really work :) on desktop it's somewhat safer. | 13:03 |
| popey | looking at those 59 bugs, I wouldn't upgrade ☻ | 13:03 |
| Mirv | ybon: yeah I recommend against trying before it's actually launched and delivered via SDK | 13:03 |
| ybon | yep, sounds reasonable :) | 13:03 |
| ybon | popey: so it seems to me that there is no way to support wheel/dblclick/keyboard on map until 5.2 :( | 13:04 |
| ybon | I may we wrong because it's hard to follow the Qt changelog | 13:04 |
| ybon | I may be* wrong | 13:04 |
| popey | oh, interesting | 13:04 |
| popey | the other option might be an overlayed slider or zoom in/out buttons overlayed? | 13:04 |
| ybon | yes, but it breaks the "space to content" principle ;) | 13:05 |
| popey | true | 13:05 |
| ybon | I've tried to get better info on #qt-mobility, but seems quite dead over there | 13:06 |
| ybon | The status as understood by me is that we need this commit to be able to mix touch and mouse events: https://qt.gitorious.org/qt/qtlocation/commit/29af7e577e34089f781322351043de39c025e01e | 13:08 |
| mzanetti | dpm: https://code.launchpad.net/~mzanetti/reminders-app/cleanup/+merge/203317 | 13:08 |
| ybon | but again, Qt is huge, and I'm playing kind of a puzzle at the moment | 13:08 |
| ybon | so I may I've missed the point | 13:09 |
| dpm | mzanetti, running ./run_on_ubuntu_touch.sh now... | 13:12 |
| dpm | mzanetti, what's -DCLICK_MODE=1 in L97? | 13:14 |
| mzanetti | dpm: sergio's changes to cmake require us to call cmake with that now | 13:14 |
| mzanetti | dpm: basically it just changes all the install paths from system to some local dir | 13:14 |
| dpm | ok | 13:15 |
| mzanetti | dpm: also it generates the .json files for click packaging | 13:15 |
| mzanetti | if you want to know more, check CMakeLists.txt and search for if(CLICK_MODE) | 13:15 |
| dpm | ok | 13:17 |
| dpm | mzanetti, I've approved it, but not top-approved as it might be worth having Sergio having a quick look at it | 13:17 |
| mzanetti | dpm: yeah, I added him to the reviewers | 13:17 |
| dpm | cool | 13:18 |
| === salem_ is now known as _salem | ||
| marcusw | qml, hmm? | 13:36 |
| popey | marcusw: hello | 13:44 |
| === gatox_bbl is now known as gatox | ||
| marcusw | popey: morning | 13:53 |
| marcusw | I was just going to ask why QML is involved here, since ubuntu is classically gtk-based | 13:53 |
| popey | marcusw: we switched to qt for ubuntu touch | 13:55 |
| marcusw | oh | 13:56 |
| marcusw | how about that | 13:56 |
| popey | indeed | 13:56 |
| popey | working out quite well ☻ | 13:56 |
| marcusw | how do you handle the qml-is-slow-as-shit problem? | 13:57 |
| popey | heh, we also support c++.. opengl.. | 13:57 |
| marcusw | qmlscene load times weren't an issue? | 13:57 |
| popey | and we report and track performance issues | 13:57 |
| popey | yeah, that's been a problem on the radar | 13:57 |
| marcusw | qmlscene slowness almost killed our platform here | 13:58 |
| marcusw | it's a billion-dollar product for us, kind of a big deal | 13:58 |
| marcusw | we ended up having this one qt guy rewrite it to not be slow | 13:59 |
| marcusw | lol | 13:59 |
| mzanetti | fginther: hi. you around yet? | 13:59 |
| marcusw | unfortunately management isn't very enthusiastic about publishing it | 14:00 |
| fginther | mzanetti, I'm here | 14:00 |
| popey | I think we're looking at alternative ways to launch. | 14:00 |
| mzanetti | fginther: the reminders-app has been renamed to reminders (and with it the AP test suite) | 14:01 |
| mzanetti | fginther: could you please change the job to execute "autopilot run reminders" | 14:01 |
| mzanetti | instead of reminders_app | 14:01 |
| fginther | mzanetti, sure | 14:05 |
| fginther | mzanetti, did the bzr trunk branch change? | 14:06 |
| mzanetti | fginther: no, not yet | 14:09 |
| mzanetti | fginther: so far only the code inside | 14:09 |
| mzanetti | dpm: do we change that? ^ | 14:09 |
| dpm | mzanetti, yeah, makes sense, but I'd double-check with fginther on the timing to do the change on Jenkins. Francis, shall we approve the branch doing the rename and then change the Jenkins job or is there a cleaner way for the transition? | 14:11 |
| fginther | dpm, I was mostly curious if more changes were coming. If there is an MP ready with the autopilot rename ready, lets go ahead and land that, then you can let me know the branch is renamed | 14:13 |
| mzanetti | fginther: https://code.launchpad.net/~mzanetti/reminders-app/cleanup/+merge/203317 | 14:15 |
| dpm | fginther, another question: would it be possible to trigger Jenkins for https://code.launchpad.net/~ted/ubuntu-calendar-app/url-dispatcher/+merge/201543 ? It seems it never ran for that MP | 14:17 |
| fginther | dpm, one moment | 14:21 |
| dpm | fginther, np, thanks!. Also, I'm cleaning up the pending branches for calendar. I've got this one from you: https://code.launchpad.net/~fginther/ubuntu-calendar-app/remove-build-deps/+merge/192579 - as per oSoMoN's comment's do I mark it as rejected, or do you need those dependencies removed for any reason? | 14:27 |
| === zoopster_ is now known as zoopster | ||
| fginther | dpm, it was old and I can't remember what I was trying to do, I've removed it | 14:32 |
| dpm | ok, thanks | 14:32 |
| dpm | popey, do you know what this branch is trying to address? https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/MonthHighlightFix/+merge/201279 | 14:32 |
| dpm | I'm not sure what I should be looking at | 14:33 |
| * popey looks | 14:33 | |
| fginther | dpm, ted's MP is running now, he's probably not a member of the calendar team | 14:34 |
| dpm | fginther, ah, no, he isn't. What do we do in those cases to get Jenkins running? | 14:34 |
| fginther | dpm, someone has to manually run the jobs through the jenkins UI or add him to one of the touch app teams | 14:36 |
| dpm | ok, no worries. If you don't mind, it sounds doable to ping you when we get external contributions. | 14:37 |
| popey | dpm: http://imgur.com/jLf40iM | 14:37 |
| popey | note the difference in the size of the white highlight, it's too big on the right window | 14:38 |
| popey | takes up all the 27 part plus the 20 part above | 14:38 |
| dpm | ah, well spotted, thanks popey | 14:38 |
| popey | i reported it during a meeting but didnt file a bug, kunal fixed it clearly ☻ | 14:39 |
| dpm | cool | 14:48 |
| popey | dpm: any merges you want me to take a look at? | 14:49 |
| dpm | popey, I'm looking at the ones I can do, but perhaps some bug triaging would be helpful: some of the Fix Committed ones need to be checked out to see if they're Fix Released and a bunch of others need to be triaged and need Importance setting | 14:50 |
| popey | lets cover them in our call in 10 | 14:50 |
| davmor2 | popey: could you look at merging dino dna with that of toad and make dinosaurs for us? | 14:51 |
| dpm | ok | 14:51 |
| popey | davmor2: again, in english? | 14:51 |
| davmor2 | popey: think Jurassic Park :) <popey> dpm: any merges you want me to take a look at? not so much fun when you have to explain it though :) | 14:52 |
| popey | davmor2: http://tumbleweed.popey.com/ | 14:54 |
| davmor2 | popey: you need to add the sadtrombone.com button to that just to make it perfect ;) | 14:54 |
| popey | ☻ | 15:01 |
| ybon | Just reading the email about app running in background: does I understand well that they are forbidden by design? | 15:01 |
| Elleo | ybon: yeah, pretty much, you might want to have a read through this thread for some discussion on it, although I'm not sure if there's been anything more recent: https://lists.launchpad.net/ubuntu-phone/msg04736.html | 15:03 |
| ybon | That's sound strange to me, but I may be out of the point | 15:04 |
| ybon | thanks for the link :) | 15:04 |
| Elleo | ybon: yeah, I think the general concern is to avoid poorly written apps hogging battery life when they're not in use | 15:05 |
| Elleo | but it does cause some issues for certain apps (like cutespotify) that need to be running in the background to work normally | 15:05 |
| ybon | yes, I thin I do understand why it's done like this | 15:07 |
| ybon | but as exposed in the email you pointed to me, there are some use cases where this will be a blocker | 15:08 |
| ybon | Ready the thread further | 15:09 |
| ybon | reading* | 15:09 |
| davmor2 | ybon: it maybe that there is a helper that will allow background operation of apps it's just not implemented currently | 15:13 |
| ybon | davmor2: good to know, thanks | 15:14 |
| ybon | for reference, a use case i have in mind is a GPS tracker | 15:15 |
| ybon | I know it's something I will need in the toolbox of a OSMer using UT | 15:15 |
| Elleo | ybon: in the short term for personal use you can simply disable powerd ("adb shell stop powerd") which will stop the phone going to sleep, so as long as the tracker is in the foreground it'll be running | 15:16 |
| Elleo | that's obviously far from ideal though | 15:16 |
| ybon | thanks good to know | 15:17 |
| ybon | but yeah, not really end user friendly ;) | 15:17 |
| ybon | ah, the GPS tracker was already highlighted as a use case in the follow up of the thread :) | 15:18 |
| fginther | mzanetti, I ran your reminders app MP and there was a test failure. Look like some internal import of reminders_app | 15:20 |
| mzanetti | fginther: that might be, yeah. I'll fix it. one sec | 15:27 |
| mzanetti | fginther: pushed | 15:27 |
| === oSoMoN_ is now known as oSoMoN | ||
| === charles_ is now known as charles | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| balloons | oSoMoN, got a moment to look at a bug? https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1271726 | 16:29 |
| ubot2` | Launchpad bug 1271726 in Ubuntu Calendar App "In Week tab labels displayed in HeaderDateComponent differ from date with localization other than English locale" [High,Triaged] | 16:29 |
| oSoMoN | balloons, not now, but I’ll put it on my list | 16:30 |
| === _salem is now known as salem_ | ||
| balloons | It's kind of confusing to read, I'd suggest to have a look at the screenshots. Essentially the date displayed vs the date stored in the object is different | 16:30 |
| balloons | in several places throughout the app.. stemming from a monday start of week locale | 16:30 |
| WebbyIT | mzanetti, o/ | 16:53 |
| === gatox_ultra is now known as gatox | ||
| === bfiller is now known as bfiller_afk | ||
| mzanetti | WebbyIT: hi | 17:50 |
| WebbyIT | mzanetti, fast question: are you on 14.04? 'cause I've some problems with some icons, I'm on 13.10... | 17:51 |
| mzanetti | WebbyIT: yeah, 14.04 here. what's the issue? the icon properties in the toolbaricons? | 17:52 |
| WebbyIT | mzanetti, yap, I sobstitute them with another just to develope, I have to remember to change again before push | 17:52 |
| mzanetti | WebbyIT: there is a ppa which gives you the latest uitk | 17:52 |
| mzanetti | dpm: do you remember which one? | 17:53 |
| WebbyIT | I have ubuntu-sdk-team-ppa-saucy.list, ubuntu-touch-coreapps-drivers-daily-saucy.list and unity-team-ppa-saucy.list | 17:54 |
| dpm | mzanetti, WebbyIT, I'd recommend only installing from the stable PPA. I've had bad experiences with breakage and dependencies when trying the bleeding edge SDK ppa | 17:54 |
| WebbyIT | unfortunately I have problems with my nvidia card, so I prefer to stay on stable version of Ubuntu | 17:55 |
| popey | +1 | 17:55 |
| mzanetti | this ppa should only update the touch components. but fair enough... I guess we can revert those icons to the old api | 17:56 |
| mzanetti | WebbyIT: feel free to propose a branch that makes it work with 13.10 | 17:56 |
| WebbyIT | mzanetti, don't worries, I have no problem, I change with some random icons | 17:56 |
| nik90 | mihir: | 18:05 |
| nik90 | hey | 18:05 |
| === bfiller_afk is now known as bfiller | ||
| mzanetti | fginther: all green btw. https://code.launchpad.net/~mzanetti/reminders-app/cleanup/+merge/203317 | 18:15 |
| mzanetti | thank you! | 18:15 |
| dpm | nik90, we updated the day for the clock hack day in the wiki | 18:15 |
| dpm | let us know if it works for you: https://wiki.ubuntu.com/Touch/CoreApps/HackDays | 18:16 |
| nik90 | dpm: don't be mad, but my exam was actually today so I dont mind tomorrow or the day after | 18:16 |
| nik90 | sry | 18:16 |
| dpm | nik90, ah well, no worries, I hope that it went well :) | 18:16 |
| nik90 | dpm: yeah it went well :) | 18:17 |
| dpm | good to hear | 18:17 |
| dpm | WebbyIT, what's exactly the problem with icons? Are they not showing up in the desktop? | 18:17 |
| WebbyIT | dpm, yes, exactly, I have some errors like | 18:18 |
| WebbyIT | file:///usr/lib/x86_64-linux-gnu/qt5/qml/Ubuntu/Components/Themes/Ambiance/ToolbarButtonStyle.qml:33:9: QML QQuickImage: Failed to get image from provider: image://theme/edit | 18:18 |
| dpm | WebbyIT, mzanetti, in those cases where the desktop theme doesn't have the icon, I think an alternative can be specified, e.g. iconSource: "image://theme/delete,edit-delete-symbolic" | 18:21 |
| om26er | boiko, ping | 18:26 |
| boiko | om26er: pong | 18:26 |
| om26er | boiko, can you review this https://code.launchpad.net/~om26er/dialer-app/support_testability/+merge/203391 | 18:26 |
| om26er | one liner | 18:26 |
| boiko | om26er: yes, sure, I just wonder when this is going to be moved from QApplication to QCoreApplication in Qt itself :-S | 18:26 |
| boiko | om26er: messaging-app will need the same fix | 18:27 |
| om26er | boiko, here: https://code.launchpad.net/~om26er/messaging-app/fix_introspection/+merge/203392 :p | 18:27 |
| boiko | om26er: ah, nice! I can review that one too, if you want | 18:27 |
| om26er | boiko, yes, that one is for you too | 18:28 |
| om26er | ;) | 18:28 |
| om26er | will need to add that to camera app as well | 18:28 |
| boiko | om26er: and address-book-app? | 18:29 |
| om26er | boiko, seems it already has it | 18:29 |
| boiko | om26er: great! | 18:29 |
| om26er | boiko, the "HistoryDelegate" in the messaging app needs a "moving" property when its being expanded. how simple is that going to be to add ? | 18:31 |
| om26er | 10x better than sleep(1) | 18:31 |
| boiko | om26er: I guess it is simple, but maybe an animating property would be better (this might cover future cases of animation) | 18:31 |
| om26er | boiko, I might need to see some examples to be able to try to implement it (and learning) | 18:33 |
| boiko | om26er: let me check the code, just a sec | 18:34 |
| labsin | popey, others, So I try to launch an app with qml plugins. It's working on the dektop in confinement with just placing the libs in ./lib/<arch> but it seems to not be working on the phone | 18:58 |
| popey | labsin: trying to test it manually here | 18:59 |
| labsin | I then tried a wrapper script that exports LD_LIBRARY_DIR and adds the -I option to qmlscene (qmlscene -I lib/<arch>) and that works on the desktop | 18:59 |
| popey | labsin: is your library built against qt4? | 19:03 |
| labsin | popey, no | 19:03 |
| labsin | I don't think so.. | 19:04 |
| popey | labsin: try not using a shell script but use Exec=qmlscene... and specify the qml file before the -I lib/foo | 19:05 |
| popey | i have a feeling the sequence is important | 19:05 |
| popey | qmlscene ./music-app.qml --file=%f -I ./plugins | 19:05 |
| popey | e.g. thats how the music app does it | 19:05 |
| popey | hmm, but we don't have the arch specific directory in the music app | 19:06 |
| labsin | popey, It only has ARM | 19:06 |
| popey | yeah | 19:06 |
| popey | can you build a click which only has arm? | 19:06 |
| popey | with no arch directory? | 19:06 |
| labsin | popey, I used the proposed layout in the ubuntu-touch maillist | 19:07 |
| popey | ah | 19:07 |
| labsin | music app probably needs to change for fat packaging | 19:07 |
| labsin | popey, I used the shell to set the linkers path because I also have a dynamic lib | 19:08 |
| labsin | popey, another try http://ubuntuone.com/2ECSspw1A5N1RwKdcTh1BM | 19:14 |
| popey | ok | 19:14 |
| popey | labsin: it starts | 19:19 |
| popey | labsin: http://popey.com/~alan/phablet/device-2014-01-27-191927.png | 19:19 |
| labsin | popey, does the second tab display anything? | 19:20 |
| popey | yes ☻ | 19:20 |
| popey | labsin: http://popey.com/~alan/phablet/device-2014-01-27-192040.png | 19:20 |
| labsin | popey, It still needs a ROM (you sould get it from your own calc, but i'll give you mine :D) | 19:21 |
| popey | haha ☻ | 19:21 |
| labsin | http://ubuntuone.com/38ngJNIRg4koudql71iwUZ | 19:21 |
| popey | this is way better than searching for MAME ROMS | 19:21 |
| popey | where do I put that? | 19:21 |
| labsin | .local/share/com.ubuntu.developer.labsin.tilem/ | 19:22 |
| popey | ok, it finds it | 19:23 |
| popey | http://popey.com/~alan/phablet/device-2014-01-27-192315.png | 19:23 |
| popey | and explodes when I click it | 19:23 |
| labsin | popey, thought so :) | 19:23 |
| popey | Unable to load ROM file "/home/phablet/.local/share/com.ubuntu.developer.labsin.tilem/ti84se.rom": Permission denied | 19:24 |
| popey | oh | 19:24 |
| * popey fixes | 19:24 | |
| labsin | popey, ow? | 19:24 |
| popey | i used adb push as root | 19:24 |
| popey | ok, better now it's owned by phablet | 19:24 |
| labsin | popey, ahhh (didn't expect that) | 19:24 |
| labsin | calc screen should go white | 19:25 |
| popey | http://popey.com/~alan/phablet/device-2014-01-27-192459.png | 19:25 |
| popey | it does | 19:25 |
| labsin | and then put it on | 19:25 |
| labsin | (left bottom) | 19:25 |
| popey | hmm | 19:25 |
| popey | not responding to that | 19:25 |
| labsin | popey, could you get the log? | 19:26 |
| popey | ya | 19:26 |
| labsin | tnx | 19:26 |
| popey | labsin: / | 19:27 |
| popey | bah | 19:27 |
| popey | http://paste.ubuntu.com/6827938/ | 19:27 |
| popey | (pressed it a few times) | 19:28 |
| popey | i need to make food, responses may slow a bit, but ping me if you need anything | 19:28 |
| labsin | ok thanx | 19:29 |
| samertm | hey all | 19:50 |
| samertm | if I were to work on this cal app bug, where would I start? https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1272625 | 19:51 |
| ubot2` | Launchpad bug 1272625 in Ubuntu Calendar App "In the Day view of the app the bottom of the 11 PM hour is cut off" [Undecided,New] | 19:51 |
| Mihir_ | samertm: hey | 19:57 |
| Mihir_ | samertm: are you trying first time ? | 19:57 |
| samertm | Mihir_: yeah | 19:57 |
| samertm | I'm here for the hack day haha | 19:58 |
| Mihir_ | samertm: did you set up everything correctly ? | 19:58 |
| samertm | I believe so, I have it running in qt creator | 19:58 |
| Mihir_ | samertm: make sure you are able to run application and then try to follow the step | 20:00 |
| samertm | Mihir_: I can recreate the bug. I'm trying to figure out how scrolling works :P | 20:01 |
| samertm | are you saying I should step through the application w/ the debugger?? | 20:01 |
| samertm | ?* only one question mark | 20:02 |
| Mihir_ | Scrolling will work with your mouse wheel | 20:02 |
| Mihir_ | as well | 20:02 |
| Mihir_ | samertm: there must be problem with some size | 20:03 |
| Mihir_ | try to look in Dayview file | 20:04 |
| samertm | Mihir_: alright, thanks for the tip :) | 20:05 |
| samertm | what are the path views? | 20:05 |
| samertm | like, PathView or PathViewBase | 20:05 |
| samertm | are they the main scrolling areas of the app? | 20:05 |
| samertm | Mihir_: I think I've found the culprit... let me do some more testing | 20:08 |
| Mihir_ | samertm: when you look from the root file, it will make call to many other classes | 20:08 |
| labsin | popey, ok maybe this is better: http://ubuntuone.com/2ECSspw1A5N1RwKdcTh1BM | 20:09 |
| samertm | Mihir_: in DayView.qml, under PathViewBase, height is probably set to the wrong value | 20:09 |
| popey | labsin: you broke it file:///opt/click.ubuntu.com/com.ubuntu.developer.labsin.tilem/0.1/TilEm.qml:5 module "Utils" is not installed | 20:14 |
| labsin | popey, damn. Thought the -I wasn't needed (because it complains in the log) | 20:15 |
| popey | labsin: maybe now it's time to setup the emulator ☻ | 20:15 |
| labsin | popey, Yes, I know. It's running for half an hour now and still a black screen | 20:15 |
| popey | yeah, takes a while | 20:15 |
| samertm | Mihir_: through trial and error, the height variable needs to be set to "column.height - viewHeader.height - dayHeader.height - 28" instead of "column.height - viewHeader.height - dayHeader.height" | 20:16 |
| samertm | :P | 20:16 |
| labsin | it seems to work once, I'll try creating it again | 20:16 |
| samertm | so that leaves the question, what header/item are we not taking into account that has a height of 28? | 20:16 |
| samertm | (btw, the bug also exists on WeekView | 20:17 |
| samertm | ) | 20:17 |
| Mihir_ | samertm: okay | 20:33 |
| Mihir_ | samertm: once you are done push it and other will review your MR and will give you the feedback | 20:33 |
| Mihir_ | :) | 20:33 |
| samertm | Mihir_: thanks! so what are the headers and where can I find them? | 20:34 |
| samertm | b/c I measured the bar that says "SUN MON TUE..." at 26 pixels | 20:35 |
| samertm | hmmm... the Day and Week views have different cut off points | 20:40 |
| samertm | but they're both wrong | 20:40 |
| WebbyIT | mzanetti, I started to work on RTF for reminders-app, but via QML I have a problem: if I try to insert some formatting tag (e.g. <b>) if there is no text in the tags, then it is insterted. Mhh, I think I'm not explained well, please see this example: http://paste.ubuntu.com/6828371/ last example is my idea to "hacks" the situation | 20:48 |
| WebbyIT | mzanetti, what do you think? | 20:48 |
| mzanetti | WebbyIT: yeah, the cursorposition doesn't match the actual position in the html | 20:49 |
| mzanetti | WebbyIT: in C++ I walk through the whole xml and count plaintext characters. | 20:50 |
| mzanetti | but not sure if thats an option to do in qml | 20:50 |
| mzanetti | I could offer you some helper from C++ though | 20:50 |
| WebbyIT | mzanetti, but problem is not cursorPosition, problem is that if I try to insert <b></b> doesn't work, but <b>a</b> yes | 20:50 |
| WebbyIT | If C++ can override this behavior, good, I think I have to study C++ :-P | 20:52 |
| WebbyIT | otherwise, insert some example text is the best idea I have | 20:52 |
| mzanetti | ah, I see | 20:52 |
| mzanetti | let me think for a minute | 20:52 |
| mzanetti | so there are 2 cases actually. | 20:53 |
| mzanetti | one if the user has selected some text and clicks the <b> button | 20:53 |
| WebbyIT | and I have not implement this yet, but it's not a problem | 20:53 |
| mzanetti | and the other if nothing is selected but the user wants to enable bold mode for the following characters | 20:53 |
| mzanetti | so I think case 1 is not a problem here | 20:53 |
| mzanetti | for case 2 I'd say we have to set a flag, boldEnabled: true | 20:54 |
| mzanetti | and then when the next char is coming in, make that bold | 20:54 |
| mzanetti | but yeah... otherwise you could just insert them in c++ | 20:55 |
| mzanetti | if that turns out to be easier (which I would think) | 20:55 |
| * WebbyIT looks the web for a C++ guide | 20:56 | |
| mzanetti | check out enmldocument.cpp | 20:56 |
| mzanetti | there's the attachimage | 20:56 |
| mzanetti | its not that hard actually | 20:56 |
| mzanetti | you have the QXmlStreamReader and QXmlStreamWriter | 20:56 |
| mzanetti | it walks through the whole tree and copies over the tags | 20:57 |
| mzanetti | from the reader to the writer | 20:57 |
| mzanetti | and at the given position just insterts the tags | 20:57 |
| WebbyIT | mzanetti, ok, thanks, I'll try to look at it, I'll ping you if I need an help! | 20:58 |
| mzanetti | WebbyIT: there's one more caveat tho | 20:58 |
| mzanetti | if we call something from c++ we need to read the text from the textedit and set it again | 20:58 |
| mzanetti | which means the view would scroll to top | 20:59 |
| mzanetti | or even flicker maybe | 20:59 |
| mzanetti | we need to get around that. perhaps remembering the position and scrolling back. at least evaluate if that would be an option | 20:59 |
| WebbyIT | seems a challenge :-) I like it! | 21:00 |
| samertm | hmm.... viewHeader.height is 38, and dayHeader.height is 64, but I can't find any elements that are those heights in pixels w/ gimp | 21:03 |
| samertm | hmm..... root.height is 552 for WeekView, but 564 for the DayView | 21:06 |
| samertm | Mihir_: any thoughts? | 21:08 |
| samertm | newbie question: where do root objects inherit their height from? | 21:14 |
| samertm | b/c in DayView, root.height is 564, and in WeekView, root.height is 552 | 21:15 |
| samertm | when I would expect them to be the same | 21:15 |
| WebbyIT | mzanetti, mhh, I think I need help on logic implementation: I was able to do a C++ function that is invoked when I press bold button (my first C++ code ever) but I have a problem: if I add only <b> then all the text until the end of text | 22:14 |
| WebbyIT | If i use writer.writeStartElement("b"); | 22:15 |
| WebbyIT | writer.writeEndElement(); | 22:15 |
| WebbyIT | nothing is write | 22:15 |
| WebbyIT | I don't know how to put text between start and end tag | 22:15 |
| mzanetti | writer.writeCharacters() to write text | 22:15 |
| WebbyIT | yes, but there is no text yet :) | 22:16 |
| mzanetti | actually... I just realize... | 22:16 |
| mzanetti | if we insert the tags in there, and then update the ui again, the textedit will optimize them away again :/ | 22:16 |
| mzanetti | dammit. sorry I didn't think of it before | 22:17 |
| WebbyIT | NP, I learn a lot of thing tonight, maybe it's the start of my study of C++ :) | 22:17 |
| mzanetti | yay | 22:17 |
| mzanetti | it wasn't that hard, was it? | 22:17 |
| WebbyIT | No, it wasn't, I look at your code and I have no problem | 22:18 |
| mzanetti | I try to keep it as readable as possible | 22:18 |
| WebbyIT | you do a good job, with some grep and sublime I find everything I need :) | 22:19 |
| mzanetti | thanks.. | 22:19 |
| mzanetti | so yeah. I think we need to do some more here | 22:19 |
| mzanetti | remembering positions and whatnot | 22:19 |
| mzanetti | and modifying the inserted text depending on those flags | 22:19 |
| WebbyIT | Mhh, I have an idea maybe, but I'll use QML :-P | 22:20 |
| mzanetti | yup. whatever you think its best. I think you know all the corner cases and possibilities now. you choose the weapons | 22:29 |
| samertm | Datsundere: hey | 22:36 |
| samertm | dpm: how much do you know about the calendar app? | 22:38 |
| dpm | hi samertm, it depends what you want to know :) What are you trying to find out about calendar? | 22:38 |
| samertm | dpm: I'm trying to figure out how the heights for the different views get set | 22:39 |
| samertm | for bug https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1272625 | 22:39 |
| ubot2` | Launchpad bug 1272625 in Ubuntu Calendar App "In the Day view of the app the bottom of the 11 PM hour is cut off" [Undecided,New] | 22:39 |
| samertm | for instance, the height of WeekView is 552 and the height of DayView (and all the other views) is 564 | 22:40 |
| samertm | and I'm trying to figure out what's causing the difference | 22:40 |
| samertm | I'm brand new to qml :P | 22:41 |
| dpm | samertm, I'm afraid I don't know off the top of my head, I'd have to look at the code, but the best thing might be to send an e-mail to this mailing list -> https://launchpad.net/~ubuntu-touch-coreapps | 22:41 |
| dpm | If you ask there, the developers will be happy to answer the question | 22:41 |
| samertm | dpm: alright, thanks! | 22:42 |
| === salem_ is now known as _salem | ||
| === balloons is now known as Guest72010 | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
