Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 29 Next »

[07:46:27 EDT(-0400)] * michelled (n=michelle@CPE001310472ade-CM0011aefd3ca8.cpe.net.cable.rogers.com) has joined ##uportal
[08:28:39 EDT(-0400)] * dstn_ (n=dstn@unaffiliated/dstn) has left ##uportal
[09:28:06 EDT(-0400)] * anastasiac (n=team@142.150.154.160) has joined ##uportal
[09:41:13 EDT(-0400)] * colinclark (n=colin@bas1-toronto09-1279543344.dsl.bell.ca) has joined ##uportal
[10:05:04 EDT(-0400)] * colinclark (n=colin@bas1-toronto09-1279543344.dsl.bell.ca) has joined ##uportal
[10:12:14 EDT(-0400)] * apetro-_ (n=apetro@CPE-65-30-162-97.wi.res.rr.com) has joined ##uportal
[10:42:14 EDT(-0400)] * EricDalquist (n=dalquist@bohemia.doit.wisc.edu) has joined ##uportal
[11:02:30 EDT(-0400)] <athena7> so, uci announcements portlet -> jasig repository?
[11:02:36 EDT(-0400)] <EricDalquist> yeah
[11:02:42 EDT(-0400)] <EricDalquist> working with Erik to make that happen rightnow
[11:02:46 EDT(-0400)] <athena7> awesome!
[11:02:50 EDT(-0400)] <athena7> that's great (smile)
[11:03:00 EDT(-0400)] <EricDalquist> yeah
[11:03:02 EDT(-0400)] <athena7> it sounds like he's done some really great work - on my plate to try it out today
[11:03:06 EDT(-0400)] <EricDalquist> figured we should finally standardize on one
[11:03:09 EDT(-0400)] <athena7> yes
[11:03:11 EDT(-0400)] <athena7> so if we do
[11:03:40 EDT(-0400)] <athena7> we may want to consider that the announcements and toro notifications portlet currently have different, non-overlapping features
[11:03:57 EDT(-0400)] <EricDalquist> yup
[11:04:08 EDT(-0400)] <EricDalquist> and work will need to be done to resolve that
[11:04:13 EDT(-0400)] <athena7> well
[11:04:27 EDT(-0400)] <athena7> a few of us had discussed the feasability of one portlet to rule them all
[11:04:36 EDT(-0400)] <athena7> since the notifications portlet isn't spring mvc at the moment either
[11:04:52 EDT(-0400)] <athena7> i know they do different things, but it seems to me like you might plausibly want both features in one portlet
[11:09:33 EDT(-0400)] * michelled (n=michelle@CPE001310472ade-CM0011aefd3ca8.cpe.net.cable.rogers.com) has joined ##uportal
[11:19:22 EDT(-0400)] * dstn (n=dstn@unaffiliated/dstn) has joined ##uportal
[11:19:39 EDT(-0400)] <dstn> athena7: do you know of any schools that are currently using the Feedback Portlet?
[11:22:50 EDT(-0400)] <athena7> no (smile)
[11:23:13 EDT(-0400)] <EricDalquist> it is on our list
[11:23:17 EDT(-0400)] <EricDalquist> post 3 upgrade
[11:23:20 EDT(-0400)] <EricDalquist> along with weather
[11:23:22 EDT(-0400)] <EricDalquist> announcements
[11:23:51 EDT(-0400)] <athena7> it's in the unicon demo
[11:28:20 EDT(-0400)] <athena7> hm, the announcements portlet seems to include the portlet servlets in its web.xml
[11:32:55 EDT(-0400)] <athena7> so dstn, about the weather portlet css
[11:33:05 EDT(-0400)] <athena7> did i miss conversation after i left?
[11:48:30 EDT(-0400)] * holdorph (n=holdorph@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[12:04:11 EDT(-0400)] * michelled_ (n=michelle@CPE001310472ade-CM0011aefd3ca8.cpe.net.cable.rogers.com) has joined ##uportal
[12:28:19 EDT(-0400)] <dstn> athena7: We talked about it. Eric mentioned the preference to have the <link>'s by default for ease of configuration but I prefer valid HTML so Eric/Holdorph suggested it be a preference option in the portlet.xml <init-param>.
[12:29:02 EDT(-0400)] <athena7> yes, that would actually be my suggestion as well
[12:29:09 EDT(-0400)] <athena7> i have a strong preference for making things work out of the box
[12:29:29 EDT(-0400)] <athena7> but having a preference option that you can toggle to get well-formed html instead sounds like a good solution
[12:29:56 EDT(-0400)] <dstn> I got looking at the <init-param> elements and defineObjects for use of the init-params in the JSP and it wasn't going to be a clean way in the JSP, I.E. use of <% %> ... and I stopped there.
[12:30:06 EDT(-0400)] <athena7> well
[12:30:27 EDT(-0400)] <athena7> we could always pull it out of the init param in the controller and drop it into the model
[12:30:59 EDT(-0400)] <holdorph> you should be able to use expression language and avoid the <% %>
[12:31:03 EDT(-0400)] <dstn> Eric suggested and interceptor and I like that idea, feels more clean
[12:32:19 EDT(-0400)] <EricDalquist> holdorph: I don't know of a way to call arbitrary methods in JSTL-EL
[12:32:24 EDT(-0400)] <dstn> holdorph: I couldn't figure out the EL way of doing it?
[12:32:39 EDT(-0400)] <EricDalquist> and PortletConfig doesn't have a nice JavaBean way to access the init params
[12:32:47 EDT(-0400)] <EricDalquist> just a getInitParameter(String) method
[12:37:20 EDT(-0400)] <holdorph> bummer. if only they had made getInitParameterNames() return a Map of the parameters instead, then it could be done.
[12:37:29 EDT(-0400)] <EricDalquist> yeah
[12:37:54 EDT(-0400)] <EricDalquist> so the other idea was to add an interceptor to the controllers
[12:38:08 EDT(-0400)] <EricDalquist> I think those can modify the model before it goes to the view
[12:38:23 EDT(-0400)] <holdorph> interceptor or putting the logic right in the controller, probably not a huge difference.
[12:38:55 EDT(-0400)] <holdorph> probably less work to put it into the controller, if the current controller is already adding items to the model.
[12:39:33 EDT(-0400)] <EricDalquist> well if there are multiple controllers ...
[12:39:43 EDT(-0400)] <EricDalquist> http://static.springframework.org/spring/docs/2.5.x/api/org/springframework/web/portlet/HandlerInterceptor.html#postHandleRender(javax.portlet.RenderRequest,%20javax.portlet.RenderResponse,%20java.lang.Object,%20org.springframework.web.portlet.ModelAndView)
[12:40:06 EDT(-0400)] <holdorph> true, i don't know much about the current portlet. if there are multiple controllers an interceptor would be easier
[12:40:23 EDT(-0400)] <holdorph> but if you only had one controller, I think putting it into the one, would be easier
[12:52:45 EDT(-0400)] <dstn> Ya, there are multiple controllers where the settings would apply
[12:52:55 EDT(-0400)] <dstn> edit/view etc
[13:03:41 EDT(-0400)] * awills (n=awills@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[13:10:22 EDT(-0400)] * athena7 (n=athena7@99.145.99.32) has joined ##uportal
[13:58:17 EDT(-0400)] <dstn> yay! interceptor works
[14:02:39 EDT(-0400)] * EricDalquist (n=dalquist@bohemia.doit.wisc.edu) has joined ##uportal
[14:03:43 EDT(-0400)] * EricDalquist (n=dalquist@bohemia.doit.wisc.edu) has joined ##uportal
[14:03:54 EDT(-0400)] <athena7> w00t
[14:04:20 EDT(-0400)] <athena7> can we have the CSS on by default?
[14:05:00 EDT(-0400)] <dstn> I'm going to get out voted if I say no so sure
[14:05:06 EDT(-0400)] <dstn> (wink)
[14:05:45 EDT(-0400)] <dstn> as for jquery, should that be enabled by default?
[14:06:17 EDT(-0400)] <athena7> lol
[14:06:32 EDT(-0400)] <athena7> well, i do strongly think that portlets should work out of the box, and this won't if the css is omitted
[14:06:39 EDT(-0400)] <athena7> not sure for jquery
[14:06:49 EDT(-0400)] <athena7> probably disabled by default, i guess
[14:07:06 EDT(-0400)] <athena7> although that's less up2.6 friendly
[14:07:20 EDT(-0400)] <EricDalquist> jquery still not happy about getting loaded more than once?
[14:07:55 EDT(-0400)] <athena7> i think that's the case
[14:08:11 EDT(-0400)] <athena7> also, eventually we're going to run into issues with having different versions of jquery
[14:08:18 EDT(-0400)] <athena7> and i'm really not sure what'll happen then
[14:08:31 EDT(-0400)] <EricDalquist> we'll wish we were writing applets (tongue)
[14:08:47 EDT(-0400)] <EricDalquist> or that JS had better namespaceing built in
[14:09:00 EDT(-0400)] <dstn> so what happens when you have multiple <links> on a page? does css get loaded twice?
[14:09:14 EDT(-0400)] <dstn> multiple of the same <link>'s that is
[14:09:28 EDT(-0400)] * dstn needs to leave in 5 min
[14:09:51 EDT(-0400)] <EricDalquist> don't know
[14:10:08 EDT(-0400)] <dstn> I'm recalling now that was part of my other concern from leaving it out
[14:18:58 EDT(-0400)] <athena7> it may depend on the browser
[14:19:18 EDT(-0400)] <athena7> i'd recommend leaving it in and documenting clearly how and why one might want to disable the import
[14:19:31 EDT(-0400)] <athena7> a lot of people may not want the extra css imported on every single portal view, too
[14:21:51 EDT(-0400)] * michelled (n=michelle@CPE001310472ade-CM0011aefd3ca8.cpe.net.cable.rogers.com) has joined ##uportal
[14:23:33 EDT(-0400)] <dstn> anyone know if 268 supports additional files like js or css per view? that'd be nice
[14:23:45 EDT(-0400)] <holdorph> 286
[14:23:51 EDT(-0400)] <EricDalquist> yes
[14:24:07 EDT(-0400)] <holdorph> what do you mean, 'per view' ?

  • No labels