Status Meetings:2006-08-28:Log

From Camino Wiki
Jump to navigation Jump to search
09:05 <@ss|work> Alright.
09:05 <@ss|work> Welcome everyone to another weekly meeting
09:05 <@ss|work> Please open the wiki page and follow along.
09:05 <@ss|work> http://wiki.caminobrowser.org/Status_Meetings:2006-08-28
09:05 <@ss|work> First up, I hand the floor to delliott who's going to talk about SoC 
                 stuff.
09:05 <@ardissone|away> (lots of stuff today)
09:05 <@delliott> Thanks.
09:05 <@delliott> I'll try to keep this quick seeing as the list is long.
09:06 <@delliott> We learnt at Uni that it's important to chat about things after a project
09:06 <@delliott> Mike and I have finished our evaluations
09:06 <@delliott> I wrote down that he smells
09:06 <@delliott> I thought I'd start by saying that as a newcomer, the community was 
                  pretty receptive.
09:07 <@delliott> Everybody did what they could to make me feel like part of the team
09:07 <@ardissone|away> :)
09:07 <@delliott> And that is not easy when new people show up and start wailing about 
                  build errors.
09:07 <@pinkerton> i do smell, and i cry a lot
09:07 <@peeja> had me fooled when i got here. :)
09:08 <@delliott> It's not too hard to get involved in the dev community so that is a good 
                  thing
09:08 <@delliott> We are working on our build docs so hopefully more people will get 
                  involved too
09:08 <@ardissone|away> (hopefully next week...)
09:08 <@delliott> Onto the progress of the project.
09:09 <@delliott> As we all know, I didn't finish the code.
09:09 <@ss|work> booo
09:09 <@ss|work> hisss
09:09 <@delliott> But I am still here, so it is pretty safe to say that it will be 
                  finished.
09:09 <@froodian> code is never finished
09:09 <@ss|work> ;)
09:09 <@froodian> you accomplished a lot regardless
09:09 <@hwaara> code is always buggy :(
09:09 <@hwaara> :)
09:09 <@peeja> and there were some bumps along the way... :(
09:09 <@ardissone|away> it was great just to be able to drag tabs for a bit :)
09:10  * ardissone|away hides
09:10 <@delliott> I took a lot of criticism from lots of different people about my project 
                  this summer and I took it personally when I should have taken it more as 
                  an interest in what I was doing.
09:10 <@delliott> So I'm gonna apologise for moaning in the channel.
09:11 <@delliott> But it has all turned out well because now I think I know the difference 
                  between moaners and people who have genuine feedback.
09:11 <@froodian> :)
09:11 <@ss|work> *cough* smorgan *cough*
09:11 <@delliott> Hehe
09:12 <@delliott> I need to do a little more work on bug 319777 and hopefully it will R+
09:12 <@delliott> Bug 160720 is a whole different level of pain
09:13 <@delliott> Mainly because we have lots of different views and getting to the right 
                  one can be problematic
09:13 <@hwaara> (unfortunately there's no bot here to tell what the bugs are about)
09:13 <@delliott> So yes, I didn't finish the code but seeing as I'd never written Obj-C 
                  before, I'm not too gutted.
09:13 !sand.mozilla.org ss|work invited thebot into the channel.
09:13 -!- thebot [pid-12949@2E23C0E1.1B475606.C8C62208.IP] has joined #camino-mtg
09:14 <@froodian> (first one is scrolling, second one is dragging)
09:14 <@hwaara> ah, thx
09:14 <@delliott> Dragging between windows is a subset of dragging
09:14 <@delliott> Once you can drag it will be a minor tweak to drag between windows
09:14 <@ss|work> On to the criticism!
09:14 <@ss|work> ;)
09:14 <@delliott> Positive stuff first
09:15 <@ss|work> Oh, right.
09:15 <@delliott> This is where everybody can join in.
09:15 <@delliott> So, positive points...
09:15 <@froodian> lots of new camino code!
09:15 <@ardissone|away> we didn't scare you off :)
09:15 <@froodian> good tab stuff in line for future releases
09:15 <@froodian> heh
09:15 <@froodian> yeah
09:15 <@ss|work> We've got a new lifer.
09:15 <@delliott> ardissone|away: I nearly left after I killed my MacBook :p
09:16 <@ardissone|away> :(
09:16 <@BruceD> someone new learns that Obj-C and the Camino code in ObjC is different, 
                but not actually scary
09:16 <@delliott> I quite enjoyed the challenge of working with people around the world
09:16 <@hwaara> great to see progress on the tabs bugs that we'd been fighting about before
09:17 <@delliott> Until you actually get stuck into open-source you have these magical 
                  ideas about what it might be like
09:17 <@hwaara> lots of patience and gray hair
09:17 <@delliott> One of the other positive things might be that the project has somebody 
                  who is fiddling with the tab code again
09:17 <@hwaara> and perseverance :)
09:17 <@froodian> cool
09:17 <@froodian> criticism?
09:17 <@ardissone|away> !!!!
09:17 <@delliott> Constructive criticism, fire away.
09:18 <@ss|work> I was actually looking for it from you, delliott.
09:18 <@delliott> Ok
09:18 <@ss|work> :)
09:18 <@ardissone|away> mhm
09:18 <@delliott> I'll start by saying that what I tried to achieve was ridiculous.
09:18 <@peeja> yes.
09:18 <@delliott> Considering my total lack of experience in Obj-C / Mozilla
09:19 <@peeja> thank god for people who take on the ridiculous. :)
09:19  * hwaara is in the same situation, fwiw
09:19 <@delliott> If we put out bounties for next year we need to think about whether or 
                  not we want to take on a wild child or somebody who can already code.
