jasig-ssp IRC Logs-2013-02-20

Owned by Apereo Infrastructure
Last updated: Feb 20, 2013
5 min read

[16:23:38 CST(-0600)] <JasonElwood> Jim- in the report criteria where "Not Used" is listed, would "Ignore Criteria" or "None Selected" be synonymous?

[16:39:18 CST(-0600)] <TonyUnicon1> hey Jason are you there?

[16:40:12 CST(-0600)] <JasonElwood> Yeah, for just a couple more minutes

[16:40:19 CST(-0600)] <TonyUnicon1> ok quick question

[16:40:33 CST(-0600)] <TonyUnicon1> ive come across an issue i need someone to make a call(smile)

[16:40:43 CST(-0600)] <TonyUnicon1> in the MyGPS Task section

[16:40:49 CST(-0600)] <TonyUnicon1> the individual tasks

[16:40:57 CST(-0600)] <TonyUnicon1> the entire task 'div' is a link

[16:41:06 CST(-0600)] <TonyUnicon1> when you click it

[16:41:16 CST(-0600)] <TonyUnicon1> it 'completes' the task

[16:41:21 CST(-0600)] <TonyUnicon1> thats all fine except

[16:41:34 CST(-0600)] <TonyUnicon1> if we want a link in the Task text

[16:41:43 CST(-0600)] <TonyUnicon1> that will link the student to the intake form

[16:41:54 CST(-0600)] <TonyUnicon1> the problem with this is you cant have 'nested' links

[16:42:11 CST(-0600)] <TonyUnicon1> the innter link gets sort of clobbered as a link and you cant get to the form

[16:42:29 CST(-0600)] <TonyUnicon1> so my question is

[16:42:55 CST(-0600)] <TonyUnicon1> can we change the checkbox to be the link to trigger the 'complete' action

[16:43:23 CST(-0600)] <JasonElwood> that makes sense and is totally reasonable

[16:43:30 CST(-0600)] <TonyUnicon1> ok thats the easy one

[16:43:33 CST(-0600)] <TonyUnicon1> the other issue is

[16:43:48 CST(-0600)] <TonyUnicon1> ive basically done that

[16:43:48 CST(-0600)] <JasonElwood> you said quick question, not questions

[16:43:52 CST(-0600)] <TonyUnicon1> (sad)

[16:43:53 CST(-0600)] <TonyUnicon1> one more

[16:43:55 CST(-0600)] <TonyUnicon1> quick quick

[16:44:04 CST(-0600)] <TonyUnicon1> the next question is

[16:44:08 CST(-0600)] <TonyUnicon1> there is a listener

[16:44:15 CST(-0600)] <TonyUnicon1> that when I hover the task

[16:44:29 CST(-0600)] <TonyUnicon1> it reveals the 'Details' button

[16:44:32 CST(-0600)] <JasonElwood> just messing

[16:44:41 CST(-0600)] <TonyUnicon1> this has the same affect

[16:44:52 CST(-0600)] <TonyUnicon1> it blocks the linking text

[16:45:14 CST(-0600)] <TonyUnicon1> but the task detail doesn't really show anything special

[16:45:25 CST(-0600)] <TonyUnicon1> its a little dialog box that repeats the task name

[16:45:44 CST(-0600)] <JasonElwood> it never shows any more detail?

[16:45:51 CST(-0600)] <TonyUnicon1> not as it is now

[16:46:00 CST(-0600)] <TonyUnicon1> there are two ways we can address this

[16:46:08 CST(-0600)] <TonyUnicon1> we can always show the details button for each task

[16:46:19 CST(-0600)] <TonyUnicon1> and not listen for the hover

[16:46:51 CST(-0600)] <TonyUnicon1> or we can ditch the button all together and just make sure we show the entire task description (right now the size is absolute so it gets cut off)

[16:47:10 CST(-0600)] <TonyUnicon1> does this make sense?

[16:47:24 CST(-0600)] <JasonElwood> it does

[16:47:58 CST(-0600)] <TonyUnicon1> this has to do with jquery mobile magic

[16:48:04 CST(-0600)] <JasonElwood> so it's absolute now for conformity

[16:48:07 CST(-0600)] <TonyUnicon1> this would be an issue issue to solve without it

[16:48:11 CST(-0600)] <TonyUnicon1> yes

[16:48:19 CST(-0600)] <TonyUnicon1> easier issue*

[16:48:51 CST(-0600)] <TonyUnicon1> we could in theory

[16:49:00 CST(-0600)] <TonyUnicon1> allow them to link

[16:49:11 CST(-0600)] <TonyUnicon1> by clicking on the 'more details' button

[16:49:16 CST(-0600)] <TonyUnicon1> then the link should be hot

[16:49:20 CST(-0600)] <TonyUnicon1> but thats annoying IMO

[16:50:27 CST(-0600)] <JasonElwood> so what's least annoying is the question: multiple details buttons, disparate space for task descriptions or clicking into details to click the intake link

[16:50:41 CST(-0600)] <TonyUnicon1> yes

