samertm | WebbyIT: already fixed it ;D | 00:02 |
---|---|---|
samertm | also I don't know how to assign bugs to myself | 00:02 |
WebbyIT | samertm, there is a pencil under "Assigned to". Click on it and you can choose to assign it to yourself | 00:02 |
WebbyIT | Let me check the MR :) | 00:03 |
samertm | WebbyIT: Okay, I have to head out. | 00:07 |
samertm | these hack days are really cool :D | 00:07 |
samertm | see you guys | 00:07 |
WebbyIT | samertm, thanks for the fix :) | 00:08 |
WebbyIT | See you :) | 00:08 |
samertm | \exit | 00:08 |
samertm | (oh god i'm such a noob) | 00:08 |
WebbyIT | lol | 00:08 |
=== WebbyIT is now known as rpadovani | ||
=== rpadovani is now known as WebbyIT | ||
WebbyIT | mzanetti, I'm a bit confused about enum, can you link me a good guide or, if you have some time, help me tomorrow? :-) | 00:11 |
mzanetti | WebbyIT: yeah, can help you tomorrow | 00:11 |
WebbyIT | mzanetti, wow, you're always online :D | 00:11 |
WebbyIT | Thanks, you are too kind :) | 00:12 |
mihir_ | popey: we are not continuing our meetings ? | 04:59 |
mihir_ | or it will be enabled after sprint ? | 05:00 |
boren | How to set up account in reminder-app on desktop? | 05:13 |
dholbach | good morning | 07:38 |
JamesTait | Good morning all; happy Freethinkers Day! :-D | 08:49 |
=== chriadam is now known as chriadam|away | ||
dpm | hi fginther, when you're up, could you trigger Jenkins for https://code.launchpad.net/~samer-f/ubuntu-calendar-app/fixes-1272625/+merge/203448 - it's an external contribution, so I think that's why it hasn't run | 10:32 |
popey | dpm: he's in the uk | 10:33 |
WebbyIT | Same problem for the calc -> https://code.launchpad.net/~samer-f/ubuntu-calculator-app/fixes-1273555/+merge/203642 | 10:33 |
dpm | ah, nice, forgot everyone is in London now! :) | 10:33 |
dpm | thanks WebbyIT | 10:33 |
dpm | Samer is rocking it | 10:34 |
WebbyIT | samer-f is on fire :D | 10:34 |
dpm | \m/ | 10:34 |
=== _salem is now known as salem_ | ||
mivoligo | nik90: hi, I'm thinking on working on bug #1272628 if it's not taken already | 11:00 |
ubot2` | Launchpad bug 1272628 in Ubuntu Clock App "Leading zero in time values is distracting" [Low,Triaged] https://launchpad.net/bugs/1272628 | 11:00 |
nik90 | mivoligo: sure go ahead :) | 11:00 |
mivoligo | nik90: I need some guidance regarding bzr though | 11:01 |
nik90 | mivoligo: definitely. Do you have bzr installed and set up on your computer? | 11:02 |
mivoligo | nik90: yes, I even did bzr branch lp:ubuntu-clock-app | 11:03 |
mivoligo | what should I do now? | 11:03 |
nik90 | mivoligo: oh that's great | 11:03 |
nik90 | mivoligo: so now you make your code changes to fix the bug | 11:03 |
nik90 | mivoligo: once you do that, you can save those changes to bzr by committing them -> bzr commit -m "commit message here" | 11:03 |
mivoligo | nik90: should I make a special branch or something? | 11:04 |
nik90 | mivoligo: and then push those changes back to clock app project by -> bzr push lp:~launchpadid/ubuntu-clock-app/branch-name | 11:04 |
nik90 | mivoligo: when you push the changes to launchpad, that's where you create a branch | 11:04 |
nik90 | until then you are already working on your own local branch | 11:05 |
nik90 | mivoligo: Take a look at https://wiki.ubuntu.com/Touch/CoreApps/DevelopmentGuide?action=AttachFile&do=get&target=Development%2Bworkflow.png | 11:05 |
mivoligo | nik90: ok, looks clear | 11:07 |
nik90 | mivoligo: can you assign the bug to yourself before starting on it | 11:08 |
popey | yeah, i tend to do "bzr branch lp:ubuntu-clock-app" and then "bzr branch ubuntu-clock-app/ fix-1273555/ | 11:08 |
popey | then do my fixes in fix-1273555/ and bzr -m commit "fixes 1273555" | 11:08 |
mivoligo | nik90: one more thing: is there any time frame I should finish the bug fix? | 11:08 |
popey | then bzr push lp:~popey/ubuntu-clock-app/fixes-1273555 | 11:08 |
nik90 | mivoligo: not really | 11:08 |
popey | works well for me | 11:08 |
mivoligo | thanks popey | 11:08 |
nik90 | popey: why not bzr branch lp:ubuntu-clock-app fix-1273555 | 11:09 |
popey | i like having both on my machine | 11:09 |
nik90 | it will branch it to that folder name directly | 11:09 |
nik90 | oh ok | 11:09 |
popey | but that's just me not knowing bzr that well ☻ | 11:09 |
nik90 | I use the qtcreator diff feature to see the code diff...it nicely puts it in green and red colors | 11:10 |
mivoligo | nik90: just assigned the bug to myself :) | 11:10 |
nik90 | mivoligo: nice :) | 11:13 |
mivoligo | nik90: in the Timer I can set “60:60” Is it a known bug? | 11:14 |
mivoligo | nik90: also I can set “30:30" for example, in the way where the hands are not aligned with each other or 30 mark: http://screencloud.net/v/qlHm | 11:18 |
nik90 | mivoligo: yup they are bugs | 11:19 |
mivoligo | nik90: ok | 11:20 |
WebbyIT | popey, nik90, try qbzr, it's gui for bzr and works as bzr, but plus q. E.g: bzr qdiff for have a colored bzr diff, bzr qcommit to choose what commit, etc etc | 11:21 |
nik90 | WebbyIT: oh that's nice.. | 11:21 |
mivoligo | nik90: I know you're working on setting alarm, so maybe I better do not change the code for that | 11:30 |
nik90 | mivoligo: feel free to change it if you want, I can always do a bzr merge and resolve the conflicts | 11:30 |
mivoligo | nik90: ok | 11:31 |
fginther | dpm, ah, sorry I missed your ping, looks like that MP is merged now | 11:32 |
WebbyIT | fginther, same problem on https://code.launchpad.net/~samer-f/ubuntu-calculator-app/fixes-1273555/+merge/203642 | 11:47 |
WebbyIT | Could you verify please? | 11:47 |
fginther | WebbyIT, if you become a member of the app team, the tests will run automatically | 11:51 |
WebbyIT | fginther, the MR is not mine, I'm member of the team (rpadovani) and I've approved it | 11:52 |
fginther | WebbyIT, ah, in that case, if you top approve it, the tests will run | 11:53 |
WebbyIT | fginther, ah, ok, thanks :) | 11:53 |
fginther | WebbyIT, you're welcome. please let me know if it doesn't work as expected | 11:54 |
WebbyIT | fginther, sure, thanks! I approved it in this moment, I'll wait a couple of hours to see if Jenkins works | 11:54 |
dpm | fginther, no worries, thanks! I did not know (or didn't remember) that top-approval would trigger the test, will bear it in mind for the next ones | 12:00 |
mivoligo | nik90: what's the proper commit message? Should it be a bug number? | 12:04 |
WebbyIT | mivoligo, there is no rule, I usually write "Fixed #123 - What I fix - Other changes" | 12:05 |
mivoligo | WebbyIT: thanks | 12:06 |
mivoligo | nik90: I've pushed the code to lp and made a merge proposal :) I hope everything is ok, should I do something in the bug report? | 12:14 |
nik90 | mivoligo: can you attach the branch to the bug report? This way the bug report status will be automatically changed when the branch is merged in | 12:16 |
=== MacSlow is now known as MacSlow|lunch | ||
nik90 | mivoligo: also you need to add a commit message to the MP itself | 12:16 |
nik90 | thats the message the jenkins bot uses to merge to trunk | 12:17 |
nik90 | brb soon | 12:17 |
nik90 | mivoligo: small fix needed..qtcreator's fault | 12:40 |
mivoligo | nik90: manifest file? | 12:40 |
nik90 | mivoligo: yeah it is present in the application's root folder..it is not shown in qtcreator | 12:41 |
nik90 | if you look at the code diff, you will notice it has been changed | 12:41 |
mivoligo | nik90: yep, I've noticed that | 12:41 |
mivoligo | nik90: it changed automagically without me knowing ;) | 12:43 |
nik90 | mivoligo: yeah qtcreator has this bug :/ | 12:43 |
nik90 | I need to report it | 12:43 |
nik90 | mzanetti: when you have time, can you review the alarm face fix branch at https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-analogue-format/+merge/201282 | 12:44 |
nik90 | mzanetti: the feature you described (with automatic hour increment etc), I have reported them as bugs and will tackle it in the next MP. This alarm face fix MP has important fixes that need to land asap. | 12:45 |
ahayzen | timp, zsombi, can any of u guys have a look at this merge proposal for me? https://code.launchpad.net/~andrew-hayzen/ubuntu-ui-toolkit/fix-swipe-delete-002/+merge/202171 | 12:46 |
mzanetti | nik90: I'm really busy with unity and using the little time I have for the reminders-app. is there someone else who could do the review? | 12:46 |
mzanetti | if noone can do it, I will. | 12:46 |
nik90 | mzanetti: yeah sure I will try to find someone else | 12:46 |
nik90 | WebbyIT: could I interest you in a MR review? :) | 12:47 |
mivoligo | nik90: I've pushed again. Is that all? | 12:49 |
mzanetti | nik90: I try to find some time today, ok? | 12:49 |
nik90 | mivoligo: most likely, just testing the patch now | 12:50 |
mzanetti | right now in the middle of tweaking the right edge animation | 12:50 |
nik90 | mzanetti: no worries, I am more looking forward to the right edge animation :) | 12:50 |
nik90 | mivoligo: one same issue | 12:56 |
nik90 | mivoligo: http://imgur.com/2MdbqkW | 12:56 |
nik90 | mivoligo: the preset label text left indentation has messed up | 12:57 |
mivoligo | nik90: noticed that as well | 12:57 |
mivoligo | nik90: will look at that | 12:57 |
nik90 | mivoligo: it would be best to anchor the preset label to column rather than the presettimeLabel and give it a larger left margin to fix the issue | 12:57 |
nik90 | its a one liner fix | 12:58 |
mivoligo | nik90: ok, so go ahead ;) | 12:58 |
nik90 | mivoligo: I cant fix it in your branch, only u can :) | 12:59 |
mivoligo | nik90: ok ok | 13:00 |
WebbyIT | nik90, just finished the launch :-) Yeah, sure, link it to me :) | 13:12 |
nik90 | WebbyIT: https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-alarm-analogue-format/+merge/201282 | 13:12 |
nik90 | WebbyIT: thnx | 13:13 |
mivoligo | nik90: what you think about doing it like World clock locations to be consequent? | 13:13 |
WebbyIT | nik90, yw. There is something phone-related or I can test it on emulator? | 13:14 |
nik90 | WebbyIT: think it should work on an emulator..not sure..it is the new alarm design | 13:14 |
nik90 | mivoligo: hmm actually thats a great idea | 13:15 |
nik90 | mivoligo: I believe the stopwatch page also uses that format | 13:15 |
mivoligo | nik90: will do that later, need to go pick up my wife from shops now ;) | 13:16 |
nik90 | mivoligo: no worries, take your time :) | 13:16 |
timp | 13:46:04 < ahayzen> timp, zsombi, can any of u guys have a look at this merge proposal for me? https://code.launchpad.net/~andrew-hayzen/ubuntu-ui-toolkit/fix-swipe-delete-002/+merge/202171 | 13:21 |
timp | renato_: ^ could you review that? you are the most familiar with the swipe-to-delete | 13:22 |
renato_ | timp sure | 13:29 |
timp | renato_: thanks! | 13:29 |
mivoligo | nik90: I've pushed the fix | 13:58 |
nik90 | mivoligo: testing | 13:59 |
renato_ | timp, approved. but I can not top approve it | 14:00 |
renato_ | I do not have permission for that :D | 14:01 |
mivoligo | nik90: I guess one more thing to do would be to use “elide” for the preset name | 14:01 |
nik90 | mivoligo: approved | 14:01 |
nik90 | mivoligo: you want me to revert the approval? | 14:01 |
nik90 | mivoligo: actually nevermind | 14:02 |
nik90 | mivoligo: I will tackle all of the elide in clock, timer, stopwatch, alarm etc in another MP | 14:02 |
mivoligo | nik90: thanks | 14:02 |
mivoligo | nik90: my very first official contribution to the core apps ;) | 14:03 |
nik90 | mivoligo: that's right :) Has a nice ring to it | 14:03 |
mivoligo | nik90: BTW is there a bug for hands positions I've mentioned before, or is it fixed in some branch? | 14:05 |
nik90 | mivoligo: I think there is a bug for it..not sure... | 14:06 |
* nik90 checks | 14:06 | |
mivoligo | nik90: I've found this one: https://bugs.launchpad.net/ubuntu-clock-app/+bug/1217902 where it's mentioned in the bug description | 14:08 |
ubot2` | Launchpad bug 1217902 in Ubuntu Clock App "[Clock app] Clock view needs to match design" [Undecided,Incomplete] | 14:08 |
nik90 | mivoligo: yup that's the one. I just need to edit the bug to make it about just one issue | 14:09 |
mivoligo | nik90: is someone working on the fix already? | 14:09 |
timp | renato_: thank you | 14:09 |
nik90 | mivoligo: hey I got a question. In zeegaree, the app width is not allowed to go below a certain limit. How did you do that? | 14:09 |
timp | renato_: I'll run the autopilot tests on device and take care of the top-approval | 14:09 |
nik90 | mivoligo: as for your question, nope no one is working on it yet | 14:09 |
mivoligo | nik90: it's handled in the python file | 14:09 |
mivoligo | nik90: there's no way to set min width in pure QML | 14:10 |
ahayzen | timp, renato_, thanks guys :) | 14:10 |
nik90 | mivoligo: ah :/ too bad | 14:10 |
mivoligo | nik90: you have to believe in people's common sense :D | 14:15 |
nik90 | mivoligo: ? | 14:15 |
mivoligo | nik90: that they do not shrink their windows to much | 14:16 |
nik90 | mivoligo: ah ...yes I suppose | 14:16 |
timp | ahayzen: I'm flashing a new image on my device now, it will take a bit of time but I try to happrove it later today | 14:18 |
ahayzen | timp, thanks | 14:21 |
ahayzen | timp, should then be able to move the music-app onto the SDK swipedelete :) | 14:21 |
timp | cool :) | 14:21 |
mivoligo | nik90: I've tried bug #1272470 but StateSaver does not work for me, I'm on Ubuntu 12.04 | 14:24 |
ubot2` | Launchpad bug 1272470 in Ubuntu Clock App "Clock app should remember the app width and height in desktop mode" [Wishlist,Triaged] https://launchpad.net/bugs/1272470 | 14:24 |
timp | ahayzen: wow, that app looks very different from the last time I tried it on desktop :) | 14:24 |
ahayzen | timp, wht the music-app? | 14:25 |
timp | ahayzen: yes | 14:25 |
ahayzen | timp, by the way to test the swipe delete use this branch... | 14:25 |
ahayzen | timp, https://code.launchpad.net/~andrew-hayzen/music-app/reorder-support-001/+merge/202172 | 14:25 |
timp | and I like the transitions when I click next song button | 14:25 |
nik90 | mivoligo: oh yeah ..I think the fix hasn't been backported to 12.04 :/ | 14:25 |
ahayzen | timp, yeah we have been working on convergence stuff recently | 14:25 |
ahayzen | renato_, did u do the swipedelete in the SDK? | 14:26 |
renato_ | ahayzen, yes | 14:26 |
ahayzen | renato_, do u mind answering a few questions? | 14:26 |
ahayzen | renato_, basically before with our own implementation we had 'Clear' instead of 'Delete' and we had this with a transparent background | 14:27 |
timp | ahayzen: looks good :) and it may be one of the first apps doing that for desktop | 14:28 |
ahayzen | renato_, the issue with the one in the SDK is i can't see how to change the text. I tried putting a label in the background indicator, but then as we have a transparent colour u can see the default one behind :/ | 14:28 |
ahayzen | timp, thanks | 14:29 |
timp | ahayzen: why "clear"? Do you mean "remove" from the list (but not delete the song)? | 14:30 |
renato_ | ahayzen, I do not remember exactly how this was implemented I need to take a look to remember it, but have you tried to put a rectangle with a label | 14:30 |
ahayzen | timp, when we were doing design it felt like 'delete' was too permanent for removing an item from the track queue | 14:31 |
timp | ahayzen, renato_ currently the text is hard-code in a Label in an internal component. No way to change it for apps | 14:31 |
ahayzen | timp, so yes Clear/Remove would be preferred but the point was more how do u change it | 14:31 |
timp | ahayzen: I prefer "Remove" :) | 14:31 |
ahayzen | timp, renato_, there is a backgroundIndicator property u use to change the background, u can put a label inside tht but then as it is transparent it shows through | 14:32 |
ahayzen | timp, renato_, furthermore we wanted to change the colour of the text so it is easier to read on dark backgrounds | 14:32 |
renato_ | ahayzen, what I am saying is to you to use a rectangle instead a label | 14:32 |
timp | renato_, ahayzen what do you think of another MR for Empty.qml in uitk that adds an alias removeText: removeLabel.text ? | 14:33 |
ahayzen | renato_, but ur label shows through because we have transparent background, unless i'm not understanding wht u mean? | 14:33 |
ahayzen | timp, it would probably be useful to be able to set the colour of the text as well | 14:34 |
timp | ahayzen: then another option is property alias removeLabel: removeLabel | 14:34 |
renato_ | ahayzen, you can set a rectangle on the backgroundIndicator and use a label inside of your rectangle | 14:34 |
timp | ahayzen: then you can set removeLabel { text: "Remove/Clear", color: "red" } | 14:34 |
ahayzen | renato_, thats wht we did | 14:35 |
timp | ^that would only set the text color, not the background color | 14:35 |
ahayzen | renato_, but then as the Rectangle { opacity: .6 } ur label could be seen | 14:35 |
ahayzen | timp, yeah thts wht we want :) | 14:35 |
renato_ | ahayzen, ok | 14:35 |
renato_ | now I understand | 14:35 |
ahayzen | renato_, sorry difficult to explain | 14:36 |
renato_ | we should make the indicator visible: false if a backgroundIndicator was set | 14:36 |
ahayzen | renato_, tht would probably be the simplest option and wht i 'expected' to happen when i tried it | 14:36 |
timp | background color would be something to go into a style/theme, not a property of the list item. | 14:37 |
timp | I think.. unless there are good reasons to have different background colors in a single app | 14:37 |
renato_ | ahayzen, this is very simple to implement, could you add a bug for that | 14:37 |
* timp bbl | 14:37 | |
ahayzen | timp, thanks for ur help see u later | 14:38 |
ahayzen | renato_, yep will do :) thanks | 14:38 |
ahayzen | renato_, something like this, i couldn't think wht to call it https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1274140 | 14:43 |
ubot2` | Launchpad bug 1274140 in Ubuntu UI Toolkit "Swipe delete labels should not be visible when background indicator is set" [Undecided,New] | 14:43 |
renato_ | ahayzen, this is ok, thanks | 14:56 |
ahayzen | renato_, np | 14:57 |
mzanetti | WebbyIT: ping | 15:11 |
ahayzen | dpm, dpm_, was a resolution to the issue of the music-app not appearing in the software license list? | 15:15 |
WebbyIT | mzanetti, pong | 15:19 |
mzanetti | WebbyIT: hey, just discovered that your commit introduces a quite bad bug | 15:20 |
WebbyIT | mzanetti, sounds bad :/ What happens? | 15:20 |
mzanetti | WebbyIT: if you open an existing note, click edit and then press return, it will delete the note | 15:20 |
mzanetti | WebbyIT: I'd say we just revert your commit for now and drop the back button in edit note | 15:21 |
WebbyIT | mzanetti, ok, sounds veeeeery bad, I'll try to found a solution asap | 15:21 |
mzanetti | WebbyIT: so you always have to save (which is exactly like evernote works) | 15:21 |
WebbyIT | mzanetti, do you have some time now to explain how enum works with QML? I tried to read on internet, but my knowledge is too poor | 15:22 |
mzanetti | WebbyIT: sure | 15:22 |
mzanetti | so, do you have the enum in c++ already= | 15:23 |
mzanetti | ? | 15:23 |
WebbyIT | mzanetti, yap, I added in enmldocument.h and I modified all functions to accept EnmlDocument::Format format | 15:23 |
mzanetti | wait, I need to check the code... gimme a moment | 15:24 |
WebbyIT | mzanetti, I haven't pushed it | 15:24 |
mzanetti | yeah... I meant the existing code | 15:24 |
WebbyIT | ah ok :) | 15:24 |
mzanetti | WebbyIT: ok... so there is one issue: qml doesn't know anything about the EnmlDocument class | 15:26 |
mzanetti | WebbyIT: qml can only know about classes inheriting from QObject (where "public QObject" is at the end of the class definition) | 15:26 |
mzanetti | now, I don't want to make EnmlDocument a QObject for various reasons | 15:27 |
mzanetti | I need a few minutes to figure something | 15:27 |
WebbyIT | ok | 15:27 |
dpm_ | hi ahayzen, I didn't follow up further, sorry, but I'll send an e-mail to the phone mailing list | 15:28 |
ahayzen | dpm_, thanks | 15:29 |
popey | ahayzen: do we have a bug for it? | 15:29 |
ahayzen | popey, i don't think so, dpm_? | 15:29 |
popey | lets get a bug first | 15:29 |
dpm | +1 | 15:30 |
ahayzen | dpm_, popey, does it affect all click apps? | 15:30 |
* popey looks | 15:30 | |
dpm | ahayzen, I don't know, I'd start reporting it for music, and then we can add the others to the same bug if they're affected | 15:30 |
ahayzen | dpm_, popey, i'll add a bug report for the music app :) | 15:30 |
dpm | \o/ | 15:30 |
dpm | thanks | 15:31 |
dpm | I should probably rejoice more on bugs fixed, rather than filed :) | 15:31 |
dpm | but we need both! | 15:31 |
popey | i dont see any core apps in the list | 15:31 |
popey | only debs | 15:32 |
popey | no click packages | 15:32 |
popey | so that isnt a music app bug, thats a system settings one imo | 15:32 |
ahayzen | popey, dpm, https://bugs.launchpad.net/music-app/+bug/1274154 | 15:32 |
ubot2` | Launchpad bug 1274154 in Ubuntu Music App "Music app doesn't appear in software licenses list" [Low,New] | 15:32 |
dpm | awesome, thanks a lot ahayzen | 15:32 |
ahayzen | dpm, popey, shall i add that we suspect all click apps are affected? | 15:33 |
* popey has | 15:33 | |
ahayzen | popey, thanks | 15:33 |
dpm | I added a bug task for system settings | 15:34 |
dpm | yay team effort :) | 15:34 |
popey | confirmed | 15:34 |
popey | ^5 | 15:34 |
ahayzen | \o/ | 15:34 |
dpm | ^5 | 15:34 |
* popey tickles Laney | 15:34 | |
* popey tickles laney with bug 1274154 | 15:35 | |
ubot2` | Launchpad bug 1274154 in ubuntu-system-settings (Ubuntu) "Music app doesn't appear in software licenses list" [Undecided,Confirmed] https://launchpad.net/bugs/1274154 | 15:35 |
popey | we spoke about this before if I recall. | 15:35 |
Laney | We get emailed about bugs | 15:35 |
Laney | there is no need to ping | 15:35 |
popey | that wasn't my intention, i wanted to ask a question | 15:35 |
popey | hey ho | 15:35 |
seb128 | popey, the licences don't list clicks | 15:35 |
Laney | okay | 15:36 |
Laney | it's because it only lists apt packages currently | 15:36 |
seb128 | popey, are click providing a copyright file/license text in a standard location? | 15:36 |
seb128 | popey, click list --manifest doesn't seem to include license info | 15:37 |
popey | seb128: I think you're right | 15:38 |
=== oSoMoN_ is now known as oSoMoN | ||
* popey rummages | 15:38 | |
Laney | I'll just add a click task there | 15:38 |
seb128 | Laney, thanks | 15:38 |
seb128 | I added a comment already asking for how to get the info | 15:39 |
seb128 | we didn't do it by then because there was no way to get those details | 15:39 |
popey | It's interesting because I don't think the store or click obligates developers to add a license to their app. | 15:40 |
popey | but anyway, low priority, sorry for interrupting you Laney / seb128 | 15:40 |
seb128 | then we can't list one... | 15:40 |
popey | indeed | 15:40 |
Laney | sounds dubious, but NMP | 15:40 |
seb128 | popey, no worry, you might want to talk to the click team about that | 15:40 |
seb128 | what Laney said :p | 15:40 |
popey | ok. | 15:40 |
seb128 | we are happy to add those to the list if click provides the info we need | 15:41 |
seb128 | until then, not a lot we can do | 15:41 |
Laney | thanks for filing it | 15:41 |
Laney | once this is done, we should require that all preinstalled applications are ubuntu-free and marked as such imho | 15:44 |
seb128 | Laney, "ubuntu-free"? | 15:46 |
Laney | http://people.canonical.com/~cjwatson/ubuntu-policy/policy.html/ch-archive.html#s-ulp | 15:46 |
seb128 | thanks | 15:47 |
mardy | mzanetti: hi! I'm having a look at the evernote account plugin, and trying to figure out how to obtain the username | 15:50 |
mzanetti | mardy: we need to merge the plugin code into lp:reminders-app for that | 15:50 |
ybon | I've a ListView inside a DefaultSheet opened in popup mode, the flick event on the listview is propagated to the mainview (i.e. a map in my case), I guess I'm doing something wrong but can't find what. If anyone feel to have any hint (code here: http://bazaar.launchpad.net/~yohanboniface/osmtouch/trunk/view/head:/components/PoiSheet.qml) Thanks :) | 15:51 |
=== mhall119_ is now known as mhall119 | ||
mardy | mzanetti: but if the plugin will do "import Evernote 0.1", will it be able to resolve the plugin? I doubt it, if it's part of a click package | 15:51 |
ybon | I've tried to set explicitally "modal: true" but same | 15:52 |
mzanetti | mardy: yeah.. that's why... we need to ship the evernote plugin with the account plugin too | 15:52 |
mzanetti | mardy: in the end we'll have it twice on the system :/ | 15:52 |
mzanetti | but that's how click works | 15:52 |
mardy | mzanetti: right | 15:52 |
mardy | mzanetti: I was wondering, is the evernote SDK open source? can we see how the UserStore class works, and just replicate it in QML/JS in the account plugin? | 15:53 |
dpm | mardy, also you asked me this earlier: so the evernote SDK is open source, yes, and all the documentation is public. They've got an "SDK" for each language binding you can use the API for. They've not got a REST API, so you're using it natively with your language of choice. And the bindings themselves are auto-generated by Thrift. In our case we could only use C++ | 15:53 |
mzanetti | mardy: have fun with that :D | 15:53 |
mzanetti | mardy: its generated code using libthrift | 15:53 |
mardy | mzanetti: no idea what that is :-) Have you looked at it? You make it sound difficult :-) | 15:54 |
dpm | mardy, and you can't use their JS SDK from QML due to JS features they use which are not supported by Qt's JS engine | 15:54 |
mzanetti | mardy: lets say its one of the the most masochistic c++ codebases I've seen so far | 15:54 |
dpm | mardy, please, don't get mzanetti started talking about thrifht! :-) | 15:55 |
mardy | dpm: which exactly? the JS engine in QML changed with Qt 5.2, maybe now it works (though I seriously doubt, as the new engine is probably simpler) | 15:55 |
mzanetti | mardy: but for maintenance I think its best if we use the existing plugin nevertheless | 15:55 |
mzanetti | mardy: instead of having 2 different plugins | 15:55 |
mardy | mzanetti: mmm... libevernoteplugin.so seems to be 22 megs :-O | 15:57 |
ybon | humm, still the same if I try to set the ListView as dismissArea | 15:57 |
mzanetti | libthrift... | 15:57 |
mzanetti | mardy: ^ | 15:57 |
mardy | mzanetti: :-) | 15:57 |
mardy | mzanetti: OK, I see why you dislike it :-) | 15:57 |
mzanetti | mardy: it'll get smaller in release builds tho | 15:57 |
mardy | I need to go now, will get back at it tomorrow | 15:57 |
mzanetti | mardy: and no... this is really not the reason I dislike it :D | 15:58 |
dpm | mardy, see points 1) and 2) to answer the question on JS here -> http://pastebin.ubuntu.com/6838544/ it's from an e-mail I sent a while ago to someone asking about why we weren't using the JS SDK | 15:58 |
mzanetti | mardy: its more the fact that it throws exception like crazy, uses boost just for the sake of using it, and well, it's api really feels like generated code too | 15:58 |
dpm | lol | 15:58 |
renato_ | nik90, https://code.launchpad.net/~renatofilho/qtorganizer5-eds/fix-1272333/+merge/203777 | 16:05 |
renato_ | could you help me to test it? | 16:05 |
mzanetti | WebbyIT: https://code.launchpad.net/~mzanetti/reminders-app/textformat-enum/+merge/203779 | 16:05 |
mzanetti | WebbyIT: please merge this into your branch. if it works for you, approve this merge | 16:08 |
mzanetti | WebbyIT: you should be able to use the enum like this: | 16:08 |
mzanetti | in QML: TextFormat.Bold, TextFormat.Italic etc. You can even define flags like "TextFormat.Bold | TextFormat.Italic" | 16:09 |
WebbyIT | mzanetti, ok, thanks. Can I assign to it a value? | 16:09 |
mzanetti | in C++: TextFormat::Bold, TextFormat::Italic. if you want to have the flags you can use "format.testFlag(TextFormat::Bold)" to test if it is set for example | 16:10 |
mzanetti | WebbyIT: yeah, in the note, create a function like this: | 16:10 |
mzanetti | Q_INVOKABLE void format(int start, int end, TextFormat::Formats); | 16:10 |
mzanetti | WebbyIT: then you can call it from qml like this: "note.format(5, 10, TextFormat.Bold | TextFormat.Italic)" | 16:11 |
mzanetti | to make 5 to 10 bold and italic in one call | 16:11 |
mzanetti | WebbyIT: in c++, you need to include textformat.h wherever you want to use it | 16:11 |
WebbyIT | mzanetti, and if start positions aren't the same? | 16:12 |
mzanetti | format(5, 5, flags) ? | 16:12 |
mzanetti | altough I'm not sure this makes any sense | 16:12 |
WebbyIT | mzanetti, eg italic from 5 to 10, bold from 7 to 10 | 16:12 |
mzanetti | ah. in that case you need 2 calls | 16:13 |
mzanetti | format(5, 10, TextFormat.Bold) | 16:13 |
mzanetti | format(7, 10, TextFormat.Italic) | 16:13 |
mzanetti | well, the other way round | 16:13 |
mzanetti | WebbyIT: let me know if you have troubles. | 16:14 |
mzanetti | or also if it works, so we can merge my textformat branch | 16:14 |
WebbyIT | mzanetti, ok, I'm going to try it, thanks for the support :) | 16:15 |
mzanetti | no problem | 16:15 |
=== gatox is now known as gatox_lunch | ||
nik90 | renato_: I will be back in a few minutes to test it | 16:38 |
nik90 | renato_: I am trying to install armf deb file on the phone | 16:55 |
nik90 | renato_: do you know how to convert your phone to a write image? | 16:55 |
ogra_ | nik90, touch /userdata/writable_image ... reboot | 17:02 |
ogra_ | err | 17:02 |
ogra_ | nik90, touch /userdata/.writable_image ... reboot | 17:02 |
ogra_ | (missed the dot) | 17:02 |
nik90 | ogra_: I did adb shell touch /userdata/.writable_image && adb reboot | 17:02 |
nik90 | ogra_: I found my several month old readme file :) | 17:02 |
ogra_ | that should be enough to install a deb | 17:02 |
nik90 | thn | 17:02 |
nik90 | thnx | 17:03 |
WebbyIT | mzanetti, mhh, I rewrite my code and I'm trying to launch the app, but I have an error in EditNotePage.qml | 17:09 |
WebbyIT | Error is Error: Unknown method parameter type: TextFormat::Formats | 17:09 |
WebbyIT | and it refers to note.format(modifier.boldStartPosition, modifier.currentPosition, TextFormat.Bold); | 17:09 |
nik90 | renato_: As of testing it now, it works! However, this is something I need to test today and tomorrow to confirm that it fixed the issue. | 17:09 |
nik90 | renato_: Although it does create another bug :P | 17:09 |
nik90 | renato_: In the bug report, I attached a sample alarm app. Using that app, if you create a recurring alarm say for Mon, Tue and Wed. Then when the alarm is saved, the alarm list shows 3 alarms instead of one. | 17:10 |
nik90 | renato_: However, if you delete any one of them, it deletes all of them. | 17:11 |
renato_ | nik90, I need to discuss that with zsombi, this works that way because the calendar app | 17:14 |
mzanetti | WebbyIT: did you merge my branch and rebuit? | 17:14 |
renato_ | but anyway this is a different bug | 17:14 |
WebbyIT | mzanetti, yap | 17:14 |
nik90 | renato_: okay, should I write this as a comment in your MR? so that we dont lose track of it | 17:14 |
mzanetti | WebbyIT: hmm... should work if the page imports Evernote 0.1 | 17:14 |
renato_ | nik90, and probably need to be fixed on alarm API | 17:14 |
nik90 | renato_: ah okay | 17:15 |
renato_ | I think you should open a new bug | 17:15 |
nik90 | renato_: I will inform zsombi tomorrow morning of this issue. | 17:15 |
nik90 | renato_: okay will report it against the UITK Alarm API | 17:15 |
WebbyIT | mzanetti, I have no error in C++ code, so I think that import works, but I dont'understand why it doesn't work in qml | 17:15 |
nik90 | renato_: although it nicely fixes the other issue I had where the alarm status was reversed sometimes :) | 17:15 |
mzanetti | WebbyIT: I jut tried it here... works fine | 17:16 |
mzanetti | WebbyIT: please post the whole error | 17:16 |
mzanetti | paste... | 17:16 |
mzanetti | ah... now I see | 17:16 |
* mzanetti tries | 17:17 | |
=== gatox_lunch is now known as gatox | ||
nik90 | mefrio: thnx for your fix to https://bugs.launchpad.net/bugs/1272470 | 17:17 |
ubot2` | Launchpad bug 1272470 in Ubuntu Clock App "Clock app should remember the app width and height in desktop mode" [Wishlist,In progress] | 17:17 |
mefrio | nik90, you and the Ubuntu project are welcome! :) | 17:17 |
nik90 | mefrio: is it ready for a merge? | 17:18 |
mefrio | nik90, yes | 17:18 |
nik90 | mefrio: if yes you should propose a MR | 17:18 |
nik90 | this way I can review and give comments | 17:18 |
mefrio | nik90, yeah, I forgot the MR | 17:18 |
nik90 | :) | 17:18 |
mefrio | nik90, here it is: https://code.launchpad.net/~mefrio-g/ubuntu-clock-app/fix-1272470/+merge/203796 | 17:18 |
* nik90 is testing the MP | 17:19 | |
nik90 | mefrio: in my testing the minimum width,height logic did not work | 17:21 |
nik90 | mefrio: it still opened with the same application size | 17:21 |
mefrio | nik90, I used 50/2 gu for the width and 75/2 gu for height | 17:22 |
mefrio | it worked here...let me check | 17:22 |
nik90 | mefrio: yeah I reduced the application width to less than 10 gu units to check the condition, but it didnt work for me | 17:22 |
nik90 | mefrio: so I tried changing the logic condition to 50 gu and 75 gu for height..still didnt | 17:23 |
=== boiko_ is now known as boiko | ||
mefrio | nik90, ouch! It's bugged :( | 17:24 |
mefrio | nik90, the problem is in the logic condition. It should be an OR | 17:25 |
mefrio | not an AND | 17:25 |
nik90 | ah :) | 17:25 |
nik90 | mefrio: one more thing | 17:25 |
nik90 | mefrio: can you revert the manifest.json file change | 17:25 |
nik90 | mefrio: bzr revert -r revno manifest.json | 17:25 |
nik90 | for revno, enter 308 | 17:25 |
mefrio | nik90, sure! | 17:26 |
nik90 | mefrio: and can you change the condition to 50 and 75 gu respectively. instead of 50/2 and 75/2 | 17:26 |
mefrio | nik90, when I opened the project with the SDK it automatically changed the manifest.json | 17:26 |
nik90 | since 50/2 is actually quite small | 17:26 |
mefrio | ok no problems | 17:26 |
nik90 | mefrio: true, it is a qtcreator bug :/ | 17:27 |
nik90 | so annoying bug | 17:27 |
nik90 | popey: who is the contact person for qtcreator plugin bugs? | 17:27 |
mefrio | nik90, ok I pushed the code | 17:28 |
mefrio | nik90, let me know if it works | 17:28 |
nik90 | will do | 17:28 |
nik90 | mefrio: yup works perfectly! nice work! | 17:30 |
mefrio | nik90, great! Thank you! | 17:30 |
=== bfiller is now known as bfiller_afk | ||
mzanetti | WebbyIT: hey | 17:58 |
WebbyIT | mzanetti, did you fix it? :-) | 17:59 |
mzanetti | WebbyIT: its not working indeed and I can't really figure why | 17:59 |
mzanetti | WebbyIT: as a temporary workaround, please change the parameter type of Note::format() to int instead of format | 17:59 |
mzanetti | WebbyIT: you can still use the Nice enum value in qml | 17:59 |
mzanetti | as in call note.format(5, 10, TextFormat.Bold) | 18:00 |
mzanetti | I'll continue trying figure what is wrong | 18:00 |
WebbyIT | mzanetti, I'l try, thansk | 18:01 |
WebbyIT | *thanks | 18:01 |
samertm | hey guys | 18:08 |
samertm | I'm new to Launchpad | 18:08 |
samertm | is there a way to see all of the commits I've made that have been merged into the trunk? | 18:08 |
WebbyIT | samertm, https://code.launchpad.net/~samer-f/ and choose "Any status" | 18:10 |
samertm | WebbyIT: thanks!! | 18:11 |
WebbyIT | mzanetti, sorry, but I have not understand what I have to change | 18:16 |
* WebbyIT is so clumsy | 18:16 | |
mzanetti | WebbyIT: I just found the issue. gimme a minute and I'll update my branch | 18:17 |
WebbyIT | mzanetti, you rock \o/ | 18:17 |
mzanetti | WebbyIT: pushed | 18:27 |
wardane | I have a quick question. If you have multiple views of windows that you want to be in your application. How do you get the program to switch between the views when a button is pressed? | 18:32 |
popey | nik90: someone on bzoltan's team | 18:35 |
nik90 | popey: erm like? | 18:36 |
popey | nik90: i don't know, hence highlighting him ☻ | 18:37 |
nik90 | timo perhaps? | 18:37 |
WebbyIT | samertm, you're a great bugs hunter :D | 18:37 |
popey | no, timo looks after qt | 18:37 |
nik90 | ok :) | 18:37 |
samertm | WebbyIT: thanks :) if I knew it was this easy to contribute I would have started ages ago | 18:37 |
samertm | the Ubuntu Core Apps Hack Days were all the push I needed | 18:38 |
popey | samertm: that's awesome to hear! | 18:39 |
WebbyIT | samertm, nice to hear \o/ | 18:39 |
mzanetti | samertm: are we allowed to quote that in the interwebs? | 18:40 |
samertm | mzanetti: sure :P | 18:40 |
mzanetti | popey: ^ ;) | 18:40 |
mhall119 | mzanetti: \o/ | 18:41 |
* mhall119 does a happy dance | 18:41 | |
WebbyIT | mzanetti, your branch works as expected \o/ | 18:48 |
mzanetti | WebbyIT: nice | 18:49 |
WebbyIT | mzanetti, do I approve it on lp? | 18:49 |
mzanetti | CI is failing... we need to land the cleanup branch first | 18:49 |
popey | mzanetti: ☻ | 18:51 |
ybon | popey: I've pushed some changes, like a marker on your position when GPS is enabled, a marker to show the exact position of a search, and mainly a fix on the ListView that was freezing (cf 1274150), feel free to run new tests :) | 18:54 |
ybon | popey: also, what do you think should be the next step to get more feedback: publishing the 0.1.0 click? sending an email on the ML? | 18:55 |
popey | both ☻ | 18:56 |
ybon | hehe, okay :) | 18:56 |
popey | also, blog, G+ in the app dev community | 18:56 |
popey | make lots of noise ☻ | 18:56 |
ybon | well, that's not what I'm good at ;) | 18:56 |
ybon | And I'm like better dead than on G+ :p | 18:57 |
popey | hah | 18:57 |
popey | ok | 18:57 |
popey | the mailing list is good, other people can pimp it elsewhere | 18:57 |
ybon | yeah, will do that :) | 18:58 |
ybon | also, tell me if you test it and you feel it can be published as 0.1.0 :) | 18:58 |
popey | just making dinner, will test shortly | 18:59 |
ybon | thanks | 18:59 |
popey | ybon: getting flickering on device | 19:08 |
popey | oh, might be my wifi | 19:09 |
popey | black background while loading tiles. might be nice to have a background tile which is "Loading..." or an icon (no translation required) which implies loading | 19:09 |
ybon | that's true | 19:10 |
popey | ybon: on desktop i get a blue dot on my location when i use "position" but not on the device | 19:10 |
ybon | I'm gonna give a try on this | 19:10 |
ybon | popey: it displays it only if the location source is GPS | 19:10 |
ybon | so not from IP | 19:10 |
ybon | (unless on desktop) | 19:10 |
ybon | this is because GeoIP on mobile is very approximative | 19:11 |
bzoltan | nik90: popey: that is me ... what is the problem? | 19:11 |
popey | ybon: i get same location on both because wifi ip is same | 19:11 |
popey | (i.e. not on 3g, so not got a random roaming ip) | 19:11 |
ybon | ok let me check that :) | 19:11 |
ybon | do you agree that the marker make sense only if location is accute enough? | 19:11 |
popey | yes | 19:12 |
ybon | k | 19:12 |
bzoltan | nik90: popey: fire a bug here if you need something -> https://bugs.launchpad.net/qtcreator-plugin-ubuntu/+bugs?orderby=-datecreated&start=0 | 19:12 |
* popey hugs bzoltan | 19:12 | |
bzoltan | It is a love day :) I got hugs from many people | 19:13 |
popey | \o/ | 19:13 |
popey | we're a very huggy company | 19:13 |
bzoltan | nik90: popey: the manifest file handling is not the most used part of the QtC and I have received little if any feedback... so bugs are welcome! Just assign to me and ping me here | 19:14 |
popey | kk | 19:15 |
ybon | grr, I don't why, no way to put a Rectangle below the map and make it visible | 19:20 |
popey | ybon: might want to change "Around me" to "Nearby" for when you are looking at a part of the map which isn't technically "around me" | 19:21 |
ybon | sounds good, yep | 19:21 |
=== bfiller_afk is now known as bfiller | ||
samertm | btw you guys | 19:28 |
samertm | I've been exposed to a lot of the core apps in the last couple days | 19:28 |
samertm | who designed all these @_@ they look amazing | 19:29 |
qtros | Hi guys! | 19:33 |
qtros | I have little trouble with my Ubuntu SDK - currently it says that there are no kits found. Can someone tell me where I should search? :) | 19:33 |
nik90 | bzoltan: sorry, I was in a call | 19:34 |
nik90 | bzoltan: yeah I had a bug where when you open qtcreator, it changes the manifest.json file | 19:35 |
nik90 | bzoltan: as a result, I need to keep reverting it before pushing to the core apps project | 19:35 |
nik90 | bzoltan: I will report a bug and let you know | 19:35 |
popey | samertm: it was a collaboration between the canonical design team and the community developers | 19:35 |
stefano92100 | Hello, I need someone to help me in a quick test. 1) Open clock-app; 2) go to Timer tab; 3) set the timer, so "Reset" button becomes visible; 4) click the "Reset" button; 5) Flickable will bounce and header will be hidden. | 19:36 |
stefano92100 | I need someone with Ubuntu 14.04 on his PC. I've tried on tablet and this does not happen. I've tried then on Ubuntu 13.04, and there's no problem too. | 19:36 |
popey | i suspect this is the OSK not appearing? | 19:37 |
nik90 | stefano92100: actually for me the header does hide | 19:37 |
nik90 | stefano92100: I have a fix in my alarm MP already, should land hopefully tonight | 19:38 |
stefano92100 | nik90: Awesome! | 19:38 |
stefano92100 | To report a wrong translation in an application, where have I to go? Just fill a bug as usual? | 19:39 |
nik90 | stefano92100: well if the translated string is wrong in a particular language, you do not need to report a bug | 19:40 |
nik90 | stefano92100: you can change the translation yourself through the launchpad website | 19:40 |
nik90 | stefano92100: I guess someone will review your translation and approve it | 19:41 |
qtros | Guys... Please, help me :) Qt Creator -> Options -> Build and Run -> Kits. Tell me paths, please ... | 19:41 |
samertm | popey: they did good work :P | 19:42 |
stefano92100 | nik90: ok, thanks. Now I have to think a good translation, because the problem is quite complex... | 19:42 |
nik90 | stefano92100: :) | 19:42 |
nik90 | qtros: let me check my comp | 19:42 |
qtros | nik90 thx! | 19:43 |
nik90 | qtros: http://i.imgur.com/aY1hNeQ.png | 19:44 |
nik90 | qtros: http://imgur.com/QYv37qL | 19:44 |
ybon | popey: done for "nearby", but https://bugs.launchpad.net/osmtouch/+bug/1274262 :s | 19:44 |
ubot2` | Launchpad bug 1274262 in OSMTouch "Add a "Loading..." message when tiles are loading" [Undecided,New] | 19:44 |
popey | samertm: that's really kind of you to say. | 19:46 |
samertm | popey: you should pass that along to the designers responsible :P | 19:47 |
popey | will do! | 19:48 |
nik90 | bzoltan: https://bugs.launchpad.net/qtcreator-plugin-ubuntu/+bug/1274265 | 19:49 |
ubot2` | Launchpad bug 1274265 in qtcreator-plugin-ubuntu "manifest.json is changed when opening a project resulting in incorrect project maintainer" [Undecided,New] | 19:49 |
=== salem_ is now known as _salem | ||
nik90 | ybon: I fixed the world clock update bug you reported :)...just need to be reviewed and merged | 21:16 |
ybon | \o/ | 21:18 |
ybon | nik90: have you tried OSMTouch yet? :) | 21:18 |
nik90 | ybon: I tried on my desktop few days back. Nice app | 21:18 |
ybon | It's time for you create bug reports for me, so ;) | 21:18 |
nik90 | ybon: looking forward to seeing it in the store. | 21:18 |
ybon | :) | 21:19 |
nik90 | ybon: ah :P..will do | 21:19 |
ybon | On its wya :) | 21:19 |
ybon | way* | 21:19 |
nik90 | ybon: may be I might even propose a fix :) | 21:19 |
ybon | \o/ | 21:19 |
=== seb128_ is now known as seb128 | ||
popey | ybon: bounced the app back to you with one error | 21:54 |
ybon | may I say thanks? ;) | 21:55 |
* ybon running to see and fix it :) | 21:55 | |
ybon | ahem, popey I'm not using emails in yohanboniface.me (only for my personal website), so maybe it's better to switch back to a more conventional com.ubuntu.developer.yohanboniface? | 21:57 |
ybon | (which I think I can't do myself | 21:57 |
ybon | ) | 21:57 |
popey | you might be able to submit a new app | 21:59 |
popey | i dont know | 21:59 |
ybon | I also can configure an email in @yohanboniface.me if it's simpler | 21:59 |
popey | it would be easier, but it's up to you | 21:59 |
ybon | okay, let's do that :) | 22:00 |
ybon | will I be able to resubmit only the click package? | 22:00 |
popey | not sure, I've not had to do this before | 22:01 |
popey | sorry | 22:01 |
popey | probably | 22:01 |
ybon | hehe ok | 22:02 |
popey | as it's only failing on the click package tests itself, not the metadata on the website | 22:02 |
ybon | ah, damn, the "me.yohanboniface" is attached to my account | 22:03 |
popey | beuno may be able to help you but I don't think he's around right now | 22:04 |
ybon | ahem, "You already have a Click Package with this package name." | 22:17 |
ybon | I've deleted the first one, but seems stil alive somewhere :s | 22:18 |
popey | ☹ | 22:22 |
popey | yeah, i think you need beuno or james_w to clean things up | 22:22 |
ybon | okay, let's wait for the saviour, so :) | 22:23 |
james_w | I don't have access to the admin to change that sort of thing, sorry | 22:35 |
ybon | :( | 22:35 |
ybon | The amount of potential saviours has been suddently divided by 2 | 22:36 |
nik90 | lol | 22:40 |
nik90 | popey: do you agree to the following keyboard shortcuts -> http://paste.ubuntu.com/6840605/ | 22:40 |
WebbyIT | mzanetti, actual approch it's wrong, there are a lot of corner case and problems. I have an idea, but I think I'll take a couple of day to implement it, also because tomorrow I have an exam | 22:57 |
WebbyIT | but I use function you implemented today! | 22:59 |
* nik90 wishes WebbyIT all the best for his exam :) | 23:02 | |
WebbyIT | thanks nik90 :) It's english exam, I'm not so good :-P | 23:02 |
nik90 | hehe | 23:02 |
popey | nik90: two keys for lap seems odd. why not enter perhaps? | 23:09 |
popey | if I was timing something and wanted to bash a button to mark a lap, I'd want a big target to hit, as I may not be looking at the keyboard, but at the line where the player crosses the line | 23:09 |
nik90 | popey: it was in the convergence document | 23:09 |
nik90 | popey: but your reasoning makes sense | 23:10 |
nik90 | I will change it | 23:10 |
popey | thanks | 23:10 |
daker | nik90: enter or space key | 23:18 |
nik90 | daker: space is used to start/stop the stopwatch | 23:18 |
nik90 | daker: so I suppose Enter key makes sense to create a lap | 23:18 |
daker | so enter | 23:19 |
nik90 | popey: done. will ask someone tomorrow to review it | 23:21 |
popey | nice one | 23:21 |
popey | thanks nik90 | 23:21 |
popey | WebbyIT: good luck! your english is great ☻ | 23:21 |
popey | \o/ bed | 23:22 |
popey | ttfn | 23:22 |
WebbyIT | popey, too kind, thanks :-) | 23:22 |
WebbyIT | bed time also for me, see you soon o/ | 23:22 |
samertm | night | 23:24 |
=== samertm is now known as samertm_afk | ||
Mihir | WebbyIT: best of luck :) | 23:45 |
=== samertm_afk is now known as samertm |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!