09:20 <@delliott> I found the reviews system gruelling. Though the people reviewing the 
                  code probably found it worse!
09:20 <@hwaara> why?
09:21 <@delliott> Perhap because the patches I was writing were larger than many first 
                  patches
09:21 <@hwaara> was it the system or the reviews in itself? :)
09:22 <@delliott> The system is fine, I think I just found the reviews difficult.
09:22 <@delliott> But most people probably don't try to submit big patches from the start
09:22 <@BruceD> Yeah, its very difficult getting code past good reviewers, until you've 
                had (lots of) practice
09:23 <@delliott> BruceD: Yes, but now I think "What will the reviewer say?"
09:23 <@pinkerton> generally the problem is getting people to do reviews and it getting 
                   backed up
09:23 <@delliott> So it makes a better coder out of you if you can stick it.
09:23 <@froodian> pinkerton: yes
09:23 <@ss|work> pinkerton: Which is something we're going to talk about later.
09:23 <@BruceD> yeah. you're later code is much better structured than your early patches
09:23 <@pinkerton> ok
09:23 <@froodian> speaking of which, we should keep this moving
09:23 <@ss|work> Yeah, we should.
09:23 <@hwaara> how much later?
09:23 <@ss|work> Today.
09:23 <@delliott> I cannot criticise much of the Camino team
09:23 <@delliott> Just myself
09:23 <@delliott> Conclusion
09:24 !sand.mozilla.org ss|work invited _Tsk_ into the channel.
09:24 <@delliott> Had a great time, nearly gave up, fought on, am still here. Beer!
09:24 <@froodian> :D
09:24 <@hwaara> nice!
09:24 <@delliott> Oh, and we need to pester Hofmann for the $500
09:24 <@ss|work> Awesome.
09:24 <@ss|work> That's pinkerton's job.
09:24 <@hwaara> can we move the review topic up a bit?
09:24 <@ardissone|away> :)
09:25 <@ss|work> It's not as important as the rest, actually...
09:25 <@delliott> Ok, I'm done. Thanks for reading :)
09:25 <@ss|work> mento: This next one is mainly for yo.
09:25 <@ss|work> you*
09:25 <@hwaara> it's important for some of us..
09:25 <@ss|work> Who can we bug when we have tbox issues during the "off hours"
09:25 <@froodian> hwaara: we'll get there
09:25 <@ss|work> hwaara: I'm not saying it's not important, just not "as" important.
09:25 <@mento> ss|work: build@mozilla.org
09:25 <@mento> #build
09:25 <@mento> preed
09:25 <@mento> coop
09:25 <@hwaara> it's dinner time here, but whatever
09:25 <@mento> rhelmer
09:25 <@ardissone|away> alos, pawn needs its crash dialogs reset
09:25 <@ss|work> mento: Alright, because currently we've been having a lot of red/orang.
09:26 <@mento> so has the trunk
09:26 <@ss|work> Yeah, what ardissone|away said.
09:26 <@ss|work> mento: Sure, but we've had more. ;)
09:26 <@ardissone|away> starting at 5pm on Fridays :/
09:26 <@mento> yeah, a little bit more
09:26 <@ss|work> And things like lost perf is going to be hellish...
09:26 <@mento> let me check pawn's dialogs now
09:26 <@ss|work> (later anyway)
09:26 <@ss|work> Next:
09:26 <@ardissone|away> maybe maya, too, since the trunk was fubared
09:27 <@ss|work> 1.0.3
09:27 <@ss|work> We have three patches that need landing.
09:27 <@ardissone|away> we have 3 patches ready
09:27 <@ardissone|away> kreeger was gonna land them this am
09:27 <@ss|work> It's very important that we get them in sooner rather than later.
09:27 <@mento> 7 crash dialogs closed
09:27 <@ss|work> So we have time to get everything in.
09:27 <@ss|work> Yuck
09:27 <@mento> thank you for pointing that out
09:27 <@ardissone|away> but obviously he didn't shwo
09:27 <@ardissone|away> mento: np
09:27 <@ss|work> So, does anyone have time to land three patches?
09:28 <@ardissone|away> (can we get #build to do that, too)
09:28 <@ss|work> (On the 1.8.0 branch)
09:28 <@ss|work> Anyone able to check it in?
09:28 <@pinkerton> mento: thx i was just doing that
09:29 <@ardissone|away> ss|work: kreeger seemed ok with doing it
09:29 <@ss|work> Alright. I'll bug him more.
09:29 <@ss|work> We're looking good for 1.0.3
09:29 <@ss|work> But we still need to know about bug 175651
09:29 <@mento> pinkerton: np
09:29 < thebot> ss|work: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=175651 maj, --, 
                Camino1.1, sfraser_bugs@smfr.org, ASSI, CJK (at least Chinese and 
                Japanese) font Preferences changes not applied to any CJK Web page
09:29 <@ss|work> mento: You goingt o have time this week for that bug?
09:29 <@ss|work> going to*
09:29 <@mento> probly
09:29  * ss|work crosses his fingers
09:30 <@ardissone|away> :)
09:30 <@ss|work> mento: Next question for you, what needs to be done for bug 343837?
09:30 <@ardissone|away> also, what do we want to do about the fubared ciphers dialog on 
                        180/103?
