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 39 Next »

[09:54:31 CST(-0600)] <athena> hey EricDalquist, just wanted to ping you about the layout.json stuff

[09:54:48 CST(-0600)] <athena> managed to close out a couple other things i was working on - is that something i can help out with today?

[09:54:54 CST(-0600)] <EricDalquist> yeah, got caught up in some other tasks yesterday afternoon

[09:55:01 CST(-0600)] <EricDalquist> hrm

[09:55:25 CST(-0600)] <EricDalquist> well you can't really test it yet but do you want add a portlet to the functional test portlets that supports dashboard window state

[09:55:42 CST(-0600)] <athena> sure, i could do that

[09:55:46 CST(-0600)] <EricDalquist> and sets the badge property on dashboard render?

[09:56:02 CST(-0600)] <athena> at a more base level though, is there something i can do to help get things back to working?

[09:56:23 CST(-0600)] <athena> have a couple big demos tomorrow, but think it's probably more useful to help out than to spend the time reverting changes

[09:57:06 CST(-0600)] <EricDalquist> right, well give me 30 minutes and I'll check back in with you on it and have a better idea

[09:57:08 CST(-0600)] <athena> sure, no worries

[09:57:19 CST(-0600)] <athena> and yes, i can totally add that other portlet (smile)

[09:57:49 CST(-0600)] <athena> i'll be gone briefly in like 20 min - have to get the dog a booster shot

[09:57:53 CST(-0600)] <athena> but that should be brief

[09:58:02 CST(-0600)] <EricDalquist> ok

[09:58:19 CST(-0600)] <EricDalquist> yeah the portlet to help test would be great, that is the easiest thing off the top of my head to split out

[10:04:11 CST(-0600)] <athena> sure (smile)

[10:11:11 CST(-0600)] <EricDalquist> so athena, are we good with the plan that unless a portlet supports DASHBOARD we just don't render it

[10:13:03 CST(-0600)] <EricDalquist> hrm, I having second thoughts about all this ...

[10:13:07 CST(-0600)] <athena> lol

[10:13:12 CST(-0600)] <EricDalquist> (tongue)

[10:13:18 CST(-0600)] <EricDalquist> we'll get it fixed today

[10:13:22 CST(-0600)] <athena> lol

[10:13:27 CST(-0600)] <EricDalquist> I just want to think through a few other issues ...

[10:13:32 CST(-0600)] <athena> well i can always revert stuff if we want more time

[10:20:36 CST(-0600)] <EricDalquist> ok ... so Im remember more now

[10:21:00 CST(-0600)] <EricDalquist> we had some logic that initialized portlet windows minimized

[10:21:13 CST(-0600)] <EricDalquist> I'm thinking that is the wrong place for that logic

[10:23:29 CST(-0600)] <athena> ahh, that sounds familliar now

[10:23:33 CST(-0600)] <athena> ok, i have to run out briefly

[10:23:36 CST(-0600)] <EricDalquist> ok

[10:23:38 CST(-0600)] <athena> but i should be back in 20 min or so

[10:52:37 CST(-0600)] <EricDalquist> athena: you back?

[10:54:44 CST(-0600)] <athena> yep

[10:54:49 CST(-0600)] <athena> took longer than it should have

[10:55:00 CST(-0600)] <EricDalquist> so I'm trying to figure out the how to deal with this

[10:55:14 CST(-0600)] <EricDalquist> I'm going to abandon the dashboard stuff for now

[10:55:21 CST(-0600)] <athena> ok

[10:55:25 CST(-0600)] <EricDalquist> and just try and figure out the minimized bit

[10:55:32 CST(-0600)] <athena> ok

[10:56:20 CST(-0600)] <EricDalquist> we have logic in place that is supposed to be minimizing portlet windows on init

[10:56:31 CST(-0600)] <EricDalquist> if the current structure or theme say to

[10:56:39 CST(-0600)] <EricDalquist> I'm going to see if I can figure out why that isn't working

[10:56:45 CST(-0600)] <EricDalquist> but I'm also not happy with that solution

[10:57:08 CST(-0600)] <EricDalquist> I had the thought to do this via a pipeline component

[10:57:19 CST(-0600)] <EricDalquist> when rendering the mobile view or the json view it would minimize all portlets

[10:57:35 CST(-0600)] <EricDalquist> unless it was rendering a single portlet (detached state)

