/srv/irclogs.ubuntu.com/2012/04/04/#ubuntuone.txt

JamesTaitThere's snow day like Wednesday - good morning all! :D08:13
=== vds` is now known as vds
urbanapemorning, all. I have a doctor's appt. this morning at 9:40 local time. I *should* be back for standup.11:48
nessitahello everyone!11:54
gatoxnessita, hi11:57
nessitahola gatox11:59
alecuhello, all!12:03
gatoxalecu, hi!12:06
alecunessita, ping?12:22
nessitahola alecu!12:23
dobeyhi all13:05
gatoxdobey, hi13:06
dobeygatox: go to doctor13:06
gatoxdobey, :P13:06
alecunessita, http://pastebin.ubuntu.com/914494/13:08
nessitaalecu: looking13:11
gatoxbrb13:24
gatoxback13:31
zacktuI want to do a complete uninstall so that I can reinstall and have the option of using an existing account.  The U1 uninstall doesn't remove config files.  Where are they?13:48
gatoxzacktu, you don't need to do that..... you can just remove the current device from the control panel13:49
zacktugatox: let me check on that to see whether i understand what you're saying13:50
gatoxzacktu, ok.... it's in the "Devices" tab inside the u1 control panel....... remove the local device... and the login widget should appear13:51
zacktui found it -- i'll explore this -- thanks much13:51
zacktubye13:51
gatoxzacktu, bye13:52
dobeynessita: so we just need to tweak the debian/copyright for storage-protocol it seems14:21
nessitadobey: one sec, I'm in mumble with alecu14:22
dobeysure14:22
alecuelopio, ping14:31
nessitadobey: hi there :-)14:41
nessitadobey: so, about the debian/copyright file... I was wondering why is different from other copyright files I've seen14:41
elopioalecu: pong.14:41
nessitadobey: I can paste examples to be more exact14:41
nessitadobey: this is protocol http://pastebin.ubuntu.com/914609/ and this is controlpanel http://pastebin.ubuntu.com/914611/14:42
nessitadobey: shall we migrate the protocol's license to the dep5 debian format?14:43
dobeynessita: it's the old format it seems. it needs to be updated14:43
dobeyyes14:43
nessitaright14:43
nessitadobey: so, why is the protocol AGPL?14:43
dobeynessita: because that's what we chose, so that people making servers on it have more of a requirement to make changes available14:44
dobeyAGPL clarifies some things about "distribution" in terms of networked servers, which GPL doesn't14:44
nessitaah, did not know that14:45
nessitadobey: also, this is http://pastebin.ubuntu.com/914615/ ussoc's copyright14:45
nessitawhich is yet another format, no?14:45
dobeynessita: no that one needs fixed to include the header stuff, and probably drop a few of the bits14:46
dobeythe Upstream-* stuff should be removed i think; and the copyright and license fields need updated14:46
nessitadobey: so we should "migrate" all to be http://dep.debian.net/deps/dep5/ compliant?14:48
dobeynessita: ideally, yes14:48
nessitadobey: ok, will migrate protocol and ask you for review, sounds good?14:48
dobeynessita: sure. it should basically be the same as the one in ubuntuone-dev-tools now, except for AGPL3 instead of GPL314:49
nessitadobey: nice pointer14:49
dobeyin 2.99.92-0ubuntu2 that is14:50
nessitadobey: question, AGPL is not under /usr/share/common-licenses/14:51
nessitais that an issue?14:52
dobeywe might need to include the full text then14:53
nessitadobey: who can confirm that?14:53
dobey#ubuntu-devel14:53
dobeythough AGPL/ALGPL really should be in that directory14:54
nessitadobey: any reason to use "Format: http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn" instead of "Format: http://dep.debian.net/deps/dep5/"?14:59
nessitame15:00
briancurtinme15:00
nessitaalecu, gatox, urbanape, dobey, thisfred: standup?15:00
gatoxme15:01
nessitaroberto and manuel are off today15:02
nessitaalecu, urbanape, dobey, thisfred: standup?15:02
dobeymeh15:02
alecuoh, me.15:02
briancurtini think urbanape may still be stuck at the doctor15:02
alecunow, where are my notes!15:02
nessitaalecu: the dog ate those15:02
nessitaok, let's15:04
nessitaDONE: ubuntu releases15:04
nessitaTODO: finish ubuntu releases, reviews, train alecu in teach lead duties15:04
nessitaBLOCKED: nopes15:04
nessitaNEXT: briancurtin15:04
briancurtinDONE: working on tests, quick mac call, more tests15:04
briancurtinTODO: try the winsys pkg for setting proper ACLs since improper access is killing some windows tests15:04
briancurtinBLOCKED: no15:04
briancurtinNEXT: gatox15:04
gatoxDONE:15:04
gatoxKeep investigating Bug #940669, follow up of some old bugs and close them because were already fixed, work on Bug #824574 (fixed the current issue, but the fix insert another problem, so not ready yet), Working on Bug #907479.15:04
gatoxTODO:15:04
gatoxFinish with u1sdtool -q should not start syncdaemon when syncdaemon is not running issue, keep working in the other issues.15:04
gatoxBLOCKED:15:04
ubot5Launchpad bug 940669 in ubuntu-sso-client (Ubuntu) "ubuntu-sso-login crashed with SIGSEGV in QSocketNotifier::setEnabled()" [High,Confirmed] https://launchpad.net/bugs/94066915:04
gatoxNo15:04
ubot5Launchpad bug 824574 in Ubuntu One Control Panel "QT UI scroll bar overlaps list area bottom border" [Low,Triaged] https://launchpad.net/bugs/82457415:04
gatoxdobey, go15:04
ubot5Launchpad bug 907479 in Ubuntu One Client "u1sdtool -q should not start syncdaemon when syncdaemon is not running" [Medium,Triaged] https://launchpad.net/bugs/90747915:04
dobeyλ DONE: releases, uploads15:05
dobeyλ TODO: finish uploads, more bug #968555, more of #96926215:05
dobeyλ BLCK: none.15:05
ubot5Error: Launchpad bug 968555 could not be found15:05
dobeyalecu15:05
alecuDONE: braindumping nessita; ssl debugging on windows15:05
alecuTODO: more braindumping, more national holidays15:05
alecuBLOCKED: no15:05
alecuNOTE: I'll return on monday, after a new round of natholidays \o/15:05
alecuNEXT: ?15:05
nessitathisfred?15:05
briancurtintodo: move somewhere that has tons of holidays15:05
gatoxbriancurtin, :P15:06
nessitaok, any comments anyone?15:07
gatoxnop15:07
nessitadobey: saw my querstion re "any reason to use "Format: http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn" instead of "Format: http://dep.debian.net/deps/dep5/""?15:12
thisfredme15:13
thisfredDONE: u1db mappin15:13
thisfreds15:13
thisfredTODO: split words mapping15:14
dobeynessita: no, the shorter link is better i think. i wasn't aware of it though.15:14
thisfredBLOCKED: no15:14
elopioalecu, you pinged me.15:15
elopiowhat can I do for you?15:15
nessitadobey: ack, second question: I asked in ubuntu-devel, and seems like APGL is not common enough to be distributed in common-licenses. They said we have to add the full text in the debian/copyright file. Shall I do that or distribute a new file called "LICENSE" or similar?15:15
alecuelopio, oh, right. I've been trying the windows installer that briancurtin sent on a pristine win7 vm15:15
alecuelopio, and I'm not able to get past the login screen.15:16
alecuelopio, (the logs show some ssl handshake error)15:16
elopioalecu: with that one I just tried the update. I'm to test it today.15:16
elopioI'll install it quickly to see if the login works here.15:17
alecuelopio, so, I'm pinging you to know if you've found somehting like that, or not.15:17
alecuelopio, the key is that this was a pristine vm, so it u1 was never installed on it.15:17
dobeynessita: has to be in the copyright file15:17
alecuelopio, I've tried running the code in my win7 dev vm, and it works fine.15:17
nessitaok15:17
dobeynessita: and properly formatted for the copyright file (leading space and . for empty lines)15:18
nessitayes15:18
elopioalecu: yes, I have a clean windows 7 for testing.15:18
alecuelopio, awesome.15:18
briancurtinelopio, alecu: i don't currently have the right vmware setup for snapshots or whatever, so i've done testing on XP and 7, but u1 has been installed and uninstalled probably hundreds of times15:19
dobeyalright. i'm off to lunch. bbiab15:21
elopioalecu: new or existing user?15:21
alecuelopio, existing15:22
nessitadobey: hum, http://dep.debian.net/deps/dep5/ is unclear about the syntax of the full license text, as examples it just lists15:25
nessitaLicense: GPL-215:25
nessita  [LICENSE TEXT]15:25
nessitadobey: you sure I need to "escape" the full text? How shall I handle "headers" such as:15:26
* gatox lunch15:26
=== gatox is now known as gatox_lunch
nessita      6 License: AGPL-3 with OpenSSL exception15:26
nessita      715:26
nessita      8                     GNU AFFERO GENERAL PUBLIC LICENSE15:26
nessita      9                        Version 3, 19 November 200715:26
nessita     10  .15:26
nessita     11  Copyright (C) 2007 Free Software Foundation, Inc. <http://fsf.org/>15:26
* briancurtin getting coffee15:29
elopioalecu: I could pass the SSO login screen.15:31
elopioalecu: now I'm stuck at syncing the cloud to your computer15:31
elopiogetting information.15:31
elopiois that your problem?15:31
alecuelopio, no: I get an error in the sso login screen, and I can never reach the control panel.15:31
beunoso, is there a reason why I wouldn't be seeing any notifications from U1 anymore in precise?15:32
alecuelopio, can you send me the control panel logs?15:32
elopioalecu: sure.15:33
elopiohttp://paste.ubuntu.com/91467415:34
elopioalecu: ^^^15:34
elopiobriancurtin, alecu: problem here seems to be that windows firewall asks for access until I'm in the dashboard.15:40
elopioI can click next on that step of the wizard, get to the dashboard, grant access, and thne my folders are shown.15:40
alecuelopio, right.15:40
elopioalecu: want a bug?15:41
alecuelopio, sure. And thanks for finding this.15:41
alecuelopio, I'll take a look at the SSL issue, to see if I can reproduce it somewhere else.15:42
elopioalecu: np. And to try to reproduce your bug, are you using your real user, or a test user created more recently?15:42
alecuelopio, I'm using a newish user; I usually do all the testing in vms with that user.15:43
elopioalecu: ok. I used my real one. Let me try again.15:43
* alecu brbs15:44
nessitadobey: when you can, would you please review? http://pastebin.ubuntu.com/914691/15:49
elopioalecu, briancurtin, bug #97349815:50
ubot5Launchpad bug 973498 in Ubuntu One Control Panel "windows firewall grants access until the dashboard is reached" [Undecided,New] https://launchpad.net/bugs/97349815:50
briancurtinelopio, alecu: i wonder if we can poke windows explicitly to add our firewall exception. typically it shows that dialog upon your app first trying to get through, which seems too late?15:53
elopiobriancurtin: I'm not sure how it worked before, because if I remember correctly, the firewall notice was always at the dashboard. For a long time I've just used test users with no cloud folders.15:55
elopioit's possible this is not a new bug.15:56
urbanapeHey, all. Sorry15:56
elopioalecu, I could pass the login with a test user too.16:17
=== gatox_lunch is now known as gatox
briancurtinelopio: using the windows firewall API i think we can programmatically add u1 to the exception list, so this could be done as a part of the installation steps16:35
briancurtinlooking into it now16:35
dobeynessita: looks ok to me16:35
nessitadobey: is it correct that the ssl exception is right after the agpl full text, without any specific "separator"?16:45
elopiobriancurtin: sounds great. +1.16:46
dobeynessita: seems correct to me. i don't see any better way to put it in16:47
nessitadobey: ack, uploading that then16:47
elopiobriancurtin: there's bug #90938916:48
ubot5Launchpad bug 909389 in Ubuntu One Windows Installer "Add firewall exceptions as part of the installation process" [Low,Triaged] https://launchpad.net/bugs/90938916:48
briancurtinelopio: ha, that's solution i was just about to implement :)16:48
elopiobriancurtin: ralsina will give you a sticker with a smiley face.16:49
dobeygrrrrrr16:56
dobeydebuild is making me unhappy16:56
nessitalunchtime on this end!16:56
nessitabrb16:56
nessitadobey: FYI, if you do the next upload of storage protocol, micah just mentioned that:17:22
nessita"For the next upload, can you please make the Maintainer: Ubuntu     Developers <ubuntu-devel-discuss@lists.ubuntu.com>17:22
nessita    Using the MOTU list as maintainer is deprecated."17:22
dobeyok17:23
=== bac`` is now known as bac
nessitaalecu: ping. Does this http://pastebin.ubuntu.com/914876/ mean that the QNetworkAccessCache is modifying the "real" system?17:49
alecunessita, do you mean "it modifies the system proxy settings"?17:59
alecunessita, the answer to this is no.17:59
nessitaalecu: ack. Any idea why those are appearing while running the suite?17:59
duanedesignnessita: was helping a user who is having trouble and they sent me tis log. It looks like they are using an old version? Isnt that the version number for the beta? https://pastebin.canonical.com/63815/17:59
alecunessita, QNetworkAccessCache holds a cache of web and proxy credentials. This cache is "global" to the application, so it may not be reset properly between tests.18:00
alecunessita, I can take a look to see why they are being printed.18:00
nessitaduanedesign: this definitely looks odd: "2012-04-03 16:11:48,755 - ubuntuone.SyncDaemon.Main - INFO - Starting Ubuntu One client version 0.1"18:00
nessitaalecu: if it's less than 15 minutes, please go ahead, if not, no need as long as we are not modifying the real system settings18:01
alecunessita, qt is printing that as a qWarning, and it seems to go to stderr.18:02
nessitaduanedesign: does the user know which version is he running?18:04
alecunessita, so, whenever the credentials are updated that warning is printed. I think it's harmless, but I don't see a quick way to make it disappear18:04
nessitaalecu: ack18:04
nessitacan I have a trivial review for https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-nigthlies-test/+merge/100850 ?18:05
gotwighey18:05
gotwigI got a problem with ubuntu one18:05
duanedesignnessita: we gave him this link http://one.ubuntu.com/windows/ubuntuone-2.0.3-windows-installer.exe18:06
gotwigI am never able to sync my facebook contacts with u118:06
nessitaduanedesign: let me ask briancurtin18:07
briancurtinnessita: i'm reading up on the conversation now18:07
nessitabriancurtin: ping. COUld you please confirm if using the latest published installer for U1, the syncdaemon log will write something like https://pastebin.canonical.com/63815/ to the log file? what puzzles me is "Starting Ubuntu One client version 0.1"18:08
gotwigdo you have same problems?18:08
alecunessita, the name of your branch has a typo!18:08
nessitabriancurtin: ok then :-)18:08
nessitaalecu: it does, sorry18:08
alecunessita, no prob :-)18:08
briancurtinnessita: i'm in my VM right now, i'll check out what version that installer prints18:08
nessitabriancurtin: thanks a lot, would you please let duanedesign know about that?18:08
briancurtinwill do18:09
alecunessita, there's a small issue with "build_string_for_pixels"18:09
nessitaalecu: shoot18:09
alecunessita, when rendering a string of many characters, the individual glyph size might be different than when rendering each character by itself.18:10
gotwighas  anyone here even synced his contacts with facebook?18:11
alecunessita, so in this case there might be some pixels off18:11
nessitabeuno: would you have any spare time to answer to gotwig?18:11
nessitaalecu: even if I'm building the string with the same character?18:12
nessitaalecu: is not like I'm mixing chars, I'm using a single one18:12
alecunessita, due to font hinting and antialiasing, yes, it might be different.18:13
alecunessita, I'm not saying it will affect this case, I'm just pointing out that it might.18:14
nessitaalecu: nice sharing since I did not know that18:14
gotwignessita: thanks :P18:15
nessitagotwig: :-) (I would answer myself but I honestly do not know)18:15
alecunessita, so, whenever trying to find out the size in pixels of a string, you can only do it by asking the font engine to render it, and retrieving the size. You cannot find the size by asking each individual glyph (the letters in this case)18:16
nessitaalecu: shall we mumble?18:18
alecunessita, so, in this case, the right thing would be to start adding letters to the label, having the widget "realize" and checking if the size is in the range you want...18:18
alecunessita, but anyway, it will probably not affect this :-)18:18
alecunessita, sure, let's mumble.18:18
gatoxalecu, wait... maybe i'm understanding wrong.... but there is another way to do that18:18
nessitaalecu: right (I was gonna ask if the branch worths the change)18:18
alecugatox, tell me!18:19
gatoxalecu, if you are talking about Qt..... and you want to know the size in pixels of a string..... you can use QFontMetrics18:19
alecugatox, right.18:19
alecugatox, it uses the same underlying principle.18:19
gatoxalecu, yes.... basically..... but don't need to added to the label first, if you want to determine what to do before showing the text18:20
alecugatox, but using a label will consider the borders or spacing that might be around it.18:20
=== beuno is now known as beuno-lunch
gatoxsorry if i'm missing something... didn't read the whoole thing18:20
alecunessita, I'm rebooting the laptop, and will start mumble.18:21
nessitaalecu: ack!18:21
alecugatox, nessita: so, what shall we do about this branch?18:21
alecugatox, do you want to propose a fix using QFontMetrics?18:21
gatoxalecu, yes...... but you could ask that info to the label widget.... and do the numbers before showing the string..... from my point of view18:21
gatoxalecu,  i don't actually know what are you talking about jejjee just read about calculating the size of a string blabla.... and i jump in18:22
gatoxjejeej18:22
alecugatox, I was reviewing the branch to fix bug #97334518:22
ubot5Launchpad bug 973345 in Ubuntu Single Sign On Client trunk "Nightlies build failure: tets failed at ubuntu_sso.qt.tests.test_common.ElidedTextTestCase.test_text_elided_if_bigger_than_max_width" [High,In progress] https://launchpad.net/bugs/97334518:22
alecugatox, can you take a look?18:23
* gatox looking18:23
alecugatox, let's discuss it after I mumble with nessita.18:23
gatoxalecu, ok18:23
briancurtinanyone know why, on windows, "Syncing the cloud to your computer" never finishes getting information? i thought it was the syncdaemon firewall exception neededing to be entered, but it doesn't appear to be it18:36
dobeybugger, here comes the storm18:57
dobeyat least it's 81F outside18:57
alecugatox, back18:58
gatoxalecu, i was looking at the branch, and it seems fine by me..... which seems to be the problem?18:58
nessitabrb18:58
alecugatox, "to find out the size in pixels of a string, you can only do it by asking the font engine to render it, and retrieving the size. You cannot find the size by asking each individual glyph (the letters in this case)"19:00
dobeybah19:01
dobeyi knew i forgot something19:01
=== beuno-lunch is now known as beuno
gatoxalecu, yes, you should render the whole string...... unless you are using a mono font19:02
alecunessita, I think this might solve the issue with those qt tests: http://qt-project.org/doc/qt-4.8/qtglobal.html#qInstallMsgHandler19:03
alecugatox, touché19:03
alecugatox, hmmm....19:03
alecugatox, not. Some mono fonts have variable width for some cjk and graphic glyphs.19:04
gatoxalecu, mmmmm that doesn't sounds mono valid for me :19:04
gatox:P19:04
gatoxalecu, did you reach any conclusion on mumble?19:04
gatoxalecu, or you were talking about something else?19:04
alecugatox, we were mumbling about nessita rotating to the team on mondays, and about the tasks I should be filling in.19:05
alecu* the other team on next monday. :P19:05
gatoxalecu, ahhhh i thought you were discussing this19:05
gatox:P19:06
gatoxalecu, so...... let me understand....the solution that you are proposing..... if for the tests...... or for the code that will display the real widgeT?19:06
gatoxs/if/is19:07
alecugatox, re: mono fonts, try printing this in your mono term: print u'\u17d8Gatox'19:07
alecugatox, is only for the tests.19:08
alecugatox, the tests need a way to get a label that has a very specific size.19:08
gatoxalecu, wowwww!!!19:08
gatoxalecu, stop breaking my term!19:08
gatoxjeje19:08
alecugatox, that's only *one* character: ៘19:09
alecuand it's veeeery wide, even on mono fonts. But that's nitpicking.19:09
gatoxalecu, ahh if it is for the tests i agree with you..... i didn't like the idea to put the string in the label..... calculate if we exced the maximum size, and then cut off that...... i thought you were proposing that for the actual code19:09
alecugatox, right: I was proposing that for the tests: We create a label, and we start adding chars to it until it reaches the size we want to test.19:10
dobeyone *printable* char?19:10
gatoxalecu, just to keep in miind..... if we want to know the size, the widget should be visible19:10
alecudobey, I suppose there are some new postscript printers that have that glyph in their font, so yes :-)19:10
gatoxalecu, during the test19:10
duanedesignbriancurtin: did you notice what version number the latest published installer prints?19:11
dobeyalecu: trying to copy and paste that out of my terminal into something else isn't working so well. i presume there's some weird RTL and other control char in there, that aren't printed, but combine glyphs or something19:11
briancurtinduanedesign: sorry, got busy trying to fix something else. i'll take a break from that and look into the version number19:12
duanedesignno worries, thank you19:12
alecugatox, yes, visible. Or the qt equivalent of "realized" in gtk terms.19:13
alecugatox, "realized" is not exactly visible nor rendered; I think it's when the size has been calculated.19:13
alecudobey should know what "realized" exactly means in gtk.19:13
gatoxalecu, yes, there are some ways to do that.... but they are not always right..... and the actual way to do that..... is supported in Qt, PySide, but not PyQt19:13
alecudobey, it's just one character: "U+17D8 KHMER SIGN BEYYAL"19:14
alecugatox, then having the widget "visible" sounds fine.19:14
gatoxalecu, yap19:14
dobeyrealized isn't mapped19:18
dobeyand mapped isn't visible19:18
leonel2hello.  changed my photos folder on my android and now I don't have autoupload,    went to  settings/configure auto-upoad / upload directory   but I can't select the new phone directory19:45
dobeykarni_: ^^19:46
karni_leonel2: hi. The upload directory tells where you *upload* pictures to, not from.19:46
* karni_ looks at the market version19:47
leonel2my  ubuntuone version is 1.0.5.319:47
karni_correct, that is the market version. I wanted to confirm one setting.19:48
briancurtinduanedesign: using the same installer you told that person to, i see "version 0.1" as well19:48
karni_leonel2: How did you change the folder on your android?19:48
briancurtinduanedesign: running ubuntuone-2.0.3-windows-installer.exe19:48
leonel2karni_: told the camera to store the pics on the external sd19:49
karni_leonel2: FYI, the following update will allow you to select any source folder with pictures to backup, whereever it is19:49
karni_leonel2: Ah, correct then. You'll have to wait few days for that to work, sorry.19:49
duanedesignbriancurtin: thank you sir19:50
leonel2karni_:  thanks so  i'll revert the pictures storage19:50
karni_leonel2: You can, however, try this: http://goo.gl/UDfxJ.qr (scan with QR code scanner)19:50
briancurtinelopio: i think i have the firewall thing figured out, at least on Windows 7. still working on XP19:50
karni_leonel2: If you do, please disable auto updates for UBuntu One Files for a few days19:50
=== karni_ is now known as karni
leonel2karni: is this the new ubuntuone for android ?  and just asking why do I need to disable autoupdates ?19:59
nessitagatox: how can I have a qcheckbox wrapping its legend?19:59
gatoxnessita, is not doing it automatically if it hasn't the needed width?20:00
karnileonel2: It's a build with fixed /mnt/sdcard-ext hardcoded - a fix that will land on the Market soon, in a better shape. Why disable - if you install this, Android Market will update the app version to the one on the Market, and you'll loose the fix again.20:00
nessitagatox: nopes, the windows grows20:00
karnileonel2: No worries, I'm the developer on the Ubuntu One team :) You are safe to install from this link.20:01
gatoxnessita, ah yes..... you will need to set a maximumsize for the checkbox20:01
nessitagatox: no, no settings maximun width20:01
nessitagatox: since, if the user maximizes the window, I want the label to "expand"20:01
karnileonel2: If it doesn't work for you, it means you have yet another mount point. In which case, you'll just have to wait few more days for when we relase the update.20:01
nessitagatox: fixed widths should be a no-no in most cases20:01
nessitagatox: I found this solution, does it make sense? http://stackoverflow.com/questions/1839194/qcheckbox-qradiobutton-line-wrap-qt4-6-0 (having a label to be the buddy of a checkbox)20:02
* gatox looking...20:02
gatoxnessita, what the person propose no..... but the suggestions below.... is the same things i'm thinking of..... i had the saame problem with the enhancedcheckbox......20:04
nessitagatox: you mean comment #2?20:04
gatoxnessita, you have to extend and reimplement the internal layout.... because the checkbox is going to try to expand to fit everything in one line..... it hasn't a wordwrap setting as far as i know20:05
gatoxnessita, the comment #2 is basically what i did with the enhancedcheckbox... with some variiations and for another reasons20:06
nessitagatox: what about the secons option, "Relayout the UI so that each radio button is put on the left of a  QLabel. Add the whole text to the QLabel and set the label as the radio  buttons' buddy. This is the least functional approach, because clicking  the label does not check the radio button."20:06
gatoxnessita, i personally would do that...... if you problem doesn't fit in the solutino in the comment #1......20:07
gatoxnessita, and the problem with the label is not a problem20:07
matiasbgatox: nessita's computer decided to restart, she will be back in a minute20:07
gatoxnessita, you can add an event filter..... or even extend and specialize the qlabel.... and add that functionality20:07
gatoxmatiasb, jeje ok thanks20:07
matiasbnp20:08
leonel2karni: thank you20:09
karnileonel2: Have you tried it?20:10
karnileonel2: This will work much better soon, stay tuned :) np!20:10
leonel2karni: testing it but I can't find the upload from option20:12
karnileonel2: It's hardcoded, it should work for you (if the mount point is the same).20:13
karnileonel2: Do you know if it's /mnt/sdcard-ext ? I believe that's what's hardcoded in the apk I provided.20:13
leonel2how do I check that20:14
karnileonel2: Turn on auto upload, and take a picture. If it stats uploading, it means it's working :)20:14
nessitahola!20:14
nessitamy computer is crazy20:15
nessitaanyways.... gatox, what did I miss?20:15
gatoxnessita, i'll copy my messages20:15
nessitagatox: thanks20:15
gatoxnessita,20:15
gatox<gatox> nessita, i personally would do that...... if you problem doesn't fit in the solutino in the comment #1......20:15
gatox<gatox> nessita, and the problem with the label is not a problem20:15
gatox<gatox> nessita, you can add an event filter..... or even extend and specialize the qlabel.... and add that functionality20:15
leonel2karni: didn't work   autoupload is on and wifi only and connected to wifi   took a picture and didn't uploaded the picture20:16
nessitagatox: what do you mean with "and the problem with the label is not a problem"?20:16
karnileonel2: What phome make/model do you have?20:16
gatoxnessita, what i said in the next line: you can add an event filter..... or even extend and specialize the qlabel.... and add that functionality20:16
leonel2karni: lg 2x P99020:16
nessitagatox: how come an eventfilter can help me in this case? (excuse my ignorance)20:17
gatoxnessita, so...... that approach it doesn't have any limitation...... only is kind of annoying having to implement that..... but is not difficult either20:17
nessitagatox: I've heard ralsina once said that event filters are not a good option... but perhaps I misunderstood?20:17
karnileonel2: oh my, yet another mount point "/mnt/sdcard/_ExternalSD" -- sorry, this will not work for you. It will, however, with the next update.20:17
gatoxnessita, you can from the class that extends the qcheckbox and includes the qlabel.... install an event filter to the qlabel.... so every event that happens in the qlabel, will be send to the function that you define first and you can process it there........ is not a good option because it has its performance issues obviusly..... but if it is for something small.... and that it doesn't affect the whole application, i think you can use it..20:18
gatox.....20:18
dobeynessita: https://code.launchpad.net/~nataliabidart/ubuntuone-windows-installer/setup-is-needed/+merge/100605 needs a commit message btw20:19
leonel2karni: thank you I can wait  keep the GREAT and GOOD work20:19
gatoxnessita, or the other solution will be to extend the qlabel.... and reimplements the events that you need20:19
gatoxnessita, do you want me to do that??20:19
gatoxnessita, i've already done that a lot of times20:19
nessitadobey: thanks (though I would swear I added it)20:19
nessitaalecu: so, any verdict for the review of https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/fix-nigthlies-test/+merge/100850 ?20:20
karnileonel2: Thank you :)20:20
dobeynessita: well i guess you haven't made the tarball yet either :)20:20
nessitadobey: nopes :-)20:20
nessitadobey: is ubuntu-release *the* channel to ask if a bug requires an exception, no?20:20
nessitaI got no answer, and wanted to confirm20:21
dobeyyeah but people could be busy/idle at the moment. it is irc after all, and time zones can conflict :)20:21
alecunessita, if the branch fixes the tests, let's go ahead. I wanted to do the "unbreakable" solution, but we can do it later if the tests keep failing.20:21
nessitaalecu: I'm not sure of it fixes since it passes locally (and always did). I think ti will fix it tough.20:22
nessitathough20:22
gatoxnessita, do you want me to implement that?? i've already done that a lot of times (i don't know if you read those messages)20:22
alecunessita, ok: I'll run the tests and approve.20:22
nessitadobey: sure, wanted to confirm my head is (still) thinking straight20:22
nessitagatox: as long as you don't set a maximum with and not use an event filter, you re welcomed to20:23
nessitagatox: can you fix that without using those things? ^20:23
gatoxyes..... i'll need to extend the checkbox as we talk..... but nothing more than that.....20:23
nessitagatox: but wait20:23
gatoxnessita, can you please assign me the bug so i know what are we talking about specifically :P20:23
nessitagatox: what about just adding a label next to the checkbox in the designer, without extending anything?20:23
nessitagatox: will that fix the bug?20:24
nessitagatox: bug report is https://bugs.launchpad.net/ubuntuone-control-panel/+bug/97370220:24
ubot5Ubuntu bug 973702 in Ubuntu One Control Panel trunk "The "Settings" panel should have its items set to wrap" [High,In progress]20:24
gatoxnessita, yes...... but it won't react as a whole thing with the checkbox if you want that20:24
nessitagatox: even if the label is set to be the "buddy", like the post says?20:25
gatoxi mean...... if you want that the focus in the checkbox..... shows the orange border including the label... that won't happend doing it as two different widgets20:25
nessitagatox: same question as before :-)20:25
gatoxnessita, where did you read that?20:26
gatoxfound it20:26
nessitagatox: ack20:26
dobeywhy does the checkbox label get an orange/pink-ish background, with the orange border around it as well?20:26
nessitadobey: border is "ours", background is from the theme20:27
gatoxnessita, when it says buddy...... i think that just means to be next to the check...... after that it says: "This is the least functional approach, because clicking the label does not check the radio button. Also, alignment is not very good."20:27
dobeynessita: it looks ugly, shouldn't we turn the background off somehow (perhaps set it to white/transparent), if we're going to add the border?20:27
dobeyalso, why do we add those borders? they are weird20:28
gatoxnessita, ahhhhhh sorry..... i know what it means now20:28
gatoxthe buddy thing20:28
nessitadobey: that background can not be styled from a qss stylesheet. To remove it from the buttons we had to do  ugly hacks to increase padding in a way that the padding will "dissapear", but the same hack can not be applied to the checkboxes20:29
nessitadobey: ralsina analyzed it, and decided with the design team to leave it like that20:29
nessitadobey: borders are design decision to highlight current focused item20:29
nessitagatox: so, can the buddy thing work?20:29
nessitagatox: anyways, I think we should wait for ralsina next week and have him making a Qt decision on that one20:30
dobey:-/20:30
nessitadobey: could you please review https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/smaller-loading/+merge/100863 ?20:30
gatoxnessita, i ccan try that if you want...... but i'm pretty sure..... that the focus style in the stylesheet is going to highlight only the check.... not the label..... and if we change the focuspolicy of the label (because by default labels doesn't have focus)..... it is going to be highlighted as a different widgets.... but that is just my opinion.... i can test this really fast20:30
dobeynessita: i already did 7 minutes ago :)20:30
nessitadobey: oh thanks!20:31
nessitagatox: let's put this on hold, and you can talk it over with ralsina on Monday, ok?20:31
nessitaI thought it was much much easier20:31
nessitalike clicking a setting in the designer20:31
gatoxnessita, ok........ also..... i'm eod now..... and i'm really need to go :P20:31
nessitagatox: byeeeeee20:31
gatoxnessita, byeeeeeeee, see you tomorrow20:31
nessitadobey: is jbicha a member of the release team?20:32
dobeygatox: doctor appointment?20:32
nessitalol20:32
dobeynessita: https://launchpad.net/~ubuntu-release/+members says no :)20:33
dobeynessita: but he is the docs guy, so just wanted to point out that docs/translations was in the clear, as far as that change goes20:33
nessitadobey: yeah, I knew that part, was curious about the first thing though20:34
dobeynessita: right. i didn't ping him for you. i pinged him to make it clear for any release team person who responds in channel :)20:34
alecunessita, approved.20:34
nessitaalecu: thanks!20:34
nessitadobey: thanks? (???)20:34
dobeynessita: so instead of having to wait for the release team person to go ping people about that, the information is already visible :)20:35
nessitadobey: ack20:35
dobeynessita: also, if you're asking, to hold off landing it in trunk, there's no reason. we will cherry pick now, so it can land in trunk regardless. but i think it's fine to stick in stable-3-0 next week as well20:36
nessitadobey: I wasn't holding off landing, it landed already, actually :-)20:36
dobeyok20:38
dobeynessita: comment in the bug description that it doesn't affect translations or docs, subscribe the release team, and if they complain we won't put it in 3.0 :)20:39
nessitawill do20:40
nessitadobey: heh, jeremy did it first :-)20:41
dobeyheh20:41
dobeyyay community! :)20:41
nessitaok, my pilates class was moved one hour early, so I will need to run to it in 10 minutes. Anyone needs something from me?20:43
dobeymate and medialunas20:45
nessitadobey: how did you know?!?!?!?20:47
dobeyheh, bring me some :)20:47
briancurtini need windows XP to not suck20:47
dobeybriancurtin: want to work on the os/2 warp 4 port?20:48
briancurtindobey: after we get done with OPENSTEP20:48
dobeywhoot20:51
nessitaok, I'm gone20:52
nessitabye all! see ya tomorrow20:52
* nessita works tomorrow20:52
dobeylater all21:30
elopiobriancurtin: cool. If you have it for today, I'll give it a try. Otherwise, rmcbride will kindly take my testing tasks for the rest of the week.21:45
briancurtinelopio: i'm doing one last test on Win7 and it might be ready. i'll let you know shortly, otherwise it wont be there until tomorrow and i'll email it to rick then21:48
elopioack.21:55
=== yofel_ is now known as yofel

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