[16:51:07 CST(-0600)] <TonyUnicon1> orrr

[16:51:17 CST(-0600)] <TonyUnicon1> actually no or

[16:51:48 CST(-0600)] <TonyUnicon1> there are more elegant solutions but its a potential quagmire

[16:51:52 CST(-0600)] <JasonElwood> unless we are going to have links inside tasks (which doesn't exist already), this would be the only case. Since it only happens once per student and not all students, clicking into details to click the link should be the least annoying from an occurrence standpoint

[16:52:06 CST(-0600)] <TonyUnicon1> There are other tasks with links

[16:52:32 CST(-0600)] <TonyUnicon1> otherwise i'd use your reasoning

[16:52:35 CST(-0600)] <TonyUnicon1> as well

[16:52:48 CST(-0600)] <JasonElwood> but the other tasks with links don't work now either right?

[16:52:58 CST(-0600)] <TonyUnicon1> they are hard coded sinclair links

[16:53:05 CST(-0600)] <TonyUnicon1> im assuming punted in as test data

[16:53:55 CST(-0600)] <TonyUnicon1> its unfortunate that this is an issue at all

[16:53:55 CST(-0600)] <JasonElwood> so right now they have to hover/click details to make the link active???

[16:54:08 CST(-0600)] <TonyUnicon1> right now you cant click the link at all

[16:54:20 CST(-0600)] <JasonElwood> even better

[16:54:22 CST(-0600)] <TonyUnicon1> the current state of the code

[16:54:32 CST(-0600)] <TonyUnicon1> before I even started it was this way

[16:54:56 CST(-0600)] <TonyUnicon1> the thing is I probably can refactor the task html

[16:55:11 CST(-0600)] <TonyUnicon1> but with my unfamiliarity of jquery mobile and css

[16:55:16 CST(-0600)] <TonyUnicon1> its an expensive task for me

[16:55:18 CST(-0600)] <JasonElwood> I think I would still go with click to click instead of always showing the details button

[16:55:25 CST(-0600)] <TonyUnicon1> ok

[16:55:41 CST(-0600)] <TonyUnicon1> I will create a ticket for the improvement though

[16:55:49 CST(-0600)] <TonyUnicon1> and maybe I can get to it later

[16:55:52 CST(-0600)] <JasonElwood> a details button would still just perform the hover action right

[16:56:09 CST(-0600)] <TonyUnicon1> yeah I think what you're suggesting is...

[16:56:16 CST(-0600)] <TonyUnicon1> user hovers over task

[16:56:22 CST(-0600)] <TonyUnicon1> button is revealed

[16:56:24 CST(-0600)] <TonyUnicon1> student clicks on detail button

[16:56:31 CST(-0600)] <TonyUnicon1> then user clicks on link

[16:57:04 CST(-0600)] <TonyUnicon1> the link will also be on the email as well

[16:57:32 CST(-0600)] <JasonElwood> the alternative would be user sees details button and the link. If the link is in the first view, they can click it

[16:57:47 CST(-0600)] <JasonElwood> but the details button appears on every task

[16:58:04 CST(-0600)] <TonyUnicon1> yes that is the ideal but it would require some refactoring to get that to work

[16:58:30 CST(-0600)] <TonyUnicon1> which is what I mean by an elegant solution would be more expensive

[16:58:42 CST(-0600)] <TonyUnicon1> the problem is

[16:58:46 CST(-0600)] <TonyUnicon1> jquery mobile by default

[16:58:55 CST(-0600)] <JasonElwood> right

[16:58:56 CST(-0600)] <TonyUnicon1> turns list items into 'buttons'

[16:59:05 CST(-0600)] <TonyUnicon1> essentially clobbering all links inside the list item

[16:59:08 CST(-0600)] <JasonElwood> so you are going with the click to click?

[16:59:19 CST(-0600)] <TonyUnicon1> i think thats the best

[16:59:24 CST(-0600)] <JasonElwood> works for me

[16:59:29 CST(-0600)] <TonyUnicon1> if you think repeating detail buttons are too cluttered

[16:59:49 CST(-0600)] <TonyUnicon1> in anycase, jquery mobile sucks, thanks for your help

[17:00:09 CST(-0600)] <JasonElwood> I would have to see it. I would be fine either way I think. If you think details buttons isn't too cluttered and saves us time, go for it.

[17:00:40 CST(-0600)] <TonyUnicon1> ok, i can explain the problem better during the demo

[17:00:47 CST(-0600)] <TonyUnicon1> we can decide to change it after that

[17:01:00 CST(-0600)] <TonyUnicon1> if you guys see it and change your mind

[17:01:00 CST(-0600)] <TonyUnicon1> its not expensive to go with click click

[17:01:24 CST(-0600)] <JasonElwood> if you want to show me before the demo, that's fine. just let me know when you have something

[17:01:27 CST(-0600)] <TonyUnicon1> so not much wasted effort if we want to go in another direction

[17:02:00 CST(-0600)] <JasonElwood> gotta leave for an appt. thanks for including me