fluid-work IRC Logs-2011-01-26

[09:24:44 CST(-0600)] <EricDalquist> using the pager how can I control how the page index links are generated?
[09:25:01 CST(-0600)] <EricDalquist> the first link is just "1" but the last is "21 (last)"
[09:25:21 CST(-0600)] <EricDalquist> I either need to get rid of the (last) or add a (first) to the 1 so it looks consistent
[09:28:10 CST(-0600)] <colinclark> Hey EricDalquist
[09:28:15 CST(-0600)] <EricDalquist> hello
[09:28:16 CST(-0600)] <colinclark> I'll have to look it up in the documentation
[09:28:30 CST(-0600)] <colinclark> I haven't ever done it myself, but I'm sure it's possible
[09:28:34 CST(-0600)] <EricDalquist> yeah I was looking through the pager API and didn't see anything that looked obvious
[09:29:45 CST(-0600)] <colinclark> Just looking through the default options for the component, there is a string for last
[09:29:51 CST(-0600)] <colinclark> set, by default, to " (last)"
[09:30:02 CST(-0600)] <EricDalquist> hah, not sure how I missed that
[09:30:20 CST(-0600)] <colinclark> So if you just want to get rid of the " (last)", just override strings.last in your options block
[09:30:27 CST(-0600)] <colinclark> with, say, ""
[09:51:07 CST(-0600)] <heidi_> hey mlam was it hard to make clicking on a paragraph of text (rich text inline edit) open the editor? looks like it used to do that? now one has to click the 'edit' button only
[09:51:30 CST(-0600)] <mlam> No, it wasn't hard at all
[09:51:40 CST(-0600)] <heidi_> howcome it was taken away
[09:51:41 CST(-0600)] <heidi_> ?
[09:52:17 CST(-0600)] <mlam> We had a chat with the designers a few weeks before the release and it was thought that the interaction was more clear by just making the edit link clickable.
[09:52:27 CST(-0600)] <mlam> Do we want to change this back?
[09:52:56 CST(-0600)] <heidi_> mlam just wondering why it changed i guess
[09:53:44 CST(-0600)] <mlam> Oh ok. Yah, it was an interaction decision. Maybe we can bring this topic back up with a couple interaction designers and see if this is still ideal?
[09:54:23 CST(-0600)] <heidi_> mlam yeah i think the consensus is that the whole thing needs revisiting. there are so many contexts of use
[09:54:35 CST(-0600)] <mlam> jhung: what are your thoughts about the edit link being the only clickable element for rich text inline edit?
[09:54:43 CST(-0600)] <mlam> ahh ok
[09:56:28 CST(-0600)] <heidi_> mlam i think if we can make it look like the mockup here: http://issues.fluidproject.org/browse/FLUID-3801
[09:56:43 CST(-0600)] <heidi_> it would be better to have the paragraph clickable, otherwise i think you're right - the edit button is the clearest
[09:57:13 CST(-0600)] <heidi_> but again, not sure if changing this stuff now is worth while, not a huge priority i don't think
[09:57:24 CST(-0600)] <mlam> ok
[09:57:34 CST(-0600)] <heidi_> k thanks for the background info (smile)
[09:57:37 CST(-0600)] <mlam> np (smile)
[09:58:03 CST(-0600)] <heidi_> good to know it's an easy switch if need be
[09:58:56 CST(-0600)] <EricDalquist> colinclark: so I'm not finding the right options: {} block to stick at "strings" option
[10:01:09 CST(-0600)] <colinclark> EricDalquist: Looks to me like it's at top-level in the Pager's options
[10:01:30 CST(-0600)] <colinclark> If you want, paste me your init block for the Pager
[10:02:07 CST(-0600)] <EricDalquist> ok I think I had it wrapped in an options : {} in the main options array
[10:02:12 CST(-0600)] <EricDalquist> I'll move it to the top-level
[10:11:54 CST(-0600)] <colinclark> cool
[10:12:12 CST(-0600)] <EricDalquist> that did it
[10:12:13 CST(-0600)] <EricDalquist> thanks!
[10:13:20 CST(-0600)] <heidi_> hey anybody, what is deadMansBlur ?
[10:22:19 CST(-0600)] <Justin_o> heidi_: I think it is basically a function that will prevent the blur event from firing in certain cases...
[10:41:25 CST(-0600)] <EricDalquist> another pager question, I'm trying to get sortable columns working
[10:42:01 CST(-0600)] <EricDalquist> when I add the <a href="javascript:;" rsf:id="status"></a> wrapper in the <th>
[10:42:11 CST(-0600)] <EricDalquist> the content in the th just disappeara
[10:42:37 CST(-0600)] <EricDalquist> inspecting with firebug it looks like fluid is simply removing that element from the DOM
[13:40:02 CST(-0600)] <michelled> fluid-everyone: who's not in Toronto and would like to be skyped into the dev meeting?
[13:40:26 CST(-0600)] * jessm is skipping – falling behind on writing
[14:07:34 CST(-0600)] <EricDalquist> just posted http://issues.fluidproject.org/browse/FLUID-4045 not sure if there is a better way to do the feature enhancements or not
[14:21:58 CST(-0600)] <colinclark> EricDalquist: That's a good way to file a feature request. Thanks!
[14:22:15 CST(-0600)] <colinclark> And thanks for the code, too!
[14:22:46 CST(-0600)] <EricDalquist> yup, had to write it for something here so I figured it would be good to share
[14:36:17 CST(-0600)] <EricDalquist> if you have a minute … I tried using sortable columns with the pager but when I add the anchor tags in the table header elements it appears that the pager is removing them from the markup
[14:40:04 CST(-0600)] <colinclark> EricDalquist: Ah, good question
[14:40:17 CST(-0600)] <colinclark> Justin_o is in a meeting now, but he can probably help you with that
[14:40:30 CST(-0600)] <EricDalquist> thanks (smile) I'll check back in a bit
[14:57:06 CST(-0600)] <Justin_o> EricDalquist: hello... I hope i can be of help to you.. please let me know when you are back
[14:57:13 CST(-0600)] <EricDalquist> I'm here
[14:57:23 CST(-0600)] <Justin_o> Hi
[14:57:25 CST(-0600)] <EricDalquist> so I have the pager working
[14:57:41 CST(-0600)] <EricDalquist> and I was trying to get sortable columns working
[14:57:54 CST(-0600)] <EricDalquist> but if I change:
[14:57:54 CST(-0600)] <EricDalquist> <thead>
[14:57:54 CST(-0600)] <EricDalquist> <tr>
[14:57:54 CST(-0600)] <EricDalquist> <th class="hrs-pager-head-date">Date</th>
[14:58:26 CST(-0600)] <EricDalquist> to <th class="hrs-pager-head-date"><a href="javascript:;" rsf:id="date">Date</a></th>
[14:58:46 CST(-0600)] <EricDalquist> both the <a> and the Date text are missing from the DOM after the page is finished loading
[14:59:17 CST(-0600)] <Justin_o> okay... do you have a columnDefs object that you pass into the component?
[14:59:30 CST(-0600)] <Justin_o> sorry.. array of objects
[15:02:00 CST(-0600)] <EricDalquist> I did when I was trying to get the sorting working
[15:02:02 CST(-0600)] <EricDalquist> let me find it
[15:02:34 CST(-0600)] <Justin_o> okay
[15:02:49 CST(-0600)] <EricDalquist> http://uportal.pastebin.com/wRAg0r8G
[15:30:19 CST(-0600)] <Justin_o> EricDalquist: in our current demo we use the up/down arrows to indicate the sorting http://build.fluidproject.org/infusion/demos/pager/html/pager.html
[15:30:50 CST(-0600)] <Justin_o> we don't have an arrow on the page by default though... i'll try to see if i can get one of our designers to comment
[15:31:10 CST(-0600)] <Justin_o> EricDalquist: also when the wiki's back up, i'll send you a link to the current CLA
[15:31:16 CST(-0600)] <EricDalquist> ok, I can add it in I was just wondering if there was a best practivce
[15:31:29 CST(-0600)] <EricDalquist> like the pager sets some CSS classes for the current page and disabled links
[15:32:04 CST(-0600)] <EricDalquist> it might be neat to have sorted-asc / sorted-dsc classes that are set on the <th> by the pager
[15:40:04 CST(-0600)] <Justin_o> EricDalquist: they should be in 1.2
[15:40:12 CST(-0600)] <Justin_o> https://source.fluidproject.org/svn/fluid/infusion/tags/infusion-1.2/src/webapp/components/pager/css/Pager.css
[15:41:08 CST(-0600)] <EricDalquist> yeah I found that too … trying to figure out why they're not getting applied
[15:41:32 CST(-0600)] <Justin_o> do you have the correct scoping?
[15:41:47 CST(-0600)] <Justin_o> or nesting i guess
[15:43:59 CST(-0600)] <Justin_o> EricDalquist: here's a link to our licensing page which contains links to the CLA
[15:44:00 CST(-0600)] <Justin_o> http://wiki.fluidproject.org/display/fluid/Fluid+Licensing
[15:44:14 CST(-0600)] <EricDalquist> great I'll get that sent in
[15:45:06 CST(-0600)] <Justin_o> EricDalquist: thanks a lot, really appreciate it
[16:04:38 CST(-0600)] <mlam> colinclark: I'll reassign FLUID-4033 back to me
[16:04:44 CST(-0600)] <colinclark> thanks
[16:04:49 CST(-0600)] <colinclark> I'll put up the patch
[16:06:38 CST(-0600)] <colinclark> mlam: http://issues.fluidproject.org/browse/FLUID-4033
[16:06:41 CST(-0600)] <colinclark> it's the -d patch.
[16:07:12 CST(-0600)] <mlam> ok, cool thanks!