/srv/irclogs.ubuntu.com/2010/08/25/#ayatana.txt

sensegood morning10:14
klattimerMacSlow: you might appreciate https://bugs.launchpad.net/notify-osd/+bug/35384311:04
klattimer:)11:04
ubot5Launchpad bug 353843 in Notify OSD "Fallback alert has leftmost button as default (affected: 0, heat: 2)" [Medium,Fix committed]11:04
klattimersimple fix11:04
MacSlowklattimer, I don't use "misbehaving" apps that use actions (or other non-recommended) notification features :)11:05
MacSlowklattimer, hm... where's the diff?11:09
MacSlowklattimer, ah... you've not merge-requested it yet... that's why.11:09
=== Cimi_ is now known as Cimi
klattimerno, I will in a short while11:14
klattimerkenvandine: can you test this updated patch for gnome-power-manager https://bugs.launchpad.net/ayatana-ubuntu/+bug/61981611:14
ubot5Launchpad bug 619816 in Ayatana Ubuntu "Battery status line too long (affected: 1, heat: 8)" [Medium,New]11:14
klattimerMacSlow: if everything went to plan it should show up here in a minute https://code.launchpad.net/~karl-qdh/notify-osd/dialog-no-default-button/+merge/3362211:15
klattimeroh... hangon, I might have not done something11:15
klattimer...shit :/11:15
klattimerI didn't commit :/11:16
klattimerMacSlow: https://code.launchpad.net/~karl-qdh/notify-osd/dialog-no-default-button/+merge/3362311:18
klattimerthat's better11:18
MacSlowklattimer, next time please link branches fixing bugs to the bug they actually are meant to fix.11:24
MacSlowklattimer, I also use the habit to name branches accordingly in addition to taht... e.g. notify-osd.fix-35384311:25
MacSlowklattimer, makes life a lot easier11:25
MacSlowklattimer, reviewed, approved and merged11:28
klattimeroh awesome :)11:29
klattimersorry about missing the bug number from the title11:30
=== MacSlow is now known as MacSlow|lunch
davidbarthCimi: back13:21
=== MacSlow|lunch is now known as MacSlow
kenvandineklattimer, sure13:40
=== oubiwann is now known as oubiwann-away
=== oubiwann-away is now known as oubiwann
LaserJockis didrocks on vacation or something?13:49
kenvandineklattimer, weird... i am sure this isn't a bug in your patch from yesterdya13:53
kenvandinebut... if i click on gpm in the indicator, the menu i get says "drop all elevated privileges13:54
kenvandine"13:54
kenvandinesounds like an indicator-application bug13:54
davidbarthkenvandine: wassup?13:57
kenvandinedavidbarth, hey13:58
kenvandinedavidbarth, when i click on the power manager indicator, the menu i see says "drop all elevated privileges"13:58
kenvandineseems to be getting the menu for something else, like that policykit thing that shows up to tell you you entered admin priv13:59
davidbarthkenvandine: didn't see that13:59
kenvandinei didn't either 20m ago14:00
davidbarthklattimer: i've manually installed your package cause apt was confused with the identical version number14:00
kenvandinesomehow it got confused14:00
davidbarthstill, i haven't see that while testing the indicator14:00
davidbarthi'm getting a polkit dialog to allow the backlight helper ro run14:00
davidbarthbut that was already the case with the stock maverick package14:01
davidbarthklattimer: but i guess i should test the one with the updated patch, that has strings closer to mpt's specs14:02
davidbarthklattimer: can you also add the icon for the battery to the menuitem as well?14:03
mptugh, we're still not using symbolic icons for the battery menu?14:05
mptghostly white icons are showing up in the "Power Statistics" window14:05
davidbarthmpt: symbolic icons was not backported to gtk2 this cycle14:05
mptdavidbarth, I thought we had a temporary "-panel" namespace we were using instead?14:06
mptSo the icon in the panel uses "somethingsomething-panel", while the icon in the Power Statistics window uses "somethingsomething"14:07
kenvandinedavidbarth, this isn't with klattimer patch... it was with stock maverick14:12
davidbarthmpt: that we do14:20
davidbarthmpt: ie, the -panel namespace is used for indicators icons and their menus as well14:20
klattimerdavidbarth: I think picking icons for the menu item may be a little difficult14:22
mptdavidbarth, ah, I guess gnome-power-manager hasn't been patched to use them, and therefore the panel-style icons have been applied to the non-"-panel" namespace so that g-p-m will notice them.14:22
klattimeras we need to know what kind of item it is, phone, mouse, keyboard, laptop14:22
davidbarthmpt: yeah14:24
davidbarthmpt: actually that's worse than that14:24
davidbarthno, not worse14:25
davidbarththe indicator and the menu icons first look for a panel version automatically, and we had provided symlinks in lucid to remap them to the proprper icons14:25
klattimerdavidbarth: yeah, the namespace happens in the icon lookup14:26
davidbarthso in maverick, either it's the icon names that have changed (and lack the symlinks behind the scene), or the symlinks have been dropped for that part of the theme14:26
klattimersounds like a likely cause of the icons being wrong14:27
kenvandineklattimer, better i think (gpm) that is... but now when discharging it just says "Laptop battery"14:28
kenvandineno time or %14:28
klattimerkenvandine: that should only be the case if you have more than 12 hours left?14:28
kenvandinewhen i first unplug i briefly see the estimating string14:28
klattimeris that possible on laptop battery14:29
* kenvandine finds that unlikely :)14:29
kenvandinepowertop says 6.3 hours14:29
kenvandinenow 4.5 hours14:29
kenvandinewhen charging it looks right14:30
klattimerhmm14:30
klattimerweird14:30
klattimerI'm using 12*60 as my number here which is 12 hours in minutes14:31
klattimerso not sure what's going on14:31
klattimerkenvandine: I've updated the patch slightly14:36
kenvandineok14:36
klattimerif you want to try that, it's attached to the bug report14:36
kenvandinewill do14:36
klattimerI can't see that it'll improve things any more than was there last, but it is slightly neater and easer to read14:36
kenvandineklattimer, same14:39
kenvandinelet me add a couple printfs int here14:39
kenvandinein there14:39
Cimidavidbarth: news from the 22px patch?14:42
kenvandineklattimer, time_to_empty_round 690014:42
kenvandineGPM_UP_TEXT_MIN_TIME 12014:42
kenvandineklattimer, that is at line 250 in src/gpm-upower.c14:43
kenvandineklattimer, powertop is reporting 5.9h left14:44
kenvandineklattimer, 6900 is way more than 12*6014:45
davidbarthCimi: right, could you refresh cody's patches and make that use a dedicated style property?14:49
kenvandineklattimer, look at         time_to_empty_round = egg_precision_round_down (time_to_empty, GPM_UP_TIME_PRECISION);14:49
davidbarthCimi: then send some screenshots for otto to evaluate the result14:49
kenvandinei think that GPM_UP_TIME_PRECISION isn't what you are expecting14:49
kenvandineso time_to_empty_round isn't really number of minutes left... which seems weird14:50
kenvandinebut GPM_UP_TIME_PRECISION is 5*6014:50
kenvandineso 5s i guess14:50
klattimeroh14:51
Cimidavidbarth: will do tomorrow14:52
CimiI'm about starting writing a blogpost for the new themes14:52
kenvandineklattimer, so time_to_empty_round is growing14:52
kenvandineit is now 1920014:52
Cimiiainfarrell: pink14:52
kenvandineand i am discharging still14:52
Cimi*ping :P14:52
kenvandinehehe14:52
klattimerhmm14:52
klattimeroh dear14:52
klattimer:/14:52
kenvandineklattimer, now it went down to 870014:52
kenvandinedoesn't seem like a very stable number :)14:53
kenvandineand now 18300... geez14:53
Cimior our ivanka magician too14:53
kenvandineklattimer, i guess look at gpm_get_timestring14:54
kenvandineto see how it converted time_to_empty_round to the string that is displayed14:54
iainfarrellcimi hi there14:54
iainfarrellwhat's up?14:54
Cimioh cool14:54
Cimisince otto is busy with unity, radiance and other tasks14:54
Cimiwe decided I could take care of writing a blog post with the updates to the latest themes14:55
Cimiand request for some feedback14:55
iainfarrellahh cool14:55
CimiI guess I need an account as a guest blogger or so14:55
iainfarrellthat's great14:55
iainfarrellyeah you do14:55
klattimerkenvandine: I've added a new patch14:56
klattimerI guess it's in seconds not minutes so added another *6014:56
klattimerthat should fix it14:56
klattimer5.08 hours remaining sound reasonable to you?14:56
Cimiiainfarrell: so? :P14:56
iainfarrellI'm finding out if I can get the account sorted14:57
kenvandineyeah14:57
iainfarrellCimi: if you write up the post14:57
iainfarrellI'll chase the accounr14:57
iainfarrell*account14:57
iainfarrellwhat's your LP ID?14:57
kenvandineklattimer, building now14:57
klattimercool14:57
klattimerthat should be it then14:57
klattimercan you can get that patch pushed out in a new package?14:57
kenvandineyeah14:58
JanCBorchardtmpt: thanks for the great comment on the Software Center sky color bug report, made my day ;)15:00
davidbarthCimi: btw, kenvandine mentionned you had the same issue as i have with the entry only displaying one character at a time?15:00
mpt:-)15:00
davidbarthCimi: do you think that can be a theme issue?15:00
Cimiit isn't15:01
kenvandineklattimer, working pretty well, but comparing the strings to the wiki page15:01
Cimibecause if you write a word and send it15:01
Cimiit sends only the last chat15:01
Cimiso it's not like having a transparent text :)15:02
kenvandineklattimer, the wiki page says "X H:MM left”15:02
klattimeryeah?15:02
kenvandinebut i get "X H and MM left"15:02
klattimerwhat are you getting?15:02
klattimeroh :/15:02
kenvandinebut i get "X H hours and MM minutes left"15:02
klattimershit, then there's a problem with the string output15:02
kenvandinebut the behavior is right :)15:02
klattimergrr15:02
kenvandineoh...15:02
klattimerok, I'll add a fix for that15:02
kenvandinecool15:02
kenvandinegetting there :)15:03
kenvandineCimi, any fixes for the theme for the text in that entry?15:03
kenvandineas in visibility... it is pretty hard to see atm15:03
Cimikenvandine: it's already fixed it locally15:05
Cimiwe will push a new version soon15:05
Cimimaybe tomorrow15:05
kenvandinewith ambiance-maverick-beta15:05
Cimilocally is my pc :P15:05
kenvandineok, cool15:05
klattimerkenvandine: added a new patch which should fix the strings15:11
kenvandineklattimer, ok, will test in a bit15:11
kenvandinethx15:11
* klattimer can't believe the battery applet's upstream code actually says in many cases "Laptop battery 2 hours 14 minutes until charged\nProvides 1 hour and 12 minutes battery runtime15:11
klattimer?!?!?!?!15:11
klattimerwho thought that was a good summary15:11
klattimer?15:11
klattimerfor a menu?15:12
kenvandinehehe15:12
kenvandineCimi, when you push the new theme changes, make sure you rename them to match the default names... so move ambiance-maverick-beta to Ambiance in the bzr tree15:20
kenvandineand drop the old stuff15:20
davidbarthklattimer: isn't that shared with a notification string instead?15:20
klattimerdavidbarth: grr, I hope not, lemme look15:26
tedgdoctormo, The Inkscape website is still in SVN.15:27
tedgdoctormo, They keep saying it is going to be migrated to a DB, so I didn't migrate it to Bazaar, but that's looking like a worse and worse idea.15:27
klattimerit's used in the tooltip15:27
klattimerbut I think that's OK right?15:27
klattimerbetter to be the same than different15:28
tedgdoctormo, I realize this should be in #inkscape, but apparently I don't have voice there :-/15:28
=== JanC_ is now known as JanC
kenvandineklattimer, very slight thing... the wiki says H:MM but it is now HH:MM15:34
kenvandineklattimer, also there is a brief point in the transition from charging -> discharging and discharging -> charging where it says "X charged" before it says "X (estimating)"15:36
kenvandinei assume that is because the value of time_to_empty_round is getting set to some crazy number which is over 12h15:36
klattimeroh, yeah, I added the wrong format string duh15:42
klattimerkenvandine: can you check the notifications to see if they have a sensible looking string15:42
kenvandinethey seem unchanged15:43
klattimercool15:43
kenvandinethe notification still includes the %15:43
klattimerright, well the brief change I don't think I can do anything about15:43
kenvandineyeah15:43
kenvandinei figured15:43
klattimerkenvandine: ok, so if we're not going to have HH:MM then should i have "X minutes" when there's less than an hour15:44
klattimermpt: same question to you ^^15:44
kenvandinei assume15:45
kenvandinelet mpt answer that though :)15:45
klattimeryeah15:45
klattimeronce we've got a definite from mpt i'll cook up one last patch and that'll probably be it15:47
mptklattimer, I'd rather just have H:MM consistently15:48
klattimerso 0:MM would be preferred for < 60 minutes15:48
ronocCimi: can you point me towards the branch you had for the rounded corners on the album art15:49
klattimerkenvandine: the patch is uploaded to the bug15:50
JanCBorchardtmpt, about the consistency thing at https://bugs.launchpad.net/ubuntu/+source/update-manager/+bug/61872315:55
ubot5Launchpad bug 618723 in One Hundred Paper Cuts "Rename Update Manager to Software Updater (affected: 3, heat: 20)" [Low,In progress]15:55
Cimironoc: https://code.launchpad.net/~cimi/indicator-sound/rounded-albumart15:56
ronocCimi: thx15:56
JanCBorchardtmpt: I disagree because most users rarely access the system settings for updating. They just get the pop-up without context-15:56
klattimerkenvandine: I accidentally didn't save the file before diffing :/ so I've uploaded another new patch which does the right thing15:57
kenvandineok15:57
mptJanCBorchardt, in that case I've specified that the title should be "Updates Available", so the name doesn't show up anyway.15:57
JanCBorchardtmpt, ok, cool15:59
kenvandineklattimer, much better :)16:07
kenvandineklattimer, uploaded16:13
kenvandinethx16:13
klattimertedg: is there a bug number for the group menu item bug?16:34
klattimerI'd like to mark something as a duplicate of it16:34
tedgklattimer, I'm not sure I know which bug you're talking about.16:40
davidbarthCimi, chaotic: btw, can you confirm that the notebook style is off the list now?16:46
chaotic?16:47
davidbarthit's still listed for tomorrow, but i don't think it sounds possible anymore: https://bugs.launchpad.net/ayatana-ubuntu/+bug/61169416:47
ubot5Launchpad bug 611694 in Ayatana Ubuntu "Implement new notebook style (affected: 1, heat: 11)" [Medium,New]16:47
davidbarthchaotic: an old work item that was on cimi's list, more tentative16:47
davidbarthdropping it from the list16:48
davidbarthCimi: same for argb, too bad but that's all too late16:49
chaoticdavidbarth: fine by me, we have made improvements anyway16:50
Cimidavidbarth: argb is useless without cody's patch16:52
ronocCimi, have you recoloured the button s in one of your branches16:53
ronoc?16:53
ronocrecoloured the background that is ?16:53
Cimironoc: ?17:00
Cimiwhat do you mean?17:00
ronocCimi, you worked on the colours last week yes of the buttons so as they can be changed with theme change17:01
ronocno ?17:01
ronocwhich branch is this ?17:01
Cimino17:03
ronocCimi, you did work on two branches last week though yeah ?17:03
Cimironoc:17:09
Cimiyes17:09
Cimibut one lacks of the custom gtk-widget name17:09
ronocCimi: can you post the branch url please ?17:09
Cimihttps://code.launchpad.net/~cimi/indicator-sound/use-gtk-colors17:10
ronocCimi: thx17:10
Cimisorry for being a little bit off today from the chat... I've a headache and I'm feeling really bad17:11
ronocCimi:  no worries, me too - running a fever topping up on lemsips at the mo :/17:11
ronocCimi: so do we need to come up with a name for the transport bar17:12
ronoc"ido-transport-bar"17:12
ronocalthough its not an ido17:12
ronocso "ayatana-transport-bar"17:12
ronocCimi: ^17:13
Cimimaybe17:17
Cimitransport bar is?17:17
ronocCimi: just name check everytime you answer, otherwise I might be watching my irc17:18
ronocCimi: transport bar is the play button, previous and next button on the currently grey/white background17:19
ronocCimi, sorry first msg was missing the operative *not17:19
Cimironoc: ok17:21
Cimironoc: chose the name you prefer17:21
Cimironoc: same thing for the scrubber please17:21
ronocCimi: "ayatana-transport-bar"17:22
ronocCimi: does this follow the same pattern as the other gtk widget names ?17:22
CimiI don't know17:23
CimiI mean17:23
Cimiusually for custon names yes17:23
ronocCimi: i.e. do you know what other gtk -widget names look like?17:23
ronocor does it make a difference ?17:23
Cimilet me see17:23
Ciminot, just for logic porpouses17:23
Cimimmm17:24
Cimiyeah could be ok17:24
ronocCimi: okay grand17:26
ronoclets use that then17:26
Cimifor scrubber?17:27
Cimiido-scrubber?17:27
ronocCimi: ido-scrub-bar17:30
ronocI reckon17:31
Cimiok17:33
ronocCimi back sorry17:56
Ciminw17:56
=== MacSlow is now known as MacSlow|break
vladatedg, ping18:32
tedgvlada, Howdy18:47
vladahi tedg18:47
vladatedg, I was sent here by #ubuntu-devel guys18:47
tedgUh, oh, never trust the ubuntu-devel guys ;)18:48
vladalooks like I have to complaint to you about this bug (which is mine, BTW)18:48
vladahttps://bugs.launchpad.net/ubuntu/+source/indicator-applet/+bug/62183818:48
ubot5Launchpad bug 621838 in indicator-applet (Ubuntu) "Keyboard layout part of indicator applet disappears (affected: 1, heat: 6)" [Undecided,New]18:48
vladatedg, I trust inkscape guys ;)18:48
tedgvlada, Yeah, I think that one is fixed.  It just won't be released until tomorrow -- we release on Thursdays.18:50
tedgvlada, It was using the wrong index to update the icons/labels on new items.18:50
vladawow! great!18:50
vladaand, did you change keyboard icon (graphic)? Current one is too fuzzy.18:51
=== MacSlow|break is now known as MacSlow
tedgvlada, I'm not in charge there, that'd be the design guys who do the icons.  chaotic might have a comment there.18:52
vladatedg, do you have hired professional for icon design or you are using community provided icons?18:54
tedgvlada, both, we have professionals charged with it but they take community contributions as well.18:55
vladacanonnical is strangly organized indeed :) At least when looked at from outside ;)18:56
vladatedg, BTW and OT: will inkscape 0.48 manage to get into 10.10?18:57
tedgvlada, I believe so, but I haven't been following that.  Scisalac and kees I believe were looking at it.18:58
ronocCimi: rounded rectangles are in -> lp:~cjcurran/indicator-sound/fix-rounded-corners-on-album-art20:31
MDC1from #nautilus: (20.04.10) cosimoc: so, this is how nautilus master looks like now -> http://people.gnome.org/~cosimoc/new-sidebar.png21:29
MDC1:D21:29
MDC1vish: ^21:29
vishMDC1: sidebar!!!!!!!21:33
MDC1oh yes :)21:33
vishMDC1: hehe , for a few sec i was wondering what i was looking for :D21:33
hyperairooh nice.21:33
MDC1aday: in case you've missed it: http://people.gnome.org/~cosimoc/new-sidebar.png21:34
adayMDC1: just saw the commit21:34
MDC1night now21:37
Cimironoc: cool21:47
CaerHi everyone22:45
CaerI would like to report a subset of this bug https://bugs.edge.launchpad.net/gnome-panel/+bug/4400222:45
ubot5Launchpad bug 44002 in gnome-panel (Ubuntu) "Trivial actions are not appearing immediately (affected: 3, heat: 11)" [Low,Triaged]22:45
CaerIt bothers me that, because of the loading of icons in gnome's main menu, sub-menus are resized the first time they are displayed22:47
Caerso there is a kind of flicker when gnome finally decides which width the icon column should be ; this could be avoided because all icons are the same size in the default config22:48
Caerdoes that qualify as a papercut? And should I create a new bug referencing this one? (I will be reading your anwsers tomorrow)22:49

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