/
fluid-work IRC Logs-2011-05-20

fluid-work IRC Logs-2011-05-20

May 20, 2011

[08:25:30 CDT(-0500)] <heidi> Justin_o we should include ariaTabs for the linearization demo - jquery UI itself isn't aria
[08:25:54 CDT(-0500)] <heidi> Justin_o https://github.com/fnagel/jQuery-Accessible-RIA/tree/master/Tabs
[08:39:39 CDT(-0500)] <Justin_o> heidi: that's too bad..
[08:39:54 CDT(-0500)] <Justin_o> this may be a good opportunity for us to write a wrapper that adds in aria
[08:40:13 CDT(-0500)] <heidi> Justin_o the link i sent is the aria helper... i don't have it working yet
[08:41:45 CDT(-0500)] <Justin_o> heidi: yes.. we might want to be cautious about using 3rd party plugins though.. we've been burned by them in the past
[08:42:24 CDT(-0500)] <heidi> Justin_o i think a lot of felix's work has been brought into jquery ui itself (not sure) - i think it's a reliable source, but i hear ya
[08:43:04 CDT(-0500)] <Justin_o> heidi: okay.. i guess just make sure to check it out first
[08:43:05 CDT(-0500)] <heidi> Justin_o he's also the one that worked on the rich dropdown we were considering using as well
[08:43:16 CDT(-0500)] <Justin_o> heidi: i see, okay
[08:46:45 CDT(-0500)] <heidi> Justin_o i'm still fuzzy on how our wrapper will work, so maybe you can help me out with that in a bit
[08:46:45 CDT(-0500)] <heidi> ?
[08:47:29 CDT(-0500)] <Justin_o> heidi: sure..let me know when you want to chat
[08:47:47 CDT(-0500)] <Justin_o> colinclark: just sent a long winded reply to your e-mail about the graph flipping stuff
[08:47:54 CDT(-0500)] <colinclark> Justin_o: oh, great
[08:48:12 CDT(-0500)] <colinclark> I hope it didn't seem like I was being critical
[08:48:18 CDT(-0500)] <colinclark> you've been a total Git wizard
[08:48:27 CDT(-0500)] <colinclark> I think things were just starting to dawn on me
[08:48:56 CDT(-0500)] <Justin_o> colinclark: no i understand..
[08:49:19 CDT(-0500)] <Justin_o> i actually think that this type of thing should probably only be done for extreme cases anyways.. like what happened last friday..
[08:49:31 CDT(-0500)] <Justin_o> the one that happened the other day we could probably have lived with
[08:50:35 CDT(-0500)] <colinclark> I'll read your email
[08:50:42 CDT(-0500)] <colinclark> and maybe, ultimately, michelled will have some thoughts
[08:51:01 CDT(-0500)] <colinclark> but my gut feeling is that this is us trying to impose stability on a fleeting and illusory concept
[08:51:04 CDT(-0500)] <Justin_o> colinclark: yes.. i'm wondering what michelled will have to say..
[08:51:04 CDT(-0500)] <colinclark> right, fish?
[09:00:38 CDT(-0500)] <michelled> I agree with Justin_o that we should leave minor flips as they are
[09:00:51 CDT(-0500)] <michelled> I think we should try to get into the habit of using the --no-ff and --log flags
[09:01:01 CDT(-0500)] <michelled> and if we have a major flip we should discuss it
[09:01:31 CDT(-0500)] <michelled> I think the black line is valuable as a documentation and communication tool
[09:01:45 CDT(-0500)] <michelled> so keeping it showing the stable points in history would be wonderful
[09:02:04 CDT(-0500)] <michelled> but we should be so strict about it that we cause ourselves a lot of hassle
[09:02:29 CDT(-0500)] <michelled> shouldn't that is
[09:05:42 CDT(-0500)] <Justin_o> michelled: i would agree with all that
[09:09:02 CDT(-0500)] <colinclark> Yeah, I think the point is that we're saying "--no-ff is a standard community coding practice"
[09:09:11 CDT(-0500)] <colinclark> Not unlike, say, JSLint
[09:09:13 CDT(-0500)] <colinclark> or unit tests
[09:09:27 CDT(-0500)] <colinclark> The key difference, I think, is the cost of addressing a failure
[09:09:35 CDT(-0500)] <colinclark> meaning, some idiot like me forgets to include --no-ff
[09:09:44 CDT(-0500)] <colinclark> In the case of JSLint, it's pretty cheap
[09:09:55 CDT(-0500)] <colinclark> Either back out the change or commit on top with the lintification
[09:10:04 CDT(-0500)] <colinclark> In the case of --no-ff, the fix is pretty costly
[09:10:22 CDT(-0500)] <colinclark> As we've said, it's only suitable for really massive situations
[09:10:41 CDT(-0500)] <colinclark> So, that suggests to me that we have to be okay with the graph flipping periodically
[09:10:49 CDT(-0500)] <colinclark> and to some unknown extent
[09:11:03 CDT(-0500)] <colinclark> I don't know if you have thoughts, Justin_o, on how much flippage is too much flippage?
[09:13:10 CDT(-0500)] <Justin_o> colinclark: my initial thoughts would be 1) if puts a branch that we really don't want to appear as the black line up... for example one that is full of experimental commits or odd commit messages or something like that... 2) if the history gets put into a state where it is difficult to discern
[09:13:30 CDT(-0500)] <Justin_o> colinclark: also maybe if the branch extends back more than a few commits/days
[10:33:26 CDT(-0500)] <jessm> standup everyone?
[10:52:36 CDT(-0500)] <heidi> hey, sorry if my standup was funky. lag issues. i'm testing aria tabs plugin/will work on wrapper w/ justin, going to include cindy's UIO control templates into fat panel, and check out anastasia's new themes
[12:08:59 CDT(-0500)] <jessm> Justin_o: ping
[12:19:54 CDT(-0500)] <Justin_o> jessm: sorry.. .i'm here now
[12:21:03 CDT(-0500)] <jessm> no worries!
[12:21:06 CDT(-0500)] <jessm> and nevermind
[12:21:09 CDT(-0500)] <jessm> lol
[12:21:32 CDT(-0500)] <Justin_o> jessm: okay..
[12:37:51 CDT(-0500)] <heidi> cindyli i've added in the divs for the control templates and added the demands blocks for the templates in the demo scripts. were there extra js files i need to include in the header as well?
[12:38:16 CDT(-0500)] <cindyli> cindyli: no
[12:38:49 CDT(-0500)] <cindyli> what the ..., heidi: no
[12:39:11 CDT(-0500)] <heidi> okay. was there more i need to do?
[12:39:13 CDT(-0500)] <cindyli> heidi: problem? why do u ask?
[12:39:35 CDT(-0500)] <heidi> not showing up in my demo yet. i put in the divs... and added the demand blocks with url to templates
[12:39:56 CDT(-0500)] <cindyli> heidi: do u have a branch that i can look at?
[12:40:21 CDT(-0500)] <heidi> cindyli do those two things sound like enough? yep, i can update my branch, but is that right?
[12:40:32 CDT(-0500)] <cindyli> heidi: sounds right
[12:40:47 CDT(-0500)] <cindyli> heidi: 1. js -> add demands block
[12:42:06 CDT(-0500)] <cindyli> 2. modify say "components/uiOptions/html/FullPageNoPreview.html", replace the controls divs with 3 placeholder divs
[12:42:38 CDT(-0500)] <cindyli> that should be it, heidi
[12:45:56 CDT(-0500)] <heidi> cindyli hm, i'm missing something then. i'll push up to github, one sec
[12:47:32 CDT(-0500)] <heidi> cindyli https://github.com/heidiv/infusion/tree/FLUID-4230 – run the fat panel demo in /demos/uiOptions
[12:47:45 CDT(-0500)] <cindyli> ok
[12:57:20 CDT(-0500)] <heidi> thoughts cindyli?
[12:57:58 CDT(-0500)] <cindyli> heidi: same problem. the template looks good though. tracing
[12:58:09 CDT(-0500)] <heidi> thanks!
[12:58:18 CDT(-0500)] <cindyli> np
[13:03:19 CDT(-0500)] <cindyli> heidi: found the issue. if u look at components/uiOptions/js/UIOptions.js, line 323
[13:03:23 CDT(-0500)] <cindyli> var form = fluid.findForm(saveButton);
[13:03:31 CDT(-0500)] <cindyli> saveButton is no longer there
[13:03:45 CDT(-0500)] <heidi> cindyli yeah that error has been there awhile. i'm not sure it's related to this tho?
[13:04:06 CDT(-0500)] <heidi> it's an issue have to fix soon... there's a jira for auto-saving when controls change (no save button)
[13:04:19 CDT(-0500)] <cindyli> yes
[13:04:34 CDT(-0500)] <heidi> but i think this should work despite that error
[13:04:41 CDT(-0500)] <heidi> anything else look wrong?
[13:05:09 CDT(-0500)] <cindyli> in the meantime, if u wanna try out the fat panel, commented out the line where bindHandlers() gets called, heidi
[13:05:28 CDT(-0500)] <heidi> in UIOptions.js ?
[13:07:47 CDT(-0500)] <cindyli> heidi: yes
[13:08:48 CDT(-0500)] <heidi> cindyli that did it - thanks!
[13:09:17 CDT(-0500)]