| * th1a is going to make dinner. | 00:02 | |
| *** menesis has quit IRC | 00:34 | |
| th1a | How we doing aelkner? | 01:08 |
|---|---|---|
| aelkner | it will be a while before i have anything to show you | 01:23 |
| aelkner | lots of classes to make/register | 01:24 |
| th1a | Moving things around? | 01:24 |
| aelkner | yes | 01:24 |
| th1a | OK. We won't be messing this extensively with most views. | 01:24 |
| aelkner | then there's the matter of changing the classes to not make the tree | 01:24 |
| aelkner | actually, wait | 01:24 |
| aelkner | i think the tree could exist within the change to the use of the heading, but i haven't gotten that far yet | 01:25 |
| aelkner | but how wold you characterize the change you want to the viewlets | 01:26 |
| aelkner | you say you want an html table there? | 01:26 |
| aelkner | so that the css you guys worked on kicked in nicely? | 01:26 |
| *** alga has quit IRC | 01:31 | |
| aelkner | th1a, the sections and groups viewlets use jstree | 01:33 |
| aelkner | the advisors is just a <ul> at the moment | 01:34 |
| aelkner | the demographics viewlet should have a display form in it | 01:34 |
| aelkner | replaceafill, ayt? | 01:39 |
| replaceafill | aelkner yes | 01:40 |
| aelkner | so for the icon, you realize that it is rendered via a background image | 01:41 |
| aelkner | i got rid of two accordion background images | 01:41 |
| aelkner | the header and the content | 01:42 |
| aelkner | but getting rid of the icon will only get rid of the icon :) | 01:42 |
| aelkner | that, btw, is why we needed to shove the text to the right | 01:42 |
| aelkner | because the background image is not a content element | 01:43 |
| replaceafill | th1a wants the icon white, correct? | 01:44 |
| aelkner | well, actually, it is a content element, just an empty span | 01:44 |
| replaceafill | yep | 01:44 |
| aelkner | yes | 01:44 |
| replaceafill | they apply the background-image to that span | 01:44 |
| replaceafill | do you want to do that right now? | 01:45 |
| replaceafill | i mean, i think i know how to solve it | 01:45 |
| aelkner | do tell | 01:45 |
| replaceafill | 1. we simple edit the icon set image and put it as our own resource file | 01:46 |
| replaceafill | or | 01:46 |
| replaceafill | 2. we build a custom jquery ui theme with white icons | 01:46 |
| replaceafill | and use that image | 01:46 |
| replaceafill | either way is easy | 01:46 |
| aelkner | not rocket science, but tedious | 01:47 |
| replaceafill | not a big deal | 01:47 |
| aelkner | i was hoping you were gong to suggest some unicode that would work as a text icon | 01:48 |
| replaceafill | nope | 01:48 |
| aelkner | btw, do you know what th1a wants changed in the viewlets? | 01:48 |
| aelkner | person info, that is | 01:49 |
| replaceafill | yes, i read the log | 01:49 |
| aelkner | and? | 01:50 |
| replaceafill | aelkner http://img829.imageshack.us/img829/7650/uiicons0c135f256x240.png | 01:51 |
| replaceafill | blue icons ;) | 01:52 |
| replaceafill | aelkner and...? | 01:52 |
| aelkner | you said you read the log, so what did you find? | 01:52 |
| aelkner | i was asking what you thought th1a wanted changed | 01:52 |
| replaceafill | that we're changing the viewlets | 01:53 |
| aelkner | in what manner was the question | 01:53 |
| aelkner | they have jstrees (some) and the advisory is just two <ul>s | 01:53 |
| replaceafill | you have to make them look like tables :) | 01:53 |
| aelkner | that's why i ask, because that can't be done | 01:54 |
| replaceafill | uh? | 01:54 |
| replaceafill | why not? | 01:54 |
| aelkner | perhaps the advisory can | 01:54 |
| aelkner | but a tree can not be represented with a table | 01:55 |
| replaceafill | why not? | 01:55 |
| aelkner | that should be obvious, are you looking at one? | 01:56 |
| replaceafill | nope | 01:56 |
| replaceafill | want me to? | 01:56 |
| aelkner | well, if you want to see my point, then yes | 01:57 |
| replaceafill | ok | 01:57 |
| replaceafill | ok, i have one | 01:58 |
| aelkner | sections? | 01:58 |
| replaceafill | yes | 01:58 |
| aelkner | so how wold that look like a table? | 01:58 |
| aelkner | tables are square | 01:58 |
| aelkner | this is heirarchical | 01:59 |
| th1a | It will just be longer. | 01:59 |
| th1a | And have columns. | 02:00 |
| replaceafill | didnt you do something similar for sections | 02:00 |
| replaceafill | one column, school year | 02:00 |
| replaceafill | next column term | 02:00 |
| th1a | Hopefully it will be tolerable since the whole thing can be collapsed anyhow. | 02:00 |
| replaceafill | and maybe section links | 02:00 |
| aelkner | i haven't made anything like that yet, but the Sections view is already a table | 02:01 |
| aelkner | though i believe it's just course, section as the columns | 02:01 |
| aelkner | but if that's whatyour asking for, th1a, i could do the year/term/section columns | 02:01 |
| aelkner | with the section still being a link | 02:02 |
| th1a | Yes. | 02:02 |
| th1a | The alternative would be an embedded accordion... | 02:02 |
| replaceafill | :S | 02:03 |
| th1a | With each term getting a section. | 02:03 |
| replaceafill | accordion inside accordion? | 02:03 |
| th1a | Chrome plated chrome! | 02:03 |
| th1a | Or the tree. | 02:03 |
| th1a | I just think the tree is kind of jarring in there. | 02:03 |
| replaceafill | i'd vote for 3 column table | 02:04 |
| aelkner | one level of accordions is enough i think | 02:04 |
| th1a | Once people have five years worth of data in here we can embed an accordion. | 02:04 |
| th1a | We just want to reverse sort by year by default. | 02:04 |
| th1a | So the current year is at top. | 02:04 |
| aelkner | ok | 02:05 |
| replaceafill | isn't it like that already? | 02:05 |
| replaceafill | i mean, the jstree? | 02:05 |
| th1a | Yes. | 02:05 |
| replaceafill | ah ok | 02:05 |
| th1a | So... Edit advisors -- button or link? | 02:06 |
| aelkner | so i used the color for table headings for the accordion headings | 02:06 |
| aelkner | wait, follow this question | 02:06 |
| th1a | There is a sub-table heading. | 02:06 |
| th1a | table sub-section. | 02:06 |
| aelkner | so the table in the viewlet itself will have sub-heading styles for its headings, right? | 02:06 |
| th1a | I'm saying yes. | 02:06 |
| th1a | At least, unless it looks stupid. | 02:07 |
| aelkner | well, there are no subheadings in the tables themselves, so i think we are safer for that | 02:07 |
| th1a | I'm not coming up with a new shade of gray. | 02:07 |
| aelkner | :) | 02:08 |
| aelkner | btw, the grey that i have for the viewlet background is wrong, isn't it? | 02:08 |
| aelkner | i need it to be white like the table guidelines i guess | 02:09 |
| aelkner | oops, that's not white | 02:09 |
| aelkner | it's just a very lite grey | 02:09 |
| aelkner | never mind | 02:09 |
| replaceafill | aelkner you should use background: none instead of background-image: none | 02:11 |
| replaceafill | remember cascading | 02:12 |
| aelkner | because they supplied the color, right? | 02:12 |
| aelkner | will do, thanks | 02:12 |
| aelkner | th1a, i'm not sure what to do with Sections Taught and Sections Attended | 02:14 |
| aelkner | right below them will be the headings for each table | 02:14 |
| aelkner | School Year Term Section | 02:15 |
| th1a | Oh, I see. | 02:15 |
| th1a | Hm. | 02:15 |
| aelkner | and that will have subheading color | 02:15 |
| th1a | We might need more colors... | 02:15 |
| aelkner | we're going to need a bigger boat :) | 02:15 |
| th1a | OK, let's try making the active accordion the dark red used in the header and the inactive ones the lighter red in the header. | 02:16 |
| th1a | Lighter = #AA0000 | 02:16 |
| th1a | Darker = #5400000 | 02:16 |
| aelkner | ok, let me try that out first | 02:17 |
| th1a | How does that look? | 02:25 |
| aelkner | replaceafill, they have a class for ui-state-active, but nothing for inactive | 02:26 |
| replaceafill | and? | 02:26 |
| th1a | They're different in the demo, right? | 02:26 |
| replaceafill | yes | 02:27 |
| replaceafill | aelkner assing the inactive color to all of them | 02:27 |
| replaceafill | and just change the active | 02:27 |
| aelkner | the active class comes after the other classes, so the other ones win | 02:30 |
| replaceafill | uh? | 02:30 |
| replaceafill | push | 02:30 |
| aelkner | pushed | 02:31 |
| aelkner | when you firebug a heading, you'll see that the active class is at the end | 02:33 |
| aelkner | if you take the background away from the ui-accordion-header class, you'll see | 02:33 |
| aelkner | the color for the active tab is different while the inactive ones will have no color of course | 02:33 |
| *** hoffman has joined #schooltool | 02:33 | |
| replaceafill | aelkner just move .ui-state-active below .ui-accordion-header in the css file | 02:34 |
| aelkner | oh :) | 02:35 |
| aelkner | so it's the order in the css, not the element | 02:35 |
| replaceafill | :/ | 02:35 |
| replaceafill | correct | 02:36 |
| aelkner | that's why the order of the files is important, right | 02:36 |
| replaceafill | and remove that h5.xxx | 02:36 |
| replaceafill | not only of files, rules inside a single file too | 02:36 |
| aelkner | well, that one we just said | 02:36 |
| replaceafill | i'd revert those colors... | 02:37 |
| replaceafill | make the active darker | 02:37 |
| replaceafill | like the primary navigation | 02:37 |
| aelkner | reverse, you mean, right | 02:37 |
| replaceafill | yes | 02:37 |
| replaceafill | sorry | 02:37 |
| * replaceafill has had a headache all day :( | 02:38 | |
| replaceafill | lack of coke ;) | 02:38 |
| aelkner | revert is on the tip of your tongue as a user of revision control | 02:38 |
| aelkner | i reversed the colors, fixed the css and pushed | 02:41 |
| aelkner | perhaps we should bring it up on the instance | 02:41 |
| aelkner | see what th1a thinks | 02:41 |
| hoffman | I've switched computers so that would help. | 02:41 |
| replaceafill | ok, a sec | 02:42 |
| replaceafill | done | 02:43 |
| hoffman | I think that looks ok for now at least. | 02:45 |
| hoffman | It doesn't look weird. | 02:45 |
| hoffman | I think that orange halo is still there. | 02:45 |
| replaceafill | yes | 02:46 |
| replaceafill | aelkner remove the border in the active class | 02:47 |
| hoffman | The links should be the lighter red there too -- that is, they're currently lighter than the lighter red in the top navigation. | 02:47 |
| hoffman | Or make it dark red. | 02:47 |
| aelkner | replaceafill, border: none? | 02:49 |
| replaceafill | yep | 02:49 |
| aelkner | i did hat, but i don't notice any difference | 02:49 |
| aelkner | then again, i don't even know what halo you're talking about | 02:49 |
| aelkner | did you try it ourself, and it worked? | 02:50 |
| replaceafill | :/ no | 02:51 |
| replaceafill | want me to? | 02:52 |
| * hoffman resists making a sarcastic joke about aelkner's vision. | 02:52 | |
| replaceafill | :D | 02:52 |
| replaceafill | hoffman you dont see it anymore, correct? | 02:52 |
| hoffman | Did you push it? | 02:53 |
| * replaceafill is resisting making jokes on aelkner :D | 02:53 | |
| replaceafill | hoffman i changed it in the testing instance.... | 02:53 |
| replaceafill | (here we go!) | 02:53 |
| hoffman | It is gone. | 02:53 |
| replaceafill | aelkner see? border: none; | 02:53 |
| replaceafill | .ui-state-active { | 02:53 |
| replaceafill | background-color: #540000; | 02:53 |
| replaceafill | border: none; | 02:53 |
| replaceafill | } | 02:53 |
| aelkner | that's what i did | 02:54 |
| hoffman | For the table inside the accordion we can mostly use the standard table css, right? | 02:54 |
| aelkner | now we are freed up to do that | 02:55 |
| hoffman | It will just be a bit narrower I guess. | 02:58 |
| *** hoffman has quit IRC | 03:27 | |
| *** aks has joined #schooltool | 06:24 | |
| *** th1a has quit IRC | 07:39 | |
| *** menesis has joined #schooltool | 09:08 | |
| *** replaceafill has quit IRC | 09:11 | |
| *** menesis has quit IRC | 09:11 | |
| *** menesis has joined #schooltool | 09:13 | |
| *** ignas has joined #schooltool | 09:33 | |
| *** alga has joined #schooltool | 11:16 | |
| *** menesis has quit IRC | 12:00 | |
| *** menesis has joined #schooltool | 13:06 | |
| *** aks has quit IRC | 14:23 | |
| *** dadeng has joined #schooltool | 14:32 | |
| *** dadeng has quit IRC | 15:17 | |
| *** dadeng has joined #schooltool | 15:41 | |
| dadeng | aelkner, hi | 15:51 |
| *** th1a has joined #schooltool | 16:08 | |
| *** replaceafill has joined #schooltool | 16:28 | |
| th1a | hi replaceafill, aelkner, yvl, menesis. | 16:31 |
| aelkner | morning | 16:32 |
| replaceafill | good morning/afternoon | 16:32 |
| menesis | hi | 16:32 |
| th1a | OK, so do we have a fresh flourish branch? | 16:33 |
| th1a | We need yvl... | 16:33 |
| yvl | morning guys! | 16:34 |
| th1a | Ah. | 16:34 |
| yvl | yes we do | 16:34 |
| yvl | I mean - we do have a fresh branch | 16:34 |
| th1a | We need you too. ;-) | 16:34 |
| yvl | :D | 16:34 |
| th1a | yvl: Does this include some changes from you? | 16:35 |
| yvl | sorry, no | 16:36 |
| yvl | it's Alan's and Douglas' changes | 16:36 |
| dadeng | aelkner hi | 16:36 |
| yvl | + fixed FlourishInfo registration | 16:36 |
| yvl | btw, I made the links red yesterday | 16:37 |
| yvl | I doubt anybody noticed | 16:37 |
| th1a | dadeng: We're having a meeting now. | 16:37 |
| yvl | they look very close to orange | 16:37 |
| replaceafill | yvl i did :) | 16:37 |
| th1a | They're too bright. | 16:37 |
| dadeng | yeah, sorry, i did not look well b4 | 16:37 |
| dadeng | later pls | 16:37 |
| th1a | OK. | 16:37 |
| th1a | We don't actually want any orange in the UI. | 16:38 |
| yvl | agreed | 16:38 |
| yvl | I just took the colour from UWG and switched the hue from orange to red | 16:39 |
| th1a | Yeah, that's too light. | 16:39 |
| th1a | Just work off the colors in the top navigation. | 16:39 |
| th1a | OK, so... aelkner, do you have the updated flourish branch? | 16:41 |
| aelkner | yes | 16:41 |
| th1a | I'm a little confused about where we are as far as getting the pending work in the branch. | 16:41 |
| aelkner | i mean, i merged wth the central branch | 16:41 |
| th1a | So... are the accordions in the central branch? | 16:42 |
| th1a | That is, I'm not seeing them. | 16:42 |
| aelkner | they are broken now because i had to create a new viewlet manager | 16:43 |
| th1a | OK, so where does that stand now? | 16:43 |
| aelkner | i need to work with yvl to get it to work | 16:44 |
| th1a | OK, so we'll get to that. | 16:44 |
| th1a | yvl: Where are you now? | 16:44 |
| yvl | trying to put breadcrumbs near second nav | 16:44 |
| th1a | IN. | 16:45 |
| th1a | IN second nav. | 16:45 |
| yvl | I meant in | 16:45 |
| yvl | sorry | 16:45 |
| th1a | OK. ;-) | 16:45 |
| th1a | replaceafill: How's Cambodia? | 16:45 |
| replaceafill | almost done | 16:46 |
| replaceafill | didnt feel too well yesterday :( | 16:46 |
| replaceafill | i'll finish today | 16:46 |
| th1a | OK, good. | 16:46 |
| th1a | So what I'd like to do is send the invoices in tomorrow with a link to the Persons index looking as good as we can make it. | 16:47 |
| th1a | So I'd like to tidy that up as much as possible. | 16:47 |
| th1a | Right now the alignment is all off for some reason. | 16:48 |
| replaceafill | th1a it's the temporary margin | 16:48 |
| th1a | Perhaps replaceafill and I could work on that a bit after the meeting. | 16:48 |
| replaceafill | we put yesterday | 16:48 |
| th1a | hm? | 16:48 |
| replaceafill | for the accordions to simulate the sidebar | 16:49 |
| th1a | Well, we need to just put a sidebar in the person page. | 16:49 |
| th1a | Correct? | 16:49 |
| yvl | I'll do that | 16:50 |
| *** menesis has quit IRC | 16:50 | |
| th1a | OK. | 16:51 |
| th1a | all right, aelkner and yvl, go ahead with your discussion. | 16:52 |
| aelkner | yvl, i created a florish version of the advisory viewlet for FlourishPersonInfo manager | 16:53 |
| aelkner | but it doesn't get rendered in the view | 16:53 |
| *** menesis has joined #schooltool | 16:54 | |
| yvl | it does | 16:54 |
| yvl | but there's nothing registered for it | 16:55 |
| aelkner | the advisory viewlet is registered against it | 16:55 |
| aelkner | it.s rght below the manager registration | 16:55 |
| yvl | right | 16:56 |
| yvl | let me look at it | 16:56 |
| th1a | replaceafill: Meanwhile, you could adjust the link css to conform to the UWG. | 17:00 |
| yvl | let me fix if | 17:00 |
| yvl | it | 17:00 |
| yvl | link css? | 17:01 |
| th1a | No underline. | 17:01 |
| replaceafill | th1a, no underline at all, correct? | 17:02 |
| th1a | On hover, iirc. | 17:03 |
| replaceafill | ah right, page 25 :) | 17:04 |
| th1a | Is yvl looking at the viewlet registration now? | 17:04 |
| yvl | yes | 17:06 |
| yvl | fixing zcml directives | 17:06 |
| th1a | Just checking. | 17:06 |
| yvl | done | 17:08 |
| yvl | aelkner, please pull | 17:08 |
| replaceafill | th1a http://69.164.203.135:6660/persons | 17:08 |
| replaceafill | links with no underline unless they're hover | 17:09 |
| replaceafill | th1a what is the change you want for the links color? | 17:09 |
| aelkner | yvl, no revisions to pull | 17:10 |
| aelkner | don't i want to merge? | 17:10 |
| aelkner | ok, i merged and got your changes | 17:11 |
| aelkner | checking now | 17:11 |
| aelkner | ok, it works now | 17:11 |
| th1a | What do you (collectively) think of that link color. | 17:12 |
| th1a | ? | 17:12 |
| aelkner | th1a, i can put the ther viewlets back in now | 17:12 |
| aelkner | but as far as having them be tables by the end of the day, i can't promise to have that ready | 17:12 |
| th1a | Good, aelkner. Any other blockers. | 17:12 |
| th1a | ? | 17:12 |
| th1a | I'm just shooting for persons index by tomorrow. | 17:13 |
| aelkner | i know, but you wanted the viewlets changed to be tables, remember? | 17:13 |
| th1a | Yes. I'm just saying I don't need you to make that promise. | 17:13 |
| aelkner | ok, that's good | 17:13 |
| aelkner | but i'll do my best to get them to be tables anyway | 17:14 |
| *** dadeng has quit IRC | 17:14 | |
| th1a | OK. Good. | 17:14 |
| aelkner | yvl, can you explain what you had to change please | 17:14 |
| replaceafill | th1a regarding the link color, i like it :) | 17:15 |
| replaceafill | it's uniform with the primary nav | 17:15 |
| yvl | viewlets were registered to IPage, same as viewlet manager | 17:15 |
| yvl | that was too specific | 17:15 |
| yvl | it is ok that viewlet managers are registered to IPage by default | 17:15 |
| yvl | and when you want to specify a manger for a "content view" like FlourishPersonInfo is, you have to explicitly do that | 17:16 |
| yvl | but it is overkill for the viewlets themselves | 17:16 |
| yvl | specifying like you did should be enough | 17:16 |
| yvl | so I loosend viewlet binding to IBrowserView | 17:16 |
| yvl | by default | 17:16 |
| yvl | hence now your registration works | 17:17 |
| aelkner | i wish i understood what you were just saying, but it's not important for the short term | 17:17 |
| yvl | simply put | 17:17 |
| aelkner | th1a needs something working for tonight, so that is going to be my focus | 17:17 |
| yvl | I thought the viewlet registration you wrote was right | 17:18 |
| yvl | and the default behaviour of flourish:viewlet was not | 17:18 |
| aelkner | ok, i understand that | 17:18 |
| aelkner | just not the diff between IPage and BrowserView registration issues | 17:18 |
| yvl | IPage is our flourish page | 17:19 |
| yvl | the whole page, that is | 17:19 |
| aelkner | only whole pages? | 17:19 |
| yvl | by default | 17:19 |
| yvl | for managers | 17:19 |
| yvl | FlourishPersonInfo is a small "view", flourish.page.Content | 17:19 |
| yvl | it implements Zope's IBrowserView | 17:19 |
| yvl | basically | 17:20 |
| yvl | when you write a template, and set it for some class | 17:20 |
| yvl | the template's view is that class | 17:20 |
| yvl | it's like building a page from small views | 17:20 |
| yvl | a page is a view, but a view is not a page | 17:21 |
| aelkner | here's the thing | 17:21 |
| aelkner | it still is not clear what you are saying here, but | 17:21 |
| yvl | I imagine :) | 17:21 |
| aelkner | i don't have time to ask you explain any deeper | 17:21 |
| yvl | ok | 17:22 |
| aelkner | we can do that tomorrow | 17:22 |
| aelkner | but | 17:22 |
| yvl | will do that eventually | 17:22 |
| aelkner | i wanted to ask you about other things | 17:22 |
| yvl | go ahead | 17:22 |
| aelkner | you saw i added flourish.zcml for group and course | 17:22 |
| yvl | yes | 17:22 |
| aelkner | i was wondering to what extent the new views are going to be sprinkled about the source code | 17:23 |
| aelkner | of the current views | 17:23 |
| aelkner | i mean, it's confusing t have these python classes, templates/ zcml mixed in with the old stuff | 17:23 |
| aelkner | but i know that it is needed | 17:23 |
| aelkner | i'm just saying, could we agree on some conventions that will make it easier | 17:24 |
| yvl | we can try | 17:24 |
| aelkner | to find, or better put, separate out one from the other | 17:24 |
| aelkner | like for starters | 17:24 |
| aelkner | could we agree going forward that all new views, registered in flourish.zcml | 17:24 |
| yvl | sure! | 17:24 |
| aelkner | be coded in flourish.py? | 17:25 |
| yvl | hmm | 17:25 |
| aelkner | that there be no flourish classes mixed in with old classes | 17:25 |
| yvl | the /browser/ packages usually have one zcml file | 17:25 |
| yvl | so it's easy to go with flourish.zcml | 17:25 |
| aelkner | well, now wth flourish.zcml, they have two | 17:25 |
| yvl | but they do have multiple .py files | 17:25 |
| yvl | at the moment I tend to put flourish views dierctly below the old ones | 17:26 |
| *** dadeng has joined #schooltool | 17:26 | |
| replaceafill | -1 on flourish.py | 17:26 |
| yvl | since they're very short | 17:26 |
| aelkner | that is what i find to be a problem | 17:26 |
| aelkner | short doesn't matter | 17:26 |
| replaceafill | think of course and section | 17:26 |
| aelkner | finding stuff does | 17:26 |
| aelkner | having stuff mixed about is confusing and wasteful of time | 17:26 |
| replaceafill | it's easy to look into course/browser/course.py | 17:26 |
| yvl | it's just that I personally think that | 17:26 |
| yvl | course.py for finding course views | 17:26 |
| replaceafill | correct | 17:27 |
| aelkner | i don't think it's easy | 17:27 |
| replaceafill | course/browser/section.py for section views | 17:27 |
| yvl | is more important that flourish.py for finding flourish views | 17:27 |
| aelkner | that's not a sentence, please restate | 17:27 |
| th1a | It is a continued sentence. | 17:28 |
| aelkner | oh | 17:28 |
| yvl | I personally think that having section.py for finding all course views in the course package is more important than flourish.py in the course package for finding all flourish views. | 17:28 |
| replaceafill | +1 on yvl pov | 17:28 |
| th1a | Keep the current view organization. | 17:28 |
| aelkner | i lose again | 17:29 |
| * th1a sighs. | 17:29 | |
| aelkner | it's getting to be a habit accepting design decisions that i don't like | 17:29 |
| yvl | :/ | 17:29 |
| replaceafill | :| | 17:29 |
| * th1a doesn't understand those emoticons. | 17:30 | |
| replaceafill | :)) | 17:30 |
| aelkner | me, neither | 17:30 |
| yvl | :) | 17:30 |
| * menesis does not like everything being named *flourish* at all | 17:30 | |
| yvl | I guess we're not happy that aelkner is sad | 17:30 |
| th1a | Anyhow, the views have to go in the individual components. | 17:30 |
| th1a | That's not really up for debate. | 17:30 |
| th1a | You break the whole component architecture then. | 17:30 |
| th1a | We might as well switch to PHP. | 17:31 |
| yvl | menesis, we can add *flourish_skin* if you like | 17:31 |
| yvl | or *in_flourish_skin* | 17:31 |
| replaceafill | i like flourish, i dont understand what it means, but i like it :D | 17:31 |
| yvl | :D | 17:32 |
| th1a | It means students flourish when their school uses SchoolTool! | 17:33 |
| yvl | :DDD | 17:33 |
| replaceafill | :))) | 17:33 |
| th1a | OK. Any other pending issues, aelkner? | 17:34 |
| yvl | by the way | 17:34 |
| yvl | if some modules get "out of control" | 17:34 |
| yvl | I mean - huge | 17:35 |
| yvl | we could definitely split them | 17:35 |
| yvl | like section.py to: section.py, section_schooltool_skin.py and section_flourish_skin.py | 17:35 |
| yvl | or something along the lines | 17:36 |
| th1a | It is about time to come up with a workflow for revising pages. | 17:38 |
| *** dadeng has quit IRC | 17:38 | |
| th1a | This is more sequential than I initially thought. | 17:38 |
| th1a | I think. | 17:38 |
| th1a | Like: | 17:39 |
| *** ignas has quit IRC | 17:39 | |
| th1a | 1) make a flourish template. | 17:39 |
| th1a | ? | 17:39 |
| th1a | Since we aren't changing views in place. | 17:39 |
| th1a | Correct? | 17:40 |
| yvl | yes | 17:41 |
| yvl | maybe even | 17:41 |
| th1a | Perhaps yvl could make a list for me of the technical steps. | 17:41 |
| yvl | 1) make sure that there is no view registered already | 17:41 |
| th1a | And I'll write a Zope 2 app using worflows. | 17:41 |
| *** dadeng has joined #schooltool | 17:41 | |
| yvl | :) | 17:41 |
| yvl | you know, I'll email you that | 17:42 |
| th1a | That's what I was thinking. | 17:42 |
| yvl | and we'll put it up for discussion tomorrow | 17:42 |
| th1a | Perfect. | 17:42 |
| th1a | OK, replaceafill, can we do a little more CSS hacking? | 17:42 |
| replaceafill | sure | 17:42 |
| th1a | Get rid of the dots. | 17:43 |
| replaceafill | ok | 17:43 |
| *** issyl0 has quit IRC | 17:43 | |
| *** issyl0 has joined #schooltool | 17:44 | |
| replaceafill | done | 17:44 |
| yvl | so... we're done here? | 17:44 |
| *** issyl0 has quit IRC | 17:44 | |
| *** issyl0 has joined #schooltool | 17:44 | |
| *** issyl0 is now known as Guest81619 | 17:44 | |
| * th1a drops the bag of gravel. | 17:44 | |
| yvl | good luck guys | 17:46 |
| yvl | and aelkner, keep your chin up | 17:46 |
| yvl | I modified flourish:viewlet because your approach was better ;) | 17:46 |
| dadeng | aelkner, is the meeting over? | 17:47 |
| th1a | The meeting is over dadeng. | 17:48 |
| th1a | aelkner, did you see dadeng's email? | 17:48 |
| dadeng | yes, i want to talk about the email | 17:49 |
| *** Guest81619 is now known as issyl0 | 17:49 | |
| aelkner | i read the email | 17:54 |
| aelkner | so student filtering doesn't work the way you like | 17:54 |
| th1a | Well... | 17:55 |
| th1a | Is everyone enrolled for every class? | 17:55 |
| th1a | I'm confused. | 17:55 |
| th1a | Or at least the example person seems to be enrolled in everything at the same time? | 17:58 |
| dadeng | well... every subject is listed in each student's view | 18:01 |
| dadeng | and when I clicked on groups under a particular student's view | 18:02 |
| dadeng | i expected that it will just show the groups that student belong to | 18:02 |
| th1a | Is the student actually enrolled in all those classes? | 18:03 |
| th1a | That's the question. | 18:03 |
| th1a | Is it a display bug or a model bug? | 18:03 |
| dadeng | no, | 18:04 |
| dadeng | even in the student home page view, all the subjects are shown | 18:05 |
| th1a | replaceafill: Sorry, I missed your "done." | 18:05 |
| replaceafill | np | 18:05 |
| th1a | dadeng: Is the student actually enrolled in all those sections. Look at one of the section pages. | 18:05 |
| aelkner | yvl, thanks for modifying flourish:viewlet | 18:06 |
| dadeng | no, the student is not enrolled in all those sections | 18:06 |
| dadeng | i can send the data.fs to u to confirm that | 18:06 |
| th1a | OK, well, I guess you need to do that. | 18:06 |
| th1a | and aelkner will take a look at it. | 18:07 |
| dadeng | ok | 18:07 |
| th1a | replaceafill, put the footer text on one line. | 18:07 |
| th1a | And get rid of that extra space at left in the body. | 18:07 |
| aelkner | dadeng, you are using a sandbox, right? | 18:07 |
| dadeng | yes | 18:08 |
| aelkner | if so, what branch does it say you are using when you do bzr info? | 18:08 |
| th1a | (from the sandbox directory...) | 18:08 |
| dadeng | Standalone tree (format: 2a) | 18:09 |
| dadeng | Location: | 18:09 |
| dadeng | branch root: . | 18:09 |
| dadeng | Related branches: | 18:09 |
| dadeng | parent branch: http://bazaar.launchpad.net/~aelkner/schooltool/schooltool.niepa/ | 18:09 |
| aelkner | thanks | 18:09 |
| dadeng | hey, my Data.fs file is 145MB | 18:11 |
| dadeng | to large for uploading around this time | 18:11 |
| replaceafill | th1a done | 18:11 |
| replaceafill | th1a i dont think that one line footer is a good idea | 18:11 |
| aelkner | replaceafill, do you know how dadeng can compress his Data.fs? | 18:11 |
| replaceafill | th1a because of translations | 18:11 |
| th1a | replaceafill: OK. | 18:12 |
| replaceafill | aelkner i use bzip2 | 18:12 |
| th1a | replaceafill: Just revert and... | 18:12 |
| th1a | PACK. | 18:12 |
| th1a | PACK FIRST. | 18:12 |
| aelkner | how? | 18:12 |
| th1a | Just a sec... | 18:12 |
| th1a | replaceafill: ...take out the extra spacing above and below. | 18:13 |
| replaceafill | aelkner Manage -> Application Control -> ZODB Control | 18:13 |
| th1a | http://book.schooltool.org/database.html?highlight=database#packing-the-database | 18:13 |
| replaceafill | :) | 18:14 |
| aelkner | dadeng, could you try that, please? | 18:14 |
| th1a | Then zip. | 18:14 |
| dadeng | ok, let me see | 18:14 |
| replaceafill | th1a done | 18:17 |
| th1a | replaceafill: can you update the live instance when you make the change. | 18:17 |
| replaceafill | just did | 18:17 |
| replaceafill | sorry, got confused with all my windows :D | 18:17 |
| * replaceafill starts closing some | 18:17 | |
| th1a | I don't feel like I'm seeing shadows in the header. | 18:17 |
| replaceafill | th1a you want them in the navs? | 18:18 |
| th1a | Yes. | 18:18 |
| replaceafill | ok | 18:18 |
| th1a | As they were intitially. | 18:18 |
| replaceafill | correct | 18:18 |
| th1a | Also, can you make the lines closer in the footer? | 18:18 |
| replaceafill | that closer? | 18:18 |
| th1a | Yeah. | 18:19 |
| th1a | Take the background color out of the footer. | 18:19 |
| replaceafill | th1a done | 18:19 |
| th1a | Also, uncapitalize the first "The Shuttleworth..." | 18:20 |
| replaceafill | th1a done | 18:20 |
| th1a | There should be space above the form. | 18:21 |
| replaceafill | hhmm is that in the uwg's? | 18:21 |
| th1a | That's probably standard for the content area. | 18:21 |
| th1a | 32 px. | 18:21 |
| * replaceafill looks | 18:21 | |
| th1a | p.10 | 18:21 |
| replaceafill | got it | 18:22 |
| th1a | Is our spacing between fields and after the fieldset right? | 18:22 |
| replaceafill | th1a done | 18:24 |
| replaceafill | i mean the 32px | 18:24 |
| th1a | I'd say there should be more margin above the table. | 18:24 |
| replaceafill | th1a check the fieldset and fields | 18:29 |
| th1a | I think that's better... | 18:30 |
| th1a | It isn't all completely delineated. | 18:30 |
| th1a | Change "Group" to "Filter by..." | 18:31 |
| replaceafill | more margin for the table | 18:31 |
| th1a | Better. | 18:31 |
| replaceafill | label changed | 18:32 |
| th1a | We need just a little more top margin in the footer. | 18:32 |
| th1a | Also the sidebar needs the 32px top margin. | 18:33 |
| replaceafill | done | 18:33 |
| replaceafill | ah ok | 18:33 |
| replaceafill | is it me or there are no sidebar examples in the guidelines? | 18:34 |
| th1a | There are not. | 18:35 |
| th1a | We're on our own. | 18:35 |
| th1a | Well, we'll get some help. | 18:35 |
| th1a | Because the book will need one too. | 18:35 |
| replaceafill | ok, 32px for the sidebar top | 18:36 |
| replaceafill | we need to fix the login form :) | 18:37 |
| th1a | Do our bottom margins correspond to the standard? | 18:38 |
| replaceafill | i think so | 18:38 |
| th1a | 32 & 64? | 18:38 |
| * replaceafill looks | 18:38 | |
| replaceafill | yes | 18:38 |
| replaceafill | 32 and 64 | 18:38 |
| th1a | OK. That's all my papercuts. | 18:39 |
| th1a | Thanks replaceafill! | 18:39 |
| * replaceafill imagines th1a with a ruler on the screen :P | 18:39 | |
| replaceafill | i'll work on the shadows | 18:39 |
| th1a | I've got two screens with different color temperatures and pixel density, so it is a little confusing. | 18:39 |
| th1a | Ah... I see that gap now. | 18:40 |
| replaceafill | in the shadows? | 18:40 |
| th1a | Yes. | 18:40 |
| th1a | First priority now is wrapping up Cambodia before the end of the day today. | 18:40 |
| th1a | (but you don't have to do it first) | 18:40 |
| replaceafill | the easiest way i can imagine to fix it is to use a wrapper around the header and page | 18:40 |
| replaceafill | ok, i'll just put the shadows and get back to cambodia | 18:41 |
| th1a | Cool. | 18:41 |
| * th1a goes to lunch. | 18:41 | |
| dadeng | aelkner, i sent the email | 18:41 |
| aelkner | dadeng, i'm having an issue with the version of the branch i'm running being older than your data | 18:53 |
| aelkner | could you please email me the output of bzr info? | 18:54 |
| replaceafill | th1a nothing like a good css hack :P | 19:03 |
| replaceafill | th1a done with the shadows | 19:03 |
| dadeng | ok | 19:06 |
| *** alga has quit IRC | 19:07 | |
| dadeng | Standalone tree (format: 2a) | 19:07 |
| dadeng | Location: | 19:07 |
| dadeng | branch root: . | 19:07 |
| dadeng | Related branches: | 19:07 |
| dadeng | parent branch: http://bazaar.launchpad.net/~aelkner/schooltool/schooltool.niepa/ | 19:07 |
| * replaceafill goes back to cambodia | 19:09 | |
| dadeng | aelkner, see it | 19:11 |
| aelkner | dadeng, what about the other command from my email? | 19:23 |
| dadeng | which one? | 19:23 |
| aelkner | ls -ld eggs/schooltool.intervention* | 19:23 |
| dadeng | no didn't get that b4 | 19:24 |
| dadeng | just hold on | 19:24 |
| dadeng | i've sent it now | 19:27 |
| aelkner | ok, i have a buildout issue that i will try to solve by removing the eggs | 19:32 |
| aelkner | ok, got it running | 19:34 |
| replaceafill | yvl took the liberty to push to flourish "trunk" directly, only small css changes :) | 19:35 |
| replaceafill | aelkner sync with flourish "trunk" to get the latest css | 19:35 |
| replaceafill | aelkner and you'll see the person view moving to the left again | 19:35 |
| replaceafill | aelkner we'll fix that once we insert a sidebar | 19:35 |
| aelkner | replaceafill, will do, thanks | 19:36 |
| aelkner | dadeng, i sent you an email with a password request | 19:36 |
| aelkner | ok, i can sign in now | 19:41 |
| aelkner | th1a, ayt? | 19:45 |
| aelkner | dadeng, what student were you and th1a discussing? | 19:46 |
| dadeng | u can pick any student, but i used aabidemi and the same password i sent u | 19:46 |
| aelkner | ok, so what class is that particular student not in that the app says he/she is in? | 19:49 |
| *** menesis has quit IRC | 20:01 | |
| *** dadeng has quit IRC | 20:27 | |
| *** dadeng has joined #schooltool | 20:40 | |
| * th1a is here | 20:50 | |
| th1a | aelkner: is the student enrolled in all those sections? | 20:51 |
| *** menesis has joined #schooltool | 21:12 | |
| aelkner | th1a, yes, but that shouldn't surprise us | 21:12 |
| th1a | It is a rather essential bit of information though. | 21:12 |
| th1a | Does this make sense in terms of what's in the spreadsheet? | 21:12 |
| th1a | That is, did the spreadsheet put her in every section? | 21:13 |
| aelkner | yes | 21:27 |
| aelkner | there are 17817 rows in the sections sheet for that term | 21:29 |
| th1a | OK. Problem solved! | 21:30 |
| th1a | aelkner? | 21:35 |
| th1a | replaceafill: ayt? | 21:53 |
| dadeng | so, is it possible to have a solution to this problem by tomorrow morning, aelkner? | 21:55 |
| th1a | dadeng: The problem is in your spreadsheet. | 21:55 |
| dadeng | well... the spreadsheet was directly exported from the ST instance | 21:56 |
| dadeng | and there was never anytime we put those repeated stuff in there | 21:56 |
| dadeng | so did it happen? | 21:57 |
| dadeng | how did it happen? | 21:57 |
| th1a | Well... do you still have the old instance? | 21:57 |
| dadeng | i have them all | 21:58 |
| dadeng | running on the initial system we setup for demo | 21:58 |
| th1a | So you have the old instance which looks correct. | 21:59 |
| dadeng | that's where i have been copying the Data.fs file out | 21:59 |
| dadeng | i'm not sure anyone looks correct until i check | 21:59 |
| dadeng | i had not use the student's module since | 21:59 |
| th1a | Well, we have to confirm that at some point this worked. | 21:59 |
| dadeng | but now that we are making progress towards real deployment | 22:00 |
| replaceafill | th1a back from lunch | 22:00 |
| dadeng | ok, give me some time to check it out now | 22:00 |
| th1a | OK. | 22:00 |
| th1a | replaceafill, http://ubuntu-news.org/ | 22:00 |
| replaceafill | :O | 22:01 |
| replaceafill | nice | 22:01 |
| th1a | Although I guess it is not officially official. | 22:01 |
| th1a | But yes, that's a good start. | 22:01 |
| replaceafill | yes, it's different from the uwg | 22:01 |
| replaceafill | 250px width | 22:02 |
| replaceafill | so, should i take those links styles and put them on our sidebar? | 22:03 |
| th1a | let me compare for a minute. | 22:03 |
| replaceafill | cool | 22:04 |
| replaceafill | font is bigger :) | 22:04 |
| th1a | I like the width we have, we do need a little clearer spacing between the two. | 22:05 |
| th1a | I like their section header. | 22:05 |
| th1a | Border, etc. | 22:06 |
| replaceafill | dotted border | 22:06 |
| th1a | The main thing we need is a gap between the sidebar and main content. | 22:08 |
| *** ignas has joined #schooltool | 22:08 | |
| th1a | Do you have a sense of how the spacing should be according to the grid? | 22:08 |
| replaceafill | 16px | 22:09 |
| replaceafill | page 10 | 22:09 |
| th1a | Well... do you feel like fiddling with this now? | 22:09 |
| replaceafill | sure, you're awake ;) | 22:09 |
| th1a | OK. | 22:09 |
| replaceafill | hold on, let me get to the server | 22:09 |
| th1a | I think we should try Aubergine links. | 22:10 |
| th1a | #772953 | 22:10 |
| replaceafill | th1a http://69.164.203.135:6660/persons/@@index.html | 22:11 |
| replaceafill | oops typo | 22:12 |
| replaceafill | i put backgrounds so you can see the gap | 22:12 |
| replaceafill | there's 16px | 22:12 |
| replaceafill | ok, removing the weird background colors... | 22:13 |
| th1a | Oh. | 22:13 |
| th1a | So the gap is right we just need to rejigger the foreground and background colors so that the gap is the background color. | 22:13 |
| th1a | That is, white. | 22:14 |
| replaceafill | correct | 22:14 |
| dadeng | aelkner, th1a, it is the same in the original machine | 22:14 |
| th1a | OK. | 22:14 |
| dadeng | where all data was manually entered | 22:14 |
| dadeng | and they were not entered that way | 22:14 |
| th1a | Well... | 22:15 |
| th1a | The enrollment process is by far the most used and tested part of SchoolTool, so it is relatively unlikely to be a bug in SchoolTool. | 22:15 |
| replaceafill | th1a changed the links colors | 22:16 |
| replaceafill | (just for the table for now) | 22:16 |
| replaceafill | th1a they looked like "visited" links | 22:17 |
| th1a | Yeah, I guess change them back. | 22:17 |
| replaceafill | reverted | 22:17 |
| replaceafill | changing the background to white, correct? | 22:18 |
| replaceafill | to match the layout of ubuntu news | 22:18 |
| th1a | Yes. | 22:19 |
| replaceafill | we need a taller sidebar :( | 22:21 |
| th1a | lol. | 22:22 |
| replaceafill | should i make it go all the way down? | 22:22 |
| replaceafill | i dont like that way though | 22:22 |
| th1a | Well, the problem is now there's no left margin for the content area. | 22:22 |
| th1a | Within the content area. | 22:23 |
| replaceafill | we can steal some of those 16px ;) | 22:23 |
| replaceafill | and break the guidelines!!! :D | 22:23 |
| th1a | We're going to have to. | 22:23 |
| th1a | There should be 16 on each side within the sidebar too, I think. | 22:24 |
| *** dadeng has quit IRC | 22:24 | |
| th1a | It is all a little cramped now. | 22:25 |
| replaceafill | refresh | 22:25 |
| replaceafill | hhmm yes | 22:25 |
| th1a | I was thinking leave the gap between the sidebar and main the same but add margin inside the main. | 22:26 |
| replaceafill | you mean padding, right? | 22:26 |
| th1a | Oh... I can never remember. | 22:26 |
| replaceafill | :D | 22:26 |
| replaceafill | me neither ;) | 22:26 |
| th1a | But essentially: | 22:26 |
| replaceafill | there | 22:27 |
| th1a | shadow border <-16->edge of sidebar<-16->text<-16->rt. edge of sidebar<-16->edge of main<-16->main content | 22:28 |
| replaceafill | there | 22:32 |
| th1a | The only question is if that leaves the sidebar wide enough. | 22:32 |
| th1a | I think it can work. | 22:33 |
| replaceafill | we're going to insert old action links there, correct? | 22:33 |
| replaceafill | "New Administrator" :) | 22:33 |
| th1a | Well... yes. | 22:33 |
| replaceafill | ah no | 22:33 |
| replaceafill | Add: | 22:33 |
| replaceafill | - Administrator | 22:33 |
| replaceafill | the ubuntu news site uses shadows for the borders of the sidebar | 22:34 |
| th1a | lol "Marius Gedminas has fixed a nice multi monitor bug, #778256 – “Notification area (‘system tray’) missing when using dual monitors of different sizes, with their bottoms aligned”" | 22:35 |
| th1a | Small world. | 22:35 |
| replaceafill | :) | 22:35 |
| th1a | We could do that. They have a rounded sidebar and square main area, which I don't like. | 22:35 |
| th1a | (mixing I don't like) | 22:35 |
| replaceafill | ah | 22:35 |
| replaceafill | little shadow for the sidebar | 22:36 |
| replaceafill | let's shadow everything ;) | 22:37 |
| * th1a has caught replaceafill's headache. | 22:37 | |
| replaceafill | :( | 22:37 |
| replaceafill | hhmm same shadow for both, but the sidebar doesnt show its left shadow | 22:40 |
| replaceafill | maybe because of its width | 22:40 |
| th1a | That's a lot of shadow. | 22:45 |
| replaceafill | remove both? | 22:45 |
| replaceafill | i just noticed our dropdown has no height specified | 22:47 |
| replaceafill | but if i set it to 20px | 22:47 |
| replaceafill | it looks good in chrome | 22:47 |
| replaceafill | but not so good in firefox | 22:47 |
| replaceafill | or maybe it's just me? | 22:47 |
| th1a | Dropdown? | 22:50 |
| replaceafill | select element | 22:50 |
| replaceafill | for groups | 22:50 |
| th1a | Ah. | 22:50 |
| th1a | Well, let's try the current sidebar width for now. | 22:51 |
| th1a | See how disciplined we can be. | 22:51 |
| th1a | Considering that it has a fixed width! | 22:51 |
| th1a | otoh, languages have different widths. | 22:55 |
| replaceafill | yes | 22:55 |
| replaceafill | that will bring trouble | 22:55 |
| th1a | OK, just make the sidebar one step wider. | 22:56 |
| replaceafill | by one step you mean how many pixels? 16? | 22:57 |
| th1a | I guess. | 22:57 |
| th1a | That's the system, right? | 22:57 |
| replaceafill | yes | 22:57 |
| replaceafill | done | 22:59 |
| *** menesis has quit IRC | 23:01 | |
| th1a | And we need the wide right margin back on the main content area. | 23:02 |
| * th1a goes to start a fire in the grill. | 23:02 | |
| *** menesis has joined #schooltool | 23:02 | |
| th1a | replaceafill: You can make the text style more like the ubuntu news site too. | 23:02 |
| replaceafill | th1a wide right margin? | 23:03 |
| th1a | replaceafill: Remember when I was asking why it wasn't centered? | 23:19 |
| replaceafill | when i told you about: 144px, 544px, 224px? | 23:20 |
| replaceafill | btw, i added the 16px to the sidebar width | 23:21 |
| replaceafill | now its: | 23:21 |
| replaceafill | 160px, 528px, 224px | 23:21 |
| th1a | Well, maybe we should take a bite out of the 224 instead. | 23:21 |
| replaceafill | ok, 160px, 544px, 208px then | 23:22 |
| replaceafill | done | 23:23 |
| replaceafill | for some reason i'm seeing the spinner in my firefox when i load the page... | 23:23 |
| th1a | The bottom margin needs to be restored too. | 23:23 |
| replaceafill | like it's not downloading completely | 23:23 |
| replaceafill | th1a they havent changed, 16px + 32px | 23:24 |
| replaceafill | the thing is the 16px are white now | 23:24 |
| replaceafill | they used to be gray | 23:24 |
| *** ignas has quit IRC | 23:24 | |
| th1a | Yes... that's the tricky part. | 23:25 |
| th1a | Do we need to turn our margins into padding or something? | 23:25 |
| replaceafill | they are paddings actually | 23:25 |
| th1a | I guess I have to go cook dinner now. | 23:27 |
| replaceafill | :) | 23:27 |
| replaceafill | i'll be around | 23:27 |
| *** menesis has quit IRC | 23:29 | |
| *** menesis has joined #schooltool | 23:29 | |
| th1a | Well, these margins need to be sorted. | 23:38 |
| *** dadeng has joined #schooltool | 23:51 | |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!