[10:57:39 CST(-0600)] <EricDalquist> does that seem reasonable?

[10:58:03 CST(-0600)] <athena> yes

[10:58:06 CST(-0600)] <athena> at least for a start

[10:58:27 CST(-0600)] <athena> we kind of also need something that can enforce that maximized portlets -> detached when the client is native

[10:58:35 CST(-0600)] <athena> i don't know if that's a related use case at all

[10:58:44 CST(-0600)] <EricDalquist> it could be easily

[10:58:54 CST(-0600)] <EricDalquist> it wouldn't be too hard to have a window state filtering component

[11:04:08 CST(-0600)] <athena> so for today for the base use case, what do we need to do? or should i just go occupy myself with something else for a whiel?

[11:04:29 CST(-0600)] <EricDalquist> something else for now, this should be pretty easy ...

[11:04:54 CST(-0600)] <athena> ok (smile)

[11:05:05 CST(-0600)] <athena> i in fact have plenty of things to do

[11:05:05 CST(-0600)] <athena> lol

[11:05:47 CST(-0600)] <EricDalquist> (smile)

[11:11:27 CST(-0600)] <EricDalquist> so we really have a component that needs to enforce a set of rules

[11:11:46 CST(-0600)] <EricDalquist> man I wish I had time to really dig into this :/

[11:11:55 CST(-0600)] <EricDalquist> for the json view

[11:12:01 CST(-0600)] <EricDalquist> and the mobile browser view

[11:12:21 CST(-0600)] <EricDalquist> everything must render in minimized or detached

[11:12:26 CST(-0600)] <EricDalquist> nothing else is ok

[11:12:36 CST(-0600)] <EricDalquist> for the desktop view, anything goes

[11:29:21 CST(-0600)] <athena> mobile browser uses focused, not detached

[11:29:30 CST(-0600)] <athena> then the native app uses detched, not focused

[11:29:36 CST(-0600)] <athena> just to be extra-helpful-confusing

[11:29:37 CST(-0600)] <EricDalquist> focused == mazimized?

[11:29:40 CST(-0600)] <EricDalquist> er maximized

[11:29:43 CST(-0600)] <athena> er, yeah, that

[11:29:46 CST(-0600)] <EricDalquist> ok

[11:29:48 CST(-0600)] <EricDalquist> (tongue)

[11:30:16 CST(-0600)] <EricDalquist> but they both use DLMMobileColumns

[11:30:26 CST(-0600)] <athena> yeah

[11:42:33 CST(-0600)] <EricDalquist> blarg

[11:42:53 CST(-0600)] <EricDalquist> so much of this is so hacky

[11:43:01 CST(-0600)] <EricDalquist> it will be really good to fix layouts and profiles (sad)

[11:43:51 CST(-0600)] <EricDalquist> like the fact that the json pipeline hard codes the structure descriptor

[11:43:57 CST(-0600)] <EricDalquist> I realize why that is done

[11:44:01 CST(-0600)] <EricDalquist> but it still sucks (tongue)

[11:47:14 CST(-0600)] <athena> yeah (tongue)

[11:47:20 CST(-0600)] <athena> going to be great to have that all refactored

[11:47:24 CST(-0600)] <athena> if there were just more time in the day . . .

[11:47:34 CST(-0600)] <EricDalquist> :/

[11:47:46 CST(-0600)] <EricDalquist> that detached vs maximized thing is a pain

[11:52:25 CST(-0600)] <athena> yeah

[11:52:30 CST(-0600)] <athena> we can ignore that for now if we need to

[11:52:40 CST(-0600)] <athena> it's not triggered often

[11:53:02 CST(-0600)] <athena> we do need to fix it eventually, but we haven't been enforcing it yet, and no one's complained too much

[11:53:08 CST(-0600)] <EricDalquist> if we were doing this right we'd figure out all the nuances of enforcing state/mode restrictions

[11:53:11 CST(-0600)] <EricDalquist> I'd love to do that eventually

[11:53:13 CST(-0600)] <athena> yeah

[11:53:17 CST(-0600)] <athena> that use case isn't a rush (smile)

[11:53:18 CST(-0600)] <EricDalquist> since the spec supports that

[12:44:42 CST(-0600)] <EricDalquist> getting closer athena ....

[12:46:08 CST(-0600)] <athena> woooo