09:30 < thebot> ss|work: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=343837 maj, --, 
                Camino1.1, nobody@mozilla.org, NEW, SSL2/weak ciphers disabled messages 
                contain "(null)" instead of "Camino"
09:30 <@ardissone|away>     *  Leave it as is
09:30 <@ardissone|away>     * Patch the string to say "Camino and this site" (lose dynamic 
                        hostname) 
09:30 <@mento> ardissone|away: you said that we can't really fix it, right?
09:30 <@mento> yeah
09:31 <@mento> it looks bad as it is now
09:31 <@ardissone|away> we could patch the string like that, but yeah
09:31 <@mento> but i think it's important to have the hostname in the string
09:31 <@ardissone|away> k
09:31 <@ardissone|away> so we do nothing there for 10x
09:31 <@ss|work> Awesome.
09:31 <@ss|work> Everything else for 1.0.3 will probably not make it.
09:32 <@ss|work> Next:
09:32 <@ss|work> 1.1
09:32 <@ardissone|away> i'd like to try and get bugs landed and relnotes ready by Fri
09:32 <@ss|work> We're _slowly_ progressing.
09:32 <@ss|work> 2 bugs/week isn't that good
09:32 <@ss|work> But it is progression.
09:32 <@ardissone|away> not as fast as 2 weeka ago
09:32 <@ardissone|away> 2 weeks ago we landed 5-8 bugs
09:32 <@ss|work> This is where kreeger would talk about RSS.
09:32 <@ss|work> But he's not here.
09:33 <@hwaara> I'd like to fix cmd-clicking/middle-clicking on bookmarks, but all 
                super-reviewers are swamped
09:33 <@froodian> the speed problem is due to reviewing
09:33 <@froodian> which leads us to our next issue
09:33 <@hwaara> it's a dogfood thingy
09:33 <@ss|work> The SR queue is actually good right now.
09:33 <@ss|work> Is the R queue that'
09:33 <@ss|work> s having problems
09:34 <@ardissone|away> mento: i have those proj patches for you and hwaara ;)
09:34 -!- SimonTheSoundMan [SimonTheSo@6F6BB858.80E17092.97A297A6.IP] has joined 
          #camino-mtg
