[00:00] But I found how to create a Dbusmenu on https://wiki.ubuntu.com/Unity/LauncherAPI#Python_Example Unfortunately they attach it to a unity quicklist. I assume there is a similar way to add the Dbusmenu to the appmenu? [00:00] hi Freddi [00:00] Freddi: ted is probably the person who could help most [00:00] Freddi: unfortunately he is done for the day [00:01] Ok, so I can reach him tomorrow? [00:01] should be able to, he works US mountain time UTC-7 (I think) [00:01] thanks! [00:02] np [00:44] jaytaoko, ping. Have a question about StaticText and StaticCairoText === cyphermox_ is now known as cyphermox [01:22] thumper, hey, if you have some time could you look at this branch that should be ready for merge now! [01:22] https://code.launchpad.net/~brandontschaefer/unity/fix-711199/+merge/89192 [01:39] bschaefer: ping [01:43] hey jaytaoko [01:44] bschaefer: reviewed [01:44] jaytaoko, so when I tried using StaticText it was not working correctly; and also giving some weird behavior [01:45] thumper, thanks, is there anything you see that I should change with it? [01:47] thumper, nevermind see it now! [01:48] :) [01:48] thumper, what do you mean by normal result view? [01:49] the grid results [01:49] I couldn't see anywhere where you maked the other view inactive [01:49] what happens there? [01:50] thumper, hmm, do you mean with the use of active_lens_view? [01:50] maybe... [01:51] thumper, that was a global I was using, it get set else where when the lens gets chagned [01:51] bschaefer: so if it isn't there, nothing shows? [01:53] thumper, each view has its own lens, and when it isn't the active lens I don't wouldn't think anything would get active in it [01:54] thumper, each lens remembers its state though, so when I type "abc" in the music lens then leave and come back it remembers that state [01:54] * thumper nods [01:54] thumper, so I don't think I have to set anything as inactive since that should have already been the case [01:54] and it works? [01:54] :) [01:54] thumper, as I only changed what happens when the on_search_finished signal gets emited() [01:55] and yes :) [01:55] I should make a small clip [01:55] ok, just the style changes then :) [01:55] thumper, also will change to use sstream [01:56] thumper, there are a few screen shots on the bug report ill make a small clip to put of there to demo it [01:56] thumper, also fixed a bug in the music lens with the help of mhr3 which was causing some problems but that is gone now [01:57] awesome [01:58] * thumper goes on the school run [01:58] thumper, have fun :) [02:10] jaytaoko, hey, I have also been looking at the IMContext, and the ibus is not connecting with the ibus-daemon when i gets created [02:10] which is weird, but that is something im looking at now [02:17] bschaefer: sorry, I have to go but I will be back later [02:19] jaytaoko, no worries, I need to dig some more :) [02:34] Could a unity dev with review powers please bless lp:~themuso/unity/fix-launcher-accessible-focus? njpatel approved it, but it hasn't been marked as approved, and I'd like to get it into trunk... It restores accessibility to the launcher. [03:12] I just tried to install unity 5.2 and it doesn't appear to have upgraded [03:13] how do I check to see which version i am running? [03:20] solclaimjak: apt-cache policyh unity if you are comfortable with the terminal. [03:20] Otherwise there is likely a way to check in software centre, but I'm not sure about how that is done, since I tend not to use that myself. [03:21] TheMuso, yeah I can approve it, ill trust njpatel judgment ;) [03:34] Well, it says I am using 5.2.0 but it doesn't appear to have the changes that the sites have said it has. I will keep looking... [03:36] i think i am using unity 2d.... [03:36] thanks for the help :) [03:48] bschaefer: Thanks very much. [04:38] bschaefer: ping [04:46] jaytaoko, hey! [04:47] bschaefer: hey [04:48] bschaefer: so you had a question about StaticText? [04:48] jaytaoko, o yes, so when I changed my StaticCairoText to use StaticText it would just not work anymore! [04:49] jaytaoko, so I wanted to ask you if there would be a reason for the StaticText to not like the use of SetText() [04:50] jaytaoko, branch for reference; https://code.launchpad.net/~brandontschaefer/unity/fix-711199/+merge/89192 [04:51] bschaefer: let me check [04:51] jaytaoko, thank you! [04:52] bschaefer: does the code crash, or it just doesn't work? [04:52] jaytaoko, it works fine, no warning either. What is weird is the text will only show up on the last lens [04:53] since there are 4, so when I set it use StaticText the other lenses dont show a message except for 1 [04:53] and when that message gets set to "" like half an S is still rendered. It is slightly wired. [04:54] jaytaoko, let me take some screenshots! [04:55] bschaefer: ok, I am downloading the branch. I will compile it shortly. [04:55] jaytaoko, sweet, I was going to ask someone to test that soon [04:56] jaytaoko, was trying to get recordmydesktop to work but nothing would render under the video! (besides my wallpaper...) [05:01] bschaefer: no worry, I am compiling [05:02] jaytaoko, cool, also if you have some time we can talk about IMContext and why ibus_bus_is_connected is returning FALSE! [05:03] jaytaoko, it might be pretty late for you, tomorrow all my focus is going into that [05:03] jaytaoko, I lost track of time haha [05:05] bschaefer: sure, let me get the IMContext branch [05:10] jaytaoko, also found this: http://src.chromium.org/svn/trunk/src/chrome/browser/chromeos/input_method/ [05:11] jaytaoko, you might have looked at it, but it seems to be doing something very similar to what we are trying to do [05:12] bschaefer: yeah. And it seems the only way to get into IM is to look at what others have done before. Otherwise it is hard to start it from scratch [05:13] bschaefer: ok, I am running your branch [05:13] jaytaoko, yeah i know, the api is very bad... [05:13] bschaefer: i see the text message when there is no result found [05:13] jaytaoko, alright, so with the StaticCairoText everything works correctly doesn't it? [05:13] jaytaoko, right now it is using the StaticCairoText [05:14] bschaefer: ok, How to I use StaticText [05:14] jaytaoko, you just have to change 2 lines of code [05:14] bschaefer: tell me the file name where I do the change [05:14] jaytaoko, in LensView.h and LensView.cpp [05:14] LensView.h change StaticCairoText to StaticText [05:15] and in LensView.cpp change new StaticCairoText to new StaticText [05:15] bschaefer: I see it [05:16] jaytaoko, wait it is working for you? [05:16] bschaefer: no still compiling [05:16] jaytaoko, o, ops thought you ment you see it haha [05:16] bschaefer: I was talking about the StaticCairoText [05:16] yeaaah [05:17] jaytaoko, but good the StaticCairoText is working for you! [05:18] bschaefer: compiling with StaticText... [05:22] jaytaoko, what is wired for me is the fact that the StaticText works with 1 out of the 4 lenses [05:23] bschaefer: it seems to be working fine here with StaticText [05:23] bschaefer: let me check again... [05:24] bschaefer: what I do is I type some bogus text in the search entry, and then it displays the "No result" message in the center [05:24] jaytaoko, for all the lenses? [05:24] bschaefer: yes, for all lenses [05:25] jaytaoko, hmm weird. Im going to recompiling nux as I pulled from trunk and it said some fonts were changed [05:25] maybe thats my problem [05:25] bschaefer: the font stuff in nux as nothing to do with Unity... it is something else [05:26] bschaefer: what is your revision of Nux? [05:26] jaytaoko, hmm, that is weird; let me take a screen shot of it half ass working on the music lens [05:26] jaytaoko, well I just pulled from trunk and am comping it right now so im not sure. [05:28] jaytaoko, in the meantime does everything look good on the no-results message to you? [05:29] bschaefer: looks fine here [05:30] jaytaoko, http://imgur.com/Uan8J [05:30] so that is a picture of what it looks like where there are results [05:30] and you can see the sliver of the 'S' in the middle [05:32] jaytaoko, and here is one of the home lens http://imgur.com/UnYOW [05:32] jaytaoko, with StaticText [05:32] bschaefer: Are you on precise? [05:32] yeah [05:33] jaytaoko, let me make sure everything is up to date. Nux just got done compiling and ill re compiling unity [05:34] jaytaoko, while that is going, I wanted to ask you where you left off on the IMContext branch [05:34] jaytaoko, and to kinda hear what your thinking/approach to this was [05:36] bschaefer: http://i.imgur.com/rzPmf.jpg this is what i get [05:36] jaytaoko, Thats good! As im guessing I messed something up ;) [05:37] bschaefer: go ahead with the IM [05:38] jaytaoko, so now that we can create the IMEContext I started seeing where it was failing in the code [05:38] jaytaoko, which happens to be here...looks up the code online one sec [05:39] http://bazaar.launchpad.net/~unity-team/nux/nux.text-entry-dev/view/head:/Nux/InputMethodIBus.cpp#L35 [05:39] jaytaoko, which is pretty much saying that the ibus_ cannont connect with the ibus-daemon! [05:40] jaytaoko, so now I have started looking through that chromeos src to see what is missing before that call [05:40] bschaefer: right. do we need some env setting [05:40] jaytaoko, that one is used when the ibus is already working [05:41] bschaefer: also tell me if I need to have some special settings on my system [05:41] jaytaoko, I will, umm I think to compile the test right now you need to include the $(IBUS_CFLAGS) in the make file [05:41] which I missed doing, because that is need when you create the IMEContext [05:42] bschaefer: can you send me the line in the make file for that [05:42] jaytaoko, yeah [05:43] in tests/Makefile.am::128 [05:43] jaytaoko, put $(IBUS_CFLAGS) \ [05:43] and under that in TestLibs add $(IBUS_LIBS) \ [05:44] jaytaoko, I should just commit these changes... [05:50] bschaefer: system crash! My system has been having issues lately... [05:51] bschaefer: can you type in the makefile info again? [05:51] jaytaoko, that sucks! Also just updated both unity and nux and the text still isn't show hmm [05:51] in tests/Makefile.am::128 [05:51] jaytaoko, put $(IBUS_CFLAGS) \ [05:51] and under that in TestLibs add $(IBUS_LIBS) \ [05:51] jaytaoko, ill be pushing those changes in a second [05:52] jaytaoko, just wanted to double check it working before I push [05:58] bschaefer: ok, compiling [06:02] jaytaoko, alright, just pushed the changed [06:02] jaytaoko, alright, just pushed the changes [06:02] took a little to compile [06:03] bschaefer: pulling [06:07] jaytaoko, another idea for getting this to work would to try to figure out why ibus isn't working through XIM with TextEntry [06:08] jaytaoko, also do you have ibus setup? [06:10] bschaefer: not sure, tell me how to [06:10] jaytaoko, so ibus should come with unity, so on the command line type "ibus-setup" [06:10] jaytaoko, let me know if it ask to start the ibus-daemon === yofel_ is now known as yofel [06:11] bschaefer: it started and I see the IBus preference panel [06:11] jaytaoko, cool, now go over a tab to input method [06:11] ok [06:12] jaytaoko, then under select an input method select chinese and pinyin [06:12] jaytaoko, then add [06:12] bschaefer: done [06:12] ok now close and go the command line and hit Ctrl + Space [06:12] and try typing [06:13] jaytaoko, and you should start typing in chinese! [06:13] jaytaoko, Ctrl + Space to leave that mode [06:14] bschaefer: ok, i type and I see chinese characters [06:14] jaytaoko, cool, that is set up then! You can also switch the Input Method to XIM by right clicking the command prompt [06:15] jaytaoko, right now it should be IBus, but under XIM ibus still works! [06:15] jaytaoko, well it should, since your ibus daemon had already started [06:15] bschaefer: ok it is IBUS [06:15] jaytaoko, switch it to X Input Method [06:16] bschaefer: so Ibus uses XIM? [06:16] and see if ibus is till working [06:16] well ibus works through XIM, which could be a possible fix to this bug (1 of the options) [06:16] bschaefer: yes, ibus is still working if I choose XIM [06:17] jaytaoko, see the reason njpatel and I used that Env var is because the when ibus was working XIM in the Dash wasn't working [06:18] jaytaoko, and it was being very weird, because XIM alone should be able to handle the ibus (I believe) [06:18] bschaefer: ok, so that is a bug? [06:19] bschaefer: what about gtk, it uses Ibus or XIM? [06:19] jaytaoko, possibly, that was kinda a while ago and I should replicate it [06:19] jaytaoko, well IBus has code for XIM, it's slightly confusing and it also has code for GTK [06:20] jaytaoko, the code that gets used for XIM works correctly but when the ibus uses the gtk version it doesn't like korean text [06:20] because of the env var [06:21] bschaefer: ok [06:21] jaytaoko, I feel like im doing a bad job at explaining this... [06:23] bschaefer: it sure isn't easy to understand [06:24] bschaefer: Are you running the text-entry example program? [06:24] jaytaoko, yeah it runs, but no ibus support. Did you comment back in the line where you create the ime? [06:25] jaytaoko, yeah ibus is very confusing, im not sure why there havn't been more problems like this... [06:25] bschaefer: no, let me do that [06:25] bschaefer: wait where is that commented line? [06:26] jaytaoko, under tests/text-entry-xtext.cpp::86 [06:26] bschaefer: wait, you are talking about tests/text-entry-xtest.cpp [06:26] bschaefer: right [06:26] jaytaoko, yeah it is where I thought you were working off [06:27] bschaefer: right, but I should turn this into and example for now rather than a test [06:27] jaytaoko, so I was looking though the other code and it seems to checks if the ibus has made a connection with the daemon a few times [06:28] ibus_controller.cc:877: ibus_init(); [06:28] ibus_engine_controller.cc:123: ibus_init(); [06:28] ibus_ui_controller.cc:126: ibus_init(); [06:29] jaytaoko, which that init according to the api and some comments I saw is what gets ibus setup to make a connection [06:29] jaytaoko, for reference: src.chromium.org/svn/trunk/src/chrome/browser/chromeos/input_method [06:30] bschaefer: looking [06:31] jaytaoko, because I swear if we can get the ibus to successfully to connect I dont see why It wouldn't work [06:31] as you have all the other signals all set up [06:33] jaytaoko, I wonder if we need to also set up an ibus_config [06:34] bschaefer: can you go on mumble or skype? [06:35] jaytaoko, don't have a webcam for skype but I can go on so you can talk haha [06:36] bschaefer: no need for web cam, just a microphone [06:36] jaytaoko, Ill get one tomorrow, forgot that is not what is required...haha [06:37] bschaefer: no problem [06:37] but I can still get on skype for now for that [06:37] jaytaoko, if it helps you [06:38] bschaefer: no if you don't have a mic, we can do that another time [06:38] bschaefer: in ibus_ui_controller.cc, ConnectToIBus is initializing ibus [06:38] jaytaoko, alright, yeah that would help. Ill have one tomorrow! [06:38] bschaefer: just like we do [06:38] jaytaoko, yup [06:38] jaytaoko, but it is also doing it in 2 other locations [06:39] ibus_controller.cc:877: ibus_init(); [06:39] ibus_engine_controller.cc:123: ibus_init(); [06:39] jaytaoko, in those 2 locations it is doing pretty much the same thing [06:39] jaytaoko, but it seems like they have to keep checking if the daemon is ready for a connection [06:40] jaytaoko, this code actually has a lot of comments [06:40] bschaefer: ok in our case we only need to do it once and make sure it ibus_bus_is_connected returns true [06:41] jaytaoko, yeah, but I wonder if the ibus_config has to be setup [06:42] jaytaoko, GDBusConnection* ibus_connection = ibus_bus_get_connection(ibus_); [06:42] jaytaoko, that line looks like more of a direct call to get the connection [06:43] bschaefer: so I checked and ibus_init is working [06:43] bschaefer: it returns a valid ibus pointer [06:44] jaytaoko, so after the ibus_init and bus_ = new... the pointer isn't NULL? [06:44] jaytaoko, im about to send the ibus_bus_is_connected through a while loop to see if it will ever connect... [06:44] bschaefer: yes, it is not null. You maybe right, we have to call ibus_bus_get_connection before we check ibus_bus_is_connected [06:46] jaytaoko, good, and yeah Im just wondering why they keep checking 3 different times if bus_ is made and ready to go [06:46] jaytaoko, and right now ibus_bus_is_connected(bus_) is always returning false... [06:47] jaytaoko, which is why im thinking something is missing...I need to take a look at that smaller code example you found [06:48] jaytaoko, for reference: http://code.google.com/p/google-gadgets-for-linux/source/browse/trunk/extensions/gtk_edit_element/ [06:48] bschaefer: I added GDBusConnection* ibus_connection = ibus_bus_get_connection(bus_); [06:49] bschaefer: I expect to receive a message in void IBusIMEContext::OnConnected(IBusBus *bus) [06:50] jaytaoko, oo, so look at ibus_ui_controller.cc ::150 [06:50] jaytaoko, yeah it should send a message there when it gets connected [06:51] so it looks like they get the connection then make a service out of it [06:51] jaytaoko, but im not really sure what the IBusPanelService is... [06:51] thumper, around? [06:53] bschaefer: how to you get to see the line numbers [06:53] o I wget it so i can mess around with it in the command line [06:53] jaytaoko, and grep [06:54] bschaefer: ok, I copied the file into my text editor [06:54] are there no PPA hooks for the unity-team PPA? [06:54] I can' [06:54] I can't file a bug using ubuntu-bug [06:56] jaytaoko, man there is so much code to dig through... [06:57] bschaefer: true! [06:57] jaytaoko, i really wish the api had more details, like I looked at the ibus_panel and it just says it is an ibus_panel haha [06:58] jaytaoko, very redundant [06:58] bschaefer: yeah, it isn't helping much [06:58] bschaefer: ok, almost 2am here. [06:58] bschaefer: I will try to continue tomorrow if possible [06:58] jaytaoko, yeah go to sleep, Ill dig some more but I spent a lot of time doing other stuff today [06:59] jaytaoko, no worries, ill spend some more time on it and a lot more tomorrow [06:59] bschaefer: thank you very much for your help! [06:59] bschaefer: cheers [06:59] jaytaoko, no worries, this is still fun (somewhat haha; when it is fixed it iwll be worth it!) [06:59] jaytaoko, you too, have a good night! [07:00] bschaefer: thanks ! [07:07] morning didrocks [07:07] hey jono [07:08] hows things? [07:08] I have been rocking the multi-monitor goodness herer [07:08] generally working well [07:08] I'm fine here, thanks! Quite cold in France though :) [07:08] ah nice! ;) [07:08] didrocks, quick q: Nick told me we can use ubuntu-bug on PPAs if there is a hook included in the PPA [07:08] do you know if this is the case? [07:09] jono: yeah, it will file bug upstreams only, but that won't really help us in that case, because for instance, crashes won't get retraced [07:10] jono: I think that we should work on a more general purpose on the crash database next cycle to support some "blessed" ppas as we are going more and more to the ppa direction [07:10] (or using the -proposed pocket) [07:10] right [07:10] if we are moving more and more to proposed, this seems like a natural next step [07:10] indeed :) [07:11] that's why I didn't installed the hook, as the most valuable info from the ppa are the crashes [07:11] crashes* [07:11] right [07:11] for the rest, the testing tool should cover the regressions :) [07:11] cool [07:12] btw, do you know if notify-osd bubbles should appear on both monitors? [07:13] jono: I think design planned that was two cycles already, but nobody in dx is maintaining the osd notification [07:14] right, I filed https://bugs.launchpad.net/unity/+bug/928095 and unfortunately it kind of means that I miss almost all the bubbles [07:14] Launchpad bug 928095 in unity "Notification bubbles don't appear on all monitors" [Undecided,New] === Guest68463 is now known as Jens === Jens is now known as _jens [09:00] mhr3, hey, pushed new changes :) [09:01] mhr3, should have question that since max_results had a default value!! [09:02] bschaefer, great :) [09:16] mhr3, also, if you get a chance could you look over my no-results branch? I fixed thumpers problems, but the more people who look at the better :) [09:16] bschaefer, yea, sure i'll give it a go [09:16] mhr3, since you worked on the no-results-hint, https://code.launchpad.net/~brandontschaefer/unity/fix-711199 [09:17] mhr3, thanks, I kept the hardcoded string in there but I will have a talk with thumper about it [09:19] mhr3, Ill owe you a lot of beer by the next uds at this rate haha [09:27] greyback: screeinfo warning fixed [09:27] tsdgeos: ok, approving (and morning!) [09:27] greyback: morning :-) [09:28] tsdgeos: you see my message on https://code.launchpad.net/~aacid/unity-2d/unity-2d_panel-newbuttons/+merge/91442 [09:28] yep [09:28] saw that [09:29] ok, just thought I'd annoy you a little more :) [09:39] greyback: which shortcut keys do you have associated for mumble-speak shortcut? [09:56] tsdgeos: ctrl+shift [09:57] greyback: can you try this? close mumble, kill all your unity-2d-launcher, start mumble, start unity-2d-launcher, check if your ctrl+shift shortcut works [09:57] tsdgeos: launcher or shell? Or does it matter [09:58] greyback: well, Kaleo complained that failed on shell and we should fix it before merging, but i can reproduce it too with launcher, so please try with launcher to see if it's a bug we introduced with shell or was already there [09:58] tsdgeos: ok, trying.. [10:00] tsdgeos: yep, I reproduce here too [10:00] greyback: ok, thanks, will open a bug against unity-2d and remove it from the stack of things to fix in -shell before merging [10:01] tsdgeos: ok. Mumble isn't receiving any keystrokes afaics. Possibly a Mumble bug (shock!) [10:02] greyback: maybe [10:02] you have to kill it and start it again [10:02] then it works [10:02] tsdgeos: but report against 2d for now, we'll deal with it later [10:03] tsdgeos: got time for a quick mumble? [10:03] greyback: sure [10:15] tsdgeos: screeninfo-refactor approved [10:15] yay [10:16] greyback: yay :) [10:19] why does the volume control have a light grey background now? :p [10:24] snadge: good question, Cimi? [10:24] Cimi: 11:19 < snadge> why does the volume control have a light grey background now? :p === kaleo is now known as Kaleo [10:25] Kaleo, bug in ido [10:26] Cimi: cheers [10:31] Kaleo: the mumble bug affects regular unity-2d too so together with greyback we agreed it doesn't make sense to delay shell merging because of that [10:32] tsdgeos: can you confirm that https://code.launchpad.net/~aacid/unity-2d/unity-2d_test_alt_f2_esc_alt_f2/+merge/91660 is for merging in trunk, and not shell? [10:32] greyback: yes, that's the test, we make it go to trunk and then merge it to shell, no? [10:33] tsdgeos: yep, just wanted to check [10:39] tsdgeos: agreed [10:40] tsdgeos: any idea since when? [10:40] tsdgeos: do we have a bug report? [10:40] nope & nope [10:42] tsdgeos: rev 893 seems like a good candidate for being the culprit [10:43] tsdgeos: and it's not [10:43] tsdgeos: bisecting [10:43] Kaleo: oki [10:46] greyback: fixed conflicts on screeinfo again https://code.launchpad.net/~unity-2d-team/unity-2d/refactor-screeninfo/+merge/91471 approve please? === _salem is now known as salem_ [10:57] tsdgeos: raaah, I reverted up to rev 800 and still no luck.. [10:57] Kaleo: you sure it's not been broken "forever" ? [10:58] tsdgeos: pretty sure as I was using mumble everyday :) [10:58] Kaleo: and starting unity-2d-launcher while mumble was on? [10:58] tsdgeos: good point [11:00] tsdgeos: right, you are correct [11:01] tsdgeos: with any revision, mumble does not break if the launcher was started before mumble [11:01] tsdgeos: and with any revision, in the opposite order, it breaks [11:01] yeah :/ [11:01] might even be a mumble bug [11:01] tsdgeos: so, it's problematic if Unity crashes when mumble is running [11:01] tsdgeos: otherwise, no big deal [11:01] true [11:01] tsdgeos: let's report the bug and forget about it [11:02] tsdgeos: priority medium I would say [11:02] ok [11:02] will do in a sec [11:02] tsdgeos: thanks! [11:19] tsdgeos: doing another round of quick testing on shell I spotted one issue [11:20] tsdgeos: the dash maximise button is still at the bottom right corner :( [11:20] Kaleo: shoot [11:20] tsdgeos: I'm figuring out why with nerochiaro [11:20] dash maximise button? [11:20] you mean the one that shows in the panel? [11:21] tsdgeos: the one button that maximises the dash; it used to be at the bottom right corner of the dash and was moved to the panel [11:21] tsdgeos: but in the shell branch, we have both.. [11:21] ahhh [11:21] do i expect selecting an album in the music lense will start that music playing, rather than just opening banshee and doing nothing ? (precise) [11:24] Kaleo: do i remove it? or wait for nerochiaro's answer? [11:26] tsdgeos: wait a minute [11:40] MacSlow: ping [11:41] greyback, what's up? [11:42] MacSlow: just something small about the filter-compact stuff. I just want to check with you that it is the lens which chooses for checkboxes to be compact or not [11:42] greyback, yes [11:42] MacSlow: ok I thought so, thanks. [11:42] The Documents lens needs to have the "Last Modified" checkboxes as compact [11:43] I'll try to get that done [11:43] greyback, I only ever heard that the music-lens was the only one to use the compact / 3-column layout [11:43] MacSlow: that's all, thank you [11:44] greyback, but my last update on that is from the rally in Budapest... so [11:44] MacSlow: sure. I'm just comparing with mockups. We had a hack in Unity2D for this, and we applied it also for LastModified, hence I noticed. [12:08] tsdgeos: ok, the conclusion is that we need to revert 958 in -shell [12:12] Kaleo: ok, want me to do that? [12:13] tsdgeos: sure [12:13] tsdgeos: so the issues are: [12:13] tsdgeos: the 1 pixel rectangle on the left [12:13] tsdgeos: the lack of shadow for the dash bottom right corner in composited mode [12:14] tsdgeos: a more proper solution is to check for composited/non composited mode in InputShapeManager [12:14] tsdgeos: and do ShapeInput or ShapeBounding depending on the case [12:17] Kaleo: and r958 reversion [12:22] Kaleo: about the 1 pixel rectangle on the left in non composited mode, i'm not sure it's worth fixing [12:23] greyback is working on something xfixes based to detect the mouse "pushing" into the launcher [12:23] so any work we do now on that will be "lost time" [12:28]