Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

[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