09:34 <@mento> nice, is it in my review q?
09:34 <@ardissone|away> sr q
09:34 <@ss|work> Yes.
09:34 <@ss|work> We have a lot of patches waiting for R.
09:34 <@mento> ok
09:34 <@ss|work> We need more reviewers over all...
09:34 <@froodian> and the most important one is haas's patch
09:34 <@ss|work> Yes.
09:35 <@ss|work> We *need* that reviewed.
09:35 <@froodian> somebody needs to take that in a major way
09:35 <@ss|work> And today we need someone to decide to review it.
09:35 <@ss|work> Who's up for it?
09:35 <@ardissone|away> it's not huge, but it's not tiny
09:36 <@ardissone|away> hwaara?
09:36 <@hwaara> I took a look
09:36 <@hwaara> I think the approach looks good
09:36 <@hwaara> I just hope the sr will know about threads :)
09:37 <@ss|work> Can you give it a full R then?
09:37 <@hwaara> (I do, but it's still not easy as a pie)
09:37 <@hwaara> ok
09:37 <@ss|work> So we can get it to the SR level?
09:37 <@hwaara> if I can get some kind of traction on my middle click bug in return :P
09:37 <@hwaara> it's been blocked for god knows how long
09:37 <@ss|work> hwaara: Does it have an sr?
09:37 <@ss|work> I haven't seen it in the queue...
09:37 <@hwaara> ss|work: no, it has nothing
09:38 <@hwaara> it's in pinkerton's sr queue
09:38 <@ardissone|away> no one wants to touch it :(
09:38 <@ss|work> pinkerton: Do you have time to look at the patch in bug 228840?
09:38 < thebot> ss|work: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=228840 enh, --, 
                Camino1.1, hwaara@gmail.com, NEW, Bookmarks toolbar - Middle-click should 
                open bookmark in new tab
09:38 <@hwaara> but pink don't have time, sfraser has said no too, and I've asked mento a 
                few times but I think it's too much for him too
09:38 <@ardissone|away> and smorgan is the review q
09:38 <@hwaara> ss|work: he said no in comment 41
09:39 <@ss|work> smorgan reviewed it before...
09:39 <@ss|work> Why don't we do this:
09:39 <@ss|work> You take a couple reviews of froodian's stuff from smorgan and give that 
                 one to smorgan.
09:40 <@froodian> i have some small stuff
09:40 <@ardissone|away> and when we see him, we'll promote that reviewv ;)
09:40 <@froodian> very small
09:40 <@hwaara> I can try, my time is limited too though
09:40 <@ss|work> hwaara: Most of froodian's stuff is pretty small.
09:40 <@froodian> hwaara: when the meeting's done i'll tell you which bugs you want ;)
09:40 <@hwaara> sure, target me!
09:40 <@froodian> k
09:41  * ardissone|away wonders how BruceD's schedule looks
09:41 <@hwaara> is anyone reviewing the RSS patches?
09:41 <@froodian> smorgan
09:41 <@ardissone|away> hwaara: smorgan was
09:41 <@BruceD> just peered at the review queue, most of the patches look bigger than I'm 
                going to have time to review properly
09:41 <@froodian> it's close
09:41 <@ardissone|away> k
09:42 <@froodian> next item?
09:42 <@ss|work> Sure
09:42 <@hwaara> BruceD: maybe if you divide some patch ?
09:42 <@ardissone|away> next mon is labor day
09:42 <@hwaara> I did that with torben's last zoom patch, it worked well
09:42 <@ss|work> Is everyone okay with moving the meeting to Tuesday?
09:42 <@ss|work> I know I'll be drunk next Monday.
09:42 <@BruceD> hwaara: what do you mean?
09:43 <@hwaara> BruceD: if it's some big patch where changes are in different places, it 
                has helped me in the past to divide the review in a few days
09:43 <@froodian> I'll have class at the time
09:43 <@hwaara> and do one part every day
09:43 <@froodian> on tuesday
09:43 <@peeja> fine with me
09:43 <@ss|work> Who else has an issue with Tuesday?
09:43 <@ss|work> pinkerton, mento:
09:43 <@Pinolo> fine with me too
09:44 <@mento> tuesday ok
09:44 <@BruceD> yeah, monday's are much better for me, as likely to be abroad for work by 
                Tuesday evening, and European hotels charge _shocking_ rates for WiFi :-(
09:44 <@ss|work> BruceD: Normally it'll be Monday
09:44 <@ss|work> Just next week we want to move it due to a holiday in the US.
09:44 <@BruceD> yeah, fine if u want to change just next weeks
09:45 <@ss|work> I'll assume pinkerton's okay with Tuesday morning unless he says 
                 otherwise.
09:45 <@ss|work> froodian: You can read the log. ;)
09:45 <@froodian> yup
09:45 <@ss|work> bug 311781
09:45 < thebot> ss|work: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=311781 nor, --, 
                Camino1.1, stridey@gmail.com, ASSI, Fix toggle/non-toggle situation in 
                Bookmarks/History by adding a new shortcut
09:45 <@ss|work> What do we want to do there?
09:45 <@ss|work> Should there be three menu items?
09:45 <@ss|work> Or is "toggle" an alternate?
09:46 <@ardissone|away> toggle is the current "default" so i'm leery of hiding/changing it
09:46 <@froodian> i say three menu items
09:46 <@ardissone|away> although half the world seems to hate it
09:46 <@froodian> plus, that way we can use cmd-shift-M
09:46 <@froodian> which is much nicer than cmd-opt-B
09:46 <@froodian> (just for ease of hitting)
09:47 <@ss|work> I'm alright with three menu items...
09:47 <@peeja> why can't cmd-B just close the BM if bookmarks are up?
09:47 <@froodian> so cmd-Y = show history, cmd-B = show bookmarks, cmd-shift-M = toggle 
                  manager
09:47 <@ss|work> I'm not sure how that'll look visually.
09:47 <@peeja> and cmd-Y if history is up?
09:47 <@froodian> peeja: because it's lopsided with history
09:48 <@peeja> lopsided?
09:48 <@froodian> cmd-b is a toggle
09:48 <@ardissone|away> y always focuses history
09:48 <@froodian> cmd-y isn't
09:48 <@ss|work> peeja: If I hit cmd-b click on the "history" collection, what closes the 
                 manager?
09:48 <@Pinolo> I don't see much the point
09:48 <@peeja> ss|work: don't know off the top of my head
09:48 <@ss|work> :)
09:48 <@ardissone|away> Cmd-Y either shows the mgr and history, or switches to history in 
                        a shown mgr