[12:46:21 CST(-0600)] <athena> i'm making some of these portlets actually have desktop views

[12:46:40 CST(-0600)] <athena> just you know, in case someone doesn't actually do all of their computing on their phone (tongue)

[12:46:52 CST(-0600)] <athena> so school gave us loaner droid phones last night

[12:47:05 CST(-0600)] <athena> i walked home with a smartphone in each pocket and it made me super happy

[12:47:11 CST(-0600)] <EricDalquist> lol

[12:48:06 CST(-0600)] <athena> though i uhhhh kinda missed the second half of lecture

[12:48:17 CST(-0600)] <athena> i have no idea what they were thinking handing them out halfway through

[12:48:27 CST(-0600)] <athena> oooh look, new toy to unbox and upgrade and set up accounts on

[12:48:48 CST(-0600)] <athena> oh wait . . . you actually want me to listen to you talk about low-level hardware protocols? errrrr not so much

[12:50:09 CST(-0600)] <EricDalquist> lol

[12:50:23 CST(-0600)] <EricDalquist> man .... its 1 already

[13:04:44 CST(-0600)] <athena> yeah

[13:04:48 CST(-0600)] <athena> day is going by too fast :/

[13:16:22 CST(-0600)] <EricDalquist> oi ... not sure why this didn't click before

[13:16:27 CST(-0600)] <EricDalquist> but when you're using layout.json

[13:16:39 CST(-0600)] <EricDalquist> the detached execution requests actually come through the standard rendering pipeline

[13:16:48 CST(-0600)] <EricDalquist> which makes sense

[13:16:51 CST(-0600)] <EricDalquist> but I didn'

[13:16:55 CST(-0600)] <EricDalquist> 't think about that

[13:21:38 CST(-0600)] <athena> yeah

[13:21:56 CST(-0600)] <athena> we just use layout.json to tell the native app what portlets are in the layout and what their URLs are

[13:22:09 CST(-0600)] <athena> that way it can create a native-based home screen that links to all the right web views

[13:24:06 CST(-0600)] <EricDalquist> grrrr

[13:41:45 CST(-0600)] <athena> not going smoothly?

[13:41:59 CST(-0600)] <EricDalquist> its going

[13:42:01 CST(-0600)] <EricDalquist> and will be fine

[13:42:08 CST(-0600)] <EricDalquist> just being reminded of all the grossness

[13:42:17 CST(-0600)] <EricDalquist> primarily again due to layouts/profiles

[13:59:59 CST(-0600)] <athena> yeah (sad)

[14:00:06 CST(-0600)] <athena> going to be SO nice when we have time to refactor that

[14:00:14 CST(-0600)] <athena> least the layers above it are all nice now (smile)

[14:00:20 CST(-0600)] <EricDalquist> mostly

[14:00:30 CST(-0600)] <EricDalquist> except for the parts where they are gross to deal with the old code (tongue)

[14:15:05 CST(-0600)] <EricDalquist> so I'm not sure how much I can do on enforcing detached vs maximized

[14:15:37 CST(-0600)] <EricDalquist> since both umobile and mobile-browser go through the normal rendering pipeline

[14:15:51 CST(-0600)] <EricDalquist> but I think I have a better fix for the minimization issue

[14:20:18 CST(-0600)] <athena> yeah - and we're actually not sending any information that delineates between detached and minimized right now

[14:20:20 CST(-0600)] <athena> so that's really ok (smile)

[14:40:06 CST(-0600)] <EricDalquist> ok, I thinkI have this fixed

[14:40:11 CST(-0600)] <EricDalquist> though it will pollute the db a little bit

[14:40:17 CST(-0600)] <EricDalquist> I don't think there is much I can do about it right now

[14:40:18 CST(-0600)] <athena> poor db

[14:40:21 CST(-0600)] <athena> (sad)

[14:40:31 CST(-0600)] <EricDalquist> we have the whole 'persistent window state' thing now

[14:40:46 CST(-0600)] <EricDalquist> and so this code results in the window state of minimized getting stored for portlets in the mobile ui

[14:40:49 CST(-0600)] <EricDalquist> which isn't really right

[14:40:51 CST(-0600)] <EricDalquist> hrm

[14:40:59 CST(-0600)] <EricDalquist> I'll try one thing to fix that ...

  • No labels