09:49 <@froodian> ok.  so, so cmd-Y = show history, cmd-B = show bookmarks, cmd-shift-M = 
                  toggle manager.  three menu items.  three toolbar items.
09:49 <@froodian> does anybody have a problem iwth that?
09:49 <@froodian> pinkerton: a=you?
09:49 <@Pinolo> Apart from reporter are ther many peolple who think that current behaviour 
                is wrong?
09:49 <@delliott> It has never bugged me.
09:49 <@ardissone|away> Pinolo: yes, people file bugs all the time
09:49 <@ardissone|away> complain in the forum
09:49 <@delliott> I never even noticed what this bug describes
09:49 <@ardissone|away> dunno about feedback
09:49 <@froodian> delliott: you'd also never noticed tab groups
09:49 <@Pinolo> ok, tnx ardissone
09:50 <@hwaara> ok, gotta go now
09:50 <@BruceD> three menu and toolbar items sounds horrible, though I sort of like the 
                key combos
09:50 <@hwaara> was this the last item?
09:50 <@peeja> froodian: here's what i don't like:
09:50 <@froodian> hwaara: not even close
09:50 <@ardissone|away> hwaara: 2-3 more bugs
09:50 -!- _Tsk_ [ludovic@183BDF58.599B7C47.C7DB8B78.IP] has joined #camino-mtg
09:50 <@peeja> if you hit cmd-b by accient, you have to press cmd-opt-B to undo it
09:50 <@ardissone|away> there's Ludo
09:50 <@hwaara> ok
09:50 <@hwaara> later all
09:50 <@Pinolo> hi Ludo!
09:50 <@froodian> peeja: or back
09:50 < _Tsk_> sorry fogot and had network issues
09:50 -!- hwaara [hwaara@5718B00C.2191EBAD.7EF74A51.IP] has left #camino-mtg []
09:50 <@BruceD> why not just make cmd+Y a toggle too!?
09:51 <@Pinolo> mmmm, cmd-shift-M....
09:51 <@ardissone|away> BruceD: don't go there ;)
09:51 <@peeja> i'm with BruceD, myself...
09:51 <@ardissone|away> there are old bugs suggesting that; see pink's reaction there
09:51 <@froodian> so if you're in bookmarks and hit history, the bookmarks manager closes?
09:51 <@ardissone|away> (i filed one of them)
09:51 < _Tsk_> # ?
09:52 <@ardissone|away> _Tsk_: bug Bug 311781
09:52 <@peeja> no, if you're in bms, history comes up
09:52 < thebot> ardissone|away: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=311781 
                nor, --, Camino1.1, stridey@gmail.com, ASSI, Fix toggle/non-toggle 
                situation in Bookmarks/History by adding a new shortcut
09:52 <@froodian> peeja: but then it's not a toggle
09:52 <@ss|work> So it's a conditional toggle.
09:52 <@ss|work> Which is weird.
09:52 <@froodian> cmd-b ALWAYS toggles the manager
09:52 <@ardissone|away> _Tsk_: first bug in "Bugs that need action" on the wiki agenda
09:52 <@froodian> (now)
09:52 <@peeja> i don't find it weird.
09:52 <@peeja> i find it intuitive
09:52 <@froodian> it's still lopsided
09:52 <@pinkerton> right, history should not be a conditional toggle
09:52  * _Tsk_ dosen't follow the wiki
09:53 <@ardissone|away> _Tsk_: the /topic
09:53 <@peeja> you can see bookmarks, or history, or none.
09:53 <@peeja> asking for what you don't see brings it up.
09:53 <@ardissone|away> no
09:53 <@BruceD> if you're viewing history and hit cmd+B that should change to the bookmarks
09:53 <@peeja> asking for what you're looking at toggles it away
09:53 <@froodian> this is why we want three
09:53 <@froodian> one for "asking what you want"
09:54 <@froodian> and one for toggling the whole manager
09:54 <@ss|work> pinkerton: Are you okay with adding a third item for "toggle manager"?
09:54 <@ardissone|away> right, what BruceD mentions is the problem with peeja's siggestion
09:54 <@pinkerton> no
09:54 <@peeja> i'm saying you can do it with two
09:54 <@ss|work> froodian: ^^
09:54 <@peeja> ardissone|away: how's that?
09:54  * peeja defers to pink
09:54 <@ardissone|away> peeja: if you're looking at history and hit cmd-b now, you don't 
                        see bookmarks, you close the mgr
09:55 <@froodian> ss|work: what am I looking at?  pinkerton saying no to third item?
09:55 <@froodian> so what're we going to do?
09:55 <@ss|work> froodian: Yeah.
09:55 <@peeja> ardissone|away: right, i meant in my suggestion. but it's past now.
09:55 <@pinkerton> i'm saying no to a 3rd item
09:55 <@froodian> what we have is lopsided and confusing
09:55 <@peeja> oh
09:55 <@froodian> pinkerton: ok.  but what we have is lopsided and confusing.  what should 
                  we do instead?
09:56 <@pinkerton> there is no perfect solution
09:56 <@froodian> conditional toggle is out, 3rd item is out...
09:56 <@pinkerton> we'll stick with what we have, moving along
09:56 <@froodian> k
09:56 <@ardissone|away> why is 3rd item worse than what we have now?
09:56 <@froodian> Bug 287708
09:56 <@ardissone|away> sigh
09:56 < thebot> froodian: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=287708 tri, --, 
                Camino1.2, stridey@gmail.com, NEW, When adding bookmark, "bookmark bar" 
                should be above "bookmarks menu" in "Create In" menu
09:56 <@froodian> everybody read what i wrote on the wiki page
09:56 <@ss|work> "e've pretty much decided that we should do comment 9, but that's a big 
                 project. Is it ok with us for froodian to make the default selected 
                 folder be the bookmarks bar (instead of the top of the list) as a 
                 stop-gap measure?"
09:56 <@froodian> i just want a stop-gap solution until we can fix it properly
09:56 <@delliott> 1. No
09:57 <@froodian> delliott: wrong bug
09:57 <@froodian> (i think, since you're numbering it)
09:57 <@delliott> D'oh
09:57 <@BruceD> changing the default without re-ordering sounds like a sensible interim 
                solution
09:58 <@froodian> anybody else?  pink?
09:58 <@pinkerton> i haven't been folowing
09:58 <@froodian> see ss|work's big comment
09:58 <@pinkerton> recall i'm triple booked
09:58 <@froodian> a few lines up
09:58 <@froodian> i know, i'm sorry
09:59 <@ss|work> And add a "W" to the beginning of my comment.
09:59 -!- SimonTheSoundMan [SimonTheSo@6F6BB858.80E17092.97A297A6.IP] has quit [Quit: 
          SimonTheSoundMan]
09:59 <@pinkerton> why can't we move the bookmark bar above it for everything?
09:59 <@froodian> because that's a big project too
10:00 <@froodian> (i think. that code is confusing as hell)
10:00 <@froodian> but i'd be willing to do that instead
10:00 <@pinkerton> get dhaas to fix it :)
10:00 <@ss|work> Haha
10:00 <@froodian> ok.  if that's your answer, that's ok
10:01 <@froodian> i already have a patch for just slecting bookmarks bar by default though
10:01 < _Tsk_> Anybody in touch with dhaas ?
10:01 <@froodian> so it'd be very very easy ;)
10:01 <@pinkerton> we should remember what the user chose, eg, if they're adding a bunch 
                   of bookmarks in a row to a folder, having to reselect that folder is a 
                   bitch
10:01 <@froodian> yeah, we'd still do that
10:01 <@pinkerton> we fixed that a long time ago (0.7?)
10:01 <@froodian> this is just for the first selection after launch
10:01 <@pinkerton> just default to bookmark bar if no previous selection?
10:01 <@pinkerton> i'm ok with that
10:01 <@froodian> yes
10:01 <@pinkerton> ship it
10:02 <@ss|work> Next
10:02 <@froodian> :D
10:02 <@ss|work> bug 159230
10:02 < thebot> ss|work: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=159230 nor, --, 
                Camino1.2, stridey@gmail.com, NEW, Should be able to bookmark a page 
                without an intermediate dialog
10:02 <@froodian> smfr postulates in comment 0 that we shouldn't have the intermediate 
                  dialog by default
10:02 <@delliott> 1. No, 2. Dunno
10:03 <@froodian> 1 isn't a yes or no question ;)
10:03 <@ss|work> Heh
10:03 <@froodian> with the fix pinkerton just approved, not having a dialog would be ok 
                  for users
10:03 <@froodian> since they'd be able to see their new bookmarks
10:03 <@froodian> on the bookmarks bar
10:03 <@ss|work> I still think we should have it by default
10:03 <@froodian> note that whatever we decide here we should also do for dragging 
                  favicons to the bookmarks bar
10:03 <@pinkerton> cmd-shift-option-control-backspace-D?
10:03 <@froodian> (I think)
10:04 <@delliott> froodian: Then wtf does the question mean?
10:04 <@delliott> :p
10:04 <@ardissone|away> then no sheet ;)
10:04 <@delliott> pinkerton: Nice shortcut.
10:04 <@ardissone|away> sheet = intermediate dialogue ;)
10:04 <@froodian> delliott: it has two clauses.  the appropriate answers are "intermediate 
                  dialog" or "non-intermediate"
10:04 <@delliott> Intermediate dialog
10:05 <@froodian> why?
10:05 <@ss|work> What does Fx/Sf/Sm/Ow do?
10:05 <@delliott> You expect to be given the choice to name your bookmark.
10:05 <@peeja> i have to agree with delliott.
10:05 <@ardissone|away> to default to no dialogue, though, we have to fix tab grouping to 
                        be keyboard-based
10:06 <@froodian> ardissone|away: mmm, yeah
10:06 <@Pinolo> uhm
10:06 <@froodian> ok, so i'm cool with leaving it having a dialog by default
10:06 <@froodian> but that'd mean that dragging to the bm bar should too
10:06 <@ardissone|away> ew
10:06 <@froodian> which i know some people don't like
10:06 <@froodian> exactly
10:06 <@ardissone|away> ;)
10:06 <@ss|work> No, I don't think dragging to the bookmark bar should be the same.
10:06 <@froodian> unless we're willing to change the decision you guys made last week
10:06 <@ss|work> It's a different action.
10:06 <@BruceD> prefer no-sheet as the default. suspect tab grouping is quite infrequent
10:06 <@delliott> I like it.
10:06 <@froodian> well, that's what people said when i was gone
10:06 <@ss|work> Dragging and dropping is different.
10:06 <@delliott> A different action to achieve the same goal.
10:07 <@ardissone|away> it's also easier to rename ex post facto on the bar
10:07 <@ss|work> mhm
10:07 <@ardissone|away> you can do so without changing your viuew
10:07 <@froodian> ok, so are we ok with dialog by default on menu item, but no dialog for 
                  d'n'd?
10:07 <@ardissone|away> yes
10:07 <@ss|work> a=me
10:07 <@delliott> I'm not ok with it, but that's neither here nor there.
10:07 <@froodian> (by default.  they'll both be possible)
10:07 <@Pinolo> the idea of changing the shortcut should have us thing very much before 
                making any changes
10:08 <@delliott> :p
10:08 <@froodian> right.  keyboard.
10:08 <@Pinolo> think
10:08 <@ardissone|away> we can revisit the dialog when we get tab grouping hooked up to 
                        the kybd
10:08 <@froodian> so, i propose we keep cmd-d for legacy
10:08 <@ss|work> I concur.
10:08 <@ardissone|away> right
10:08 <@froodian> but move to cmd-k for default keyboard shortcut
10:08 <@ss|work> With both of those statements.
10:08 <@ss|work> pinkerton: ^^
10:08 <@froodian> so that cmd-opt-K can be "bookmark without intermediate dialog"
10:09 <@pinkerton> huh?
10:09 <@pinkerton> cmd-k wha?
10:09 <@ss|work> pinkerton: Are you okay with changing the bookmark keyboard shortcut to 
                 cmd-k (so we can have alternates) but keep cmd-d as legacy?
10:09 <@Pinolo> is there a Camino shortcut map somewhere that we could keep ann eye on to 
                see if all makes sense?
10:09 <@froodian> the problem is that we need more bookmark-related keyboard shortcuts
10:10 <@delliott> Pinolo: Well volunteered!
10:10 <@froodian> but all modifiers concerned with D are taken
10:10 <@pinkerton> hmmm
10:10 <@ardissone|away> Pinolo: cbo/support
10:10 <@ss|work> But cbo is down currently
10:10 <@ardissone|away> :/
10:10 <@pinkerton> i guess we could play with it, but we absolutely have to continue 
                   supporting cmd-d
10:10 <@froodian> yes
10:10 <@ss|work> Of course.
10:10 <@froodian> absolutely
10:10 <@ardissone|away> absolutely
10:10 <@ss|work> Definitely.
10:10 <@delliott> Continue with the prospect of dropping in the future?
10:10 <@ardissone|away> that was never in question
10:11 <@ss|work> We'll probably never drop it.
10:11 <@delliott> Then why add in a new one?
10:11 <@delliott> Seems like overkill.
10:11  * ardissone|away who thought D was a good shortcut for bookmark
10:11 <@ss|work> *sigh*
10:11 <@ss|work> 10:09 <@froodian> the problem is that we need more bookmark-related 
                 keyboard shortcuts
10:11 <@ss|work> 10:10 <@froodian> but all modifiers concerned with D are taken
10:11  * froodian was just copy/pasting that
10:11 <@ss|work> :)
10:11  * ardissone|away was too
10:11 <@delliott> Grumble
10:11 <@ss|work> That looks like it, overall.
10:12 <@froodian> awesome
10:12 <@ss|work> We touched on the other bugs earlier.
10:12 <@peeja> phew.
10:12 <@froodian> thank you guys for sticking it out for my specific bug list :)
10:12 <@ardissone|away> mhm
10:12 <@ss|work> Next week our meeting is on Tuesday
10:12 <@ss|work> pinkerton: ^^
10:13 <@ss|work> Have a good week
10:13 <@ardissone|away> fix lots of bugs ;)
10:13 <@ss|work> And see most of you back here next week
10:13 <@ardissone|away> review lots of patches ;)
10:13 <@ss|work> Do lots of triage
10:13 <@ss|work> Find a new web host
10:13 <@ss|work> Oh wait, that's me.
10:13 <@froodian> eat lots of pie
10:13 <@delliott> Have beer
10:13 <@ardissone|away> finish the build instructions
10:13 <@delliott> ardissone|away: I've been taking screenshots while doing a fresh install.
10:14 <@delliott> Worthwhile?
10:14 <@Pinolo> just a little thing aboyut the topcrasher
10:14 <@ardissone|away> delliott: cool
10:14 <@ardissone|away> Pinolo: what's new?
10:14 <@Pinolo> I've been using the 180branch build and bloglines for a week and it looks 
                robust
10:14 <@ardissone|away> good :)
10:14 <@Pinolo> never a crash
10:14 <@froodian> sweet
10:14 <@ardissone|away> :D
10:14 <@ardissone|away> i'll poke meg in the bug and then wfm it