| *** th1a has quit IRC | 00:14 | |
| *** ignas has quit IRC | 00:34 | |
| *** ignas has joined #schooltool | 00:44 | |
| *** ignas has quit IRC | 01:22 | |
| *** replaceafill has quit IRC | 01:34 | |
| *** menesis has quit IRC | 03:09 | |
| *** alga has quit IRC | 03:50 | |
| *** aks has joined #schooltool | 06:16 | |
| *** aks has joined #schooltool | 06:16 | |
| *** alga has joined #schooltool | 09:36 | |
| *** alga has quit IRC | 11:08 | |
| *** menesis has joined #schooltool | 11:35 | |
| *** ignas has joined #schooltool | 12:39 | |
| *** aks has quit IRC | 14:19 | |
| *** menesis has quit IRC | 14:22 | |
| *** ignas has quit IRC | 14:54 | |
| *** ignas has joined #schooltool | 14:56 | |
| *** menesis has joined #schooltool | 15:05 | |
| *** replaceafill has joined #schooltool | 16:04 | |
| *** th1a has joined #schooltool | 16:27 | |
| * th1a is getting his summer schedule dialed in. | 16:27 | |
| th1a | Up at 7:30, hit the skatepark for about 45 minutes, back home, shower, log on IRC 9:27. | 16:28 |
|---|---|---|
| th1a | hi yvl, menesis, aelkner, replaceafill. | 16:29 |
| replaceafill | good morning/afternoon | 16:29 |
| th1a | How was your birthday, aelkner? | 16:29 |
| th1a | Enjoying the Phillies? | 16:29 |
| yvl | morning guys | 16:30 |
| aelkner | morning | 16:31 |
| menesis | hi | 16:31 |
| aelkner | yes, the phillies have given me a nice bday | 16:31 |
| th1a | OK, first off, I want thank you guys for your work this month. | 16:31 |
| th1a | Unless we all fall to pieces in the next six weeks, this is going to be the biggest, most impressive release of SchoolTool ever. | 16:32 |
| th1a | And the timing will be perfect. | 16:32 |
| * th1a notes we will not be falling to pieces in the next six weeks. | 16:32 | |
| th1a | And, beyond verbal praise, you'll all get bonuses this month. | 16:33 |
| th1a | I'll email you individually about the particulars. | 16:33 |
| * yvl seconds the kudos | 16:33 | |
| yvl | nice work, guys! | 16:33 |
| th1a | Well, "all" meaning yvl, aelkner and replaceafill. | 16:33 |
| replaceafill | :) | 16:33 |
| th1a | menesis got a nice bonus for the release. | 16:34 |
| th1a | And has had a relatively relaxing June. | 16:34 |
| menesis | true | 16:34 |
| th1a | No slight intended menesis. ;-) | 16:35 |
| menesis | except for two days when the server went down | 16:35 |
| th1a | Yeah... we haven't been getting those warnings lately, right? | 16:35 |
| th1a | We should try to move before the October release. | 16:36 |
| th1a | OK, summer vacation. | 16:37 |
| menesis | I get those warnings every few weeks | 16:37 |
| menesis | what I need first is access to DNS settings for schooltool.org | 16:38 |
| th1a | OK. I'll look into that menesis. | 16:39 |
| replaceafill | +1 on us managing the dns :) | 16:39 |
| menesis | then I can at least add a fallback IP pointing to schooltool.pov.lt | 16:39 |
| th1a | Vacation: I'm going to be at the in-laws from July 11-15 and my parents around July 26 to August 1. | 16:40 |
| th1a | But I'll be online daily throughout. | 16:40 |
| aelkner | up in Maine? | 16:40 |
| th1a | That's pretty similar to Justas's plans. | 16:41 |
| aelkner | beautiful in the summer | 16:41 |
| th1a | Jennifer's parents are in Maine and mine are in PA. | 16:41 |
| th1a | I guess I don't feel particularly strongly about coordinating vacations one way or another. | 16:42 |
| th1a | I don't like to have nobody working on SchoolTool for a week in case something comes up. | 16:42 |
| th1a | So just let me know aelkner and replaceafill when you decide what to do. | 16:43 |
| replaceafill | i'll keep working during the summer | 16:43 |
| aelkner | i have no vacation plans as of yet, but i'll let you know if i decide to take off | 16:43 |
| th1a | OK. | 16:44 |
| th1a | Moving on... | 16:44 |
| th1a | yvl: How's the wizard coming? | 16:44 |
| th1a | (if it is a wizard) | 16:45 |
| yvl | it is :) | 16:45 |
| yvl | hit a few snags today | 16:45 |
| yvl | but now over them, moving on | 16:45 |
| yvl | the non-modal version of it is almost done | 16:45 |
| yvl | I'll work on making it modal tomorrow | 16:45 |
| yvl | spent (little) time trying to decide if I want JSON to serialize dates automatically or explicitly | 16:46 |
| yvl | I think we'll end up with almost manual conversion | 16:46 |
| yvl | string format as per RFC3999 IIRC | 16:47 |
| yvl | kind of had to choose between this and ASP.NET style | 16:47 |
| yvl | so if anyone wants to communicate dates / times to javascript from the view | 16:47 |
| yvl | refert to ttwizard.py | 16:47 |
| yvl | refer | 16:48 |
| yvl | or me :) | 16:48 |
| th1a | btw, do you want to discuss your change of calendar widgets in Cambodia, replaceafill? | 16:48 |
| replaceafill | ah sure, it was a small change | 16:48 |
| replaceafill | people in cambodia complained about the jquery ui calendar widget | 16:49 |
| replaceafill | saying that it's difficult to navigate to distant years | 16:49 |
| replaceafill | like if you want to go to 1991 or so | 16:49 |
| replaceafill | you have to click click click the arrows :) | 16:49 |
| replaceafill | but the jquery ui widget has options for that | 16:50 |
| replaceafill | you can display the months and a set of years in dropdowns | 16:50 |
| replaceafill | $(".date-field").datepicker({ | 16:50 |
| replaceafill | dateFormat: 'dd-mm-yy', | 16:50 |
| replaceafill | changeMonth: true, | 16:50 |
| replaceafill | changeYear: true, | 16:50 |
| replaceafill | yearRange: 'c-20:c+05' | 16:50 |
| replaceafill | }); | 16:50 |
| yvl | (can you please point at the current working branch at some point? ) | 16:50 |
| replaceafill | yvl ah sure | 16:51 |
| replaceafill | yvl maybe next week you can review it | 16:51 |
| replaceafill | i talked to th1a we should do it at some point | 16:51 |
| replaceafill | it's a mess i'm still cleaning btw :) | 16:51 |
| yvl | don't worry about the mess ;) | 16:51 |
| th1a | Perhaps we should at least use that widget whenever we're entering date of birth, if not always. | 16:52 |
| replaceafill | the jquery docs for the change is at http://jqueryui.com/demos/datepicker/#dropdown-month-year | 16:52 |
| th1a | I don't know if it is disadvantageous in regular cases for some reason. | 16:52 |
| replaceafill | for the cambodia case i also applied a little css: | 16:53 |
| replaceafill | .ui-datepicker select.ui-datepicker-month, | 16:53 |
| replaceafill | .ui-datepicker select.ui-datepicker-year { | 16:53 |
| replaceafill | height: 33px; | 16:53 |
| replaceafill | } | 16:53 |
| * aelkner is back in five minutes | 16:53 | |
| replaceafill | the khmer fonts make the month dropdown higher than the year's | 16:53 |
| replaceafill | so they looked uneven | 16:53 |
| replaceafill | http://69.164.203.135:7080/auth/@@login.html?username=manager&password=schooltool&LOGIN= | 16:53 |
| replaceafill | that's the cambodia vps, if you click that link you'll get redirected to the add view for school years | 16:54 |
| replaceafill | you can see the widget there | 16:54 |
| replaceafill | please don't add a school year ;) | 16:54 |
| yvl | ok :) | 16:56 |
| * aelkner is back | 16:56 | |
| yvl | the calendar is glitching for some reason in FFox 3.5 | 16:56 |
| th1a | I guess timetables don't actually use the calendar widget anyhow, so I'm just diverting off topic anyhow. | 16:56 |
| replaceafill | :) | 16:57 |
| replaceafill | yvl really? :( | 16:57 |
| replaceafill | i'm using 3.5.16 and it doesnt... | 16:57 |
| yvl | weird | 16:57 |
| yvl | well - nice changes in any case | 16:57 |
| replaceafill | i think it's a valid argument (the one about the clicks) | 16:58 |
| yvl | true | 16:58 |
| yvl | we should definitely add year/month ranges | 16:59 |
| th1a | Finding someone's birthday is a different problem than a date this year. | 16:59 |
| th1a | So yes, something must be done. | 16:59 |
| yvl | we can always fine-tune year ranges for specific forms of course | 17:00 |
| th1a | Yes. | 17:00 |
| th1a | OK. | 17:00 |
| yvl | and it even updates the dropdown if you click out of the year span | 17:00 |
| yvl | right, let's move on | 17:00 |
| th1a | Back to yvl. Sorry to interrupt. | 17:00 |
| yvl | that's ok, I'm done actually | 17:01 |
| th1a | So we should be able to have a crack at the wizard tomorrow? | 17:02 |
| yvl | yes | 17:02 |
| * yvl is a bit hesitant to merge timetabling to flourish branch | 17:03 | |
| yvl | but it's better to merge earlier than later | 17:03 |
| th1a | Yes, we need to do it. | 17:03 |
| yvl | just FYI | 17:03 |
| yvl | I broke something when merging | 17:03 |
| yvl | i.e. we'll find bugs | 17:04 |
| yvl | the branches diverged, and it was a bit too big chunk to review properly | 17:04 |
| yvl | but nothing serious I think | 17:04 |
| th1a | OK. | 17:05 |
| yvl | by the way | 17:05 |
| yvl | can I temporarily add the modal dialong link somewhere? | 17:05 |
| yvl | as in - in a random place | 17:05 |
| th1a | For... new timetables? | 17:05 |
| yvl | yes | 17:06 |
| yvl | to call up the wizard | 17:06 |
| yvl | just for you guys to see it | 17:06 |
| th1a | Uh... | 17:06 |
| yvl | unless I also flourish the container tomorrow | 17:06 |
| yvl | well, now that I said it | 17:07 |
| th1a | Just don't stick it somewhere where we have to look at it all the time. | 17:07 |
| yvl | right | 17:07 |
| yvl | that's what crossed my mind just now | 17:07 |
| th1a | I told aelkner he couldn't jam a convenience link in the top bar. | 17:07 |
| yvl | right | 17:08 |
| th1a | OK. | 17:08 |
| th1a | Looking forward to it. | 17:08 |
| th1a | Thanks yvl. | 17:08 |
| th1a | aelkner, anything new to report? | 17:09 |
| th1a | I hope you weren't sitting at home coding all night. | 17:09 |
| aelkner | i got some in while enjoying the game :) | 17:09 |
| aelkner | how about that Worley | 17:09 |
| aelkner | you guys are injured, so... | 17:09 |
| aelkner | anyway, i created the two link managers, Add and Actions | 17:10 |
| aelkner | my instance is updated with them | 17:10 |
| aelkner | i changed the title to and edit link in the table | 17:10 |
| aelkner | and the edit view now redirects to the table on Apply as well as Cancel | 17:11 |
| aelkner | the add views still redirect to index.html of the new object on Add | 17:11 |
| aelkner | i was going to ask, should it go back to the table? | 17:11 |
| aelkner | also, they are not modal yet | 17:11 |
| aelkner | that's next | 17:11 |
| aelkner | i was hoping that yvl would stick around another hour | 17:12 |
| aelkner | so i could ask aything about current modal paradigm if i need to | 17:12 |
| th1a | Is that pushed to the test server? | 17:12 |
| aelkner | yes | 17:12 |
| aelkner | th1a, should we redirect to table on succesfful add? | 17:12 |
| th1a | yes. | 17:13 |
| replaceafill | one comment about forms: | 17:13 |
| aelkner | and do we even need index.html of field now that we have the table? | 17:13 |
| replaceafill | should all forms subclass the expanded page? | 17:13 |
| replaceafill | http://69.164.203.135:6660/demographics/addText.html | 17:13 |
| replaceafill | for instance, shows the sidebar | 17:13 |
| aelkner | expanded page has no sidebar? | 17:13 |
| replaceafill | correct | 17:14 |
| aelkner | i was asking before (a couple days ago) | 17:14 |
| aelkner | why is it called expanded if it has less? | 17:14 |
| aelkner | it's not important, i'm just curious | 17:15 |
| aelkner | replaceafill, ayway, it's going to be modal, so... | 17:15 |
| replaceafill | ok | 17:15 |
| aelkner | yvl, are you around another hour or so? | 17:15 |
| yvl | aelkner, as it was said before, the content area is expanded | 17:15 |
| yvl | sure, aelkner | 17:16 |
| aelkner | how, expanded? | 17:16 |
| yvl | larger | 17:16 |
| aelkner | by not including the sidebar | 17:16 |
| yvl | takes up space where sidebar was | 17:16 |
| aelkner | ok | 17:16 |
| yvl | but I'm ok with renaming to something else | 17:17 |
| aelkner | th1a, should i get rid of index.html of fields and make edit view the index.html view? | 17:17 |
| aelkner | yvl, how about WidePage | 17:17 |
| aelkner | it's not important though | 17:17 |
| aelkner | let's forget about the ame for now | 17:18 |
| th1a | aelkner: Well... I wouldn't change the name in case we decide we do need a regular index.html view view later. | 17:18 |
| aelkner | ok | 17:18 |
| aelkner | i'm done my report | 17:19 |
| th1a | (where later might be this afternoon) ;-) | 17:19 |
| th1a | Thanks aelkner. | 17:19 |
| aelkner | right | 17:19 |
| th1a | replaceafill, ? | 17:19 |
| replaceafill | th1a i finished cambodia's tasks | 17:19 |
| replaceafill | i can work on flourish again | 17:19 |
| th1a | Ah. So you need tasks? | 17:20 |
| replaceafill | yes | 17:20 |
| th1a | Start with the password form. | 17:20 |
| replaceafill | ah | 17:20 |
| th1a | And we'll go from there. | 17:20 |
| replaceafill | ok | 17:20 |
| th1a | Also, the spacing in the footer got spread out a bit when we jiggered the spacings. | 17:21 |
| th1a | So basically I'll keep replaceafill and aelkner busy today. | 17:21 |
| replaceafill | ah, true | 17:21 |
| th1a | OK, thanks guys. | 17:21 |
| * th1a drops the bag of gravel. | 17:21 | |
| yvl | thanks th1a | 17:23 |
| yvl | aelkner, so - modal dialogs | 17:23 |
| yvl | ajax'y form submitting is WIP at the moment | 17:24 |
| * th1a goes to make brunch. | 17:24 | |
| yvl | bon appetite, th1a | 17:24 |
| yvl | so making them open modal should be quite easy | 17:25 |
| yvl | but there's nothing IIRC done on submitting and error validation | 17:25 |
| * replaceafill goes to get breakfast | 17:30 | |
| aelkner | yvl, so are you working on some wiring for modals i should know about | 17:30 |
| yvl | bon appetite, replaceafill | 17:31 |
| yvl | aelkner, I think I should :) | 17:31 |
| aelkner | i like the idea :) | 17:31 |
| aelkner | should i wait until tomorrow to do the modals then? | 17:31 |
| yvl | give me 30-40 minutes... | 17:31 |
| aelkner | ah, perfect | 17:31 |
| aelkner | let's talk then | 17:31 |
| aelkner | th1a, demo instance now redirects to table on successful add | 18:00 |
| th1a | Can you easily make the reorder page use a table? | 18:02 |
| aelkner | how woul you like it to look? | 18:03 |
| th1a | Just... tabular. | 18:03 |
| aelkner | we have three things, the dropdown, for reordering | 18:03 |
| aelkner | the checkbox, for deleting | 18:04 |
| aelkner | and the title link | 18:04 |
| th1a | Just a table above the buttons. | 18:04 |
| th1a | And all the buttons should have the standard style now. | 18:04 |
| aelkner | make those three things the three columns of the table? | 18:04 |
| th1a | Yeah. | 18:04 |
| th1a | No header. | 18:04 |
| aelkner | ok, should be no big deal | 18:04 |
| yvl | sorry aelkner | 18:04 |
| yvl | hacky approach didn't work out | 18:04 |
| yvl | so nothing more today | 18:05 |
| yvl | you can make the links open modal dialogs | 18:05 |
| aelkner | ok, we can table the modals for now | 18:05 |
| th1a | So what can't we do with modals at this point? | 18:05 |
| th1a | We don't have ajaxy validation? | 18:05 |
| yvl | add buttons won't work | 18:05 |
| yvl | or rather work weird | 18:05 |
| yvl | won't reload into the modal dialog | 18:05 |
| aelkner | can you have seomthing for us by tomorrow morning? | 18:05 |
| yvl | yes | 18:05 |
| th1a | OK. | 18:06 |
| th1a | No biggie. | 18:06 |
| yvl | but you can make them open modals, just mimic the preferences button | 18:06 |
| aelkner | open modals? | 18:06 |
| aelkner | what do you mean by that? | 18:06 |
| yvl | click link -> opens a modal dialog | 18:06 |
| th1a | Pop one open that doesn't actually work. | 18:06 |
| yvl | but renders correctly visually | 18:06 |
| th1a | We don't need that now though. | 18:07 |
| yvl | cool | 18:07 |
| aelkner | yeah, it can wait until tomorrow | 18:07 |
| yvl | all right then, see you guys soon ;) | 18:07 |
| aelkner | have a good one | 18:07 |
| th1a | Thanks yvl. | 18:07 |
| yvl | thanks guys | 18:08 |
| yvl | good luck! | 18:08 |
| th1a | replaceafill: ayt? | 18:08 |
| replaceafill | th1a yes | 18:08 |
| th1a | Look at http://69.164.203.135:36660/demographics/reorder.html | 18:08 |
| replaceafill | th1a i'm reviewing a merge request by menesis | 18:08 |
| replaceafill | ah ok | 18:08 |
| th1a | are you looking at it replaceafill? | 18:09 |
| replaceafill | yes | 18:09 |
| th1a | OK. | 18:09 |
| th1a | Let's go over some general points. | 18:09 |
| replaceafill | ok | 18:09 |
| th1a | This kind of form shouldn't have a sidebar. | 18:09 |
| th1a | related question: What's the deal with pages that just have a little gray rectangle at the top of the sidebar? | 18:10 |
| th1a | I mean, I know they should have no links, but we need to be rid of the box. | 18:10 |
| replaceafill | th1a we have a default padding for the sidebar | 18:11 |
| replaceafill | that's what you see | 18:11 |
| replaceafill | even if it's empty | 18:11 |
| replaceafill | let me check the page template | 18:11 |
| th1a | How do we hide/get rid of it. | 18:11 |
| th1a | ? | 18:11 |
| th1a | We got rid of it in the persons/add views. | 18:12 |
| * replaceafill looks for a page with empty sidebar | 18:12 | |
| replaceafill | th1a yes, because the forms use the expanded page class | 18:12 |
| replaceafill | now i can't find a page with empty sidebar... | 18:13 |
| th1a | perhaps it should be the NoSidebar page class. | 18:13 |
| replaceafill | th1a can you point me to a page with empty sidebar please?!!? | 18:14 |
| replaceafill | i can't find any :( | 18:14 |
| replaceafill | ah /reports | 18:14 |
| replaceafill | th1a http://69.164.203.135:6660/persons/alvaro/reports | 18:17 |
| replaceafill | conditioned the sidebar | 18:17 |
| replaceafill | <div class="refine" tal:define="refine view/providers/page_refine|nothing" | 18:18 |
| replaceafill | tal:condition="refine"> | 18:18 |
| replaceafill | i guess this way forms don't have to subclass expanded page anymore just to get rid of the sidebar | 18:19 |
| * replaceafill tests it | 18:19 | |
| aelkner | th1a, +1 on the class name | 18:19 |
| th1a | Is it too late to just change that replaceafill? | 18:19 |
| th1a | It is pretty confusing. | 18:19 |
| replaceafill | th1a you mean the class name? | 18:19 |
| th1a | Yes. | 18:20 |
| replaceafill | hhm i dont' think so, few classes are using it | 18:21 |
| th1a | Why don't we just do that then? | 18:21 |
| replaceafill | 12 references only | 18:21 |
| replaceafill | ok | 18:21 |
| th1a | replaceafill: Let's discuss the rest of this before you do that. | 18:23 |
| replaceafill | ah... i was already doing it :D | 18:24 |
| replaceafill | http://69.164.203.135:6660/persons/alvaro/@@edit.html | 18:24 |
| th1a | aelkner: Do you follow the point about the sidebar? | 18:24 |
| replaceafill | for example is a regular page now | 18:24 |
| th1a | (not being there) | 18:24 |
| replaceafill | with that change we probably wont need the NoSidebar class | 18:24 |
| th1a | OK. | 18:24 |
| th1a | Well, that's simple enough. | 18:24 |
| replaceafill | yes, unless there are viewlets registered for that refined manager the sidebar wont be there | 18:25 |
| replaceafill | ok, let's disscuss the rest | 18:25 |
| th1a | Do you understand why we don't want it there, aelkner? | 18:25 |
| th1a | In the meantime... | 18:25 |
| aelkner | sorry, wasn't following the conversation, looking back at it now... | 18:25 |
| th1a | ... | 18:27 |
| aelkner | th1a, we all agreed to changing the class name to avoid confusing while discussing which classes to use | 18:28 |
| replaceafill | oops, typo on my instance :) | 18:28 |
| aelkner | is there anything else i missed? | 18:28 |
| th1a | I'm just saying, do you understand that pages like this should not have sidebars? | 18:28 |
| aelkner | yes | 18:28 |
| th1a | OK. | 18:28 |
| th1a | Breadcrumbs. | 18:28 |
| th1a | What should be up there and how? | 18:28 |
| aelkner | specifically for demos or in general? | 18:29 |
| th1a | Or should we just wait until we have a manage page. | 18:29 |
| th1a | For this page. | 18:29 |
| aelkner | this? | 18:29 |
| th1a | The demographics page, the reorder fields page. | 18:29 |
| aelkner | School -> Demographic Fields? | 18:30 |
| * replaceafill thinks we should work breadcrumbs last | 18:30 | |
| th1a | Probably. | 18:30 |
| replaceafill | after we have most parts working | 18:30 |
| th1a | OK. | 18:30 |
| replaceafill | just my $0.02 | 18:30 |
| th1a | Also, You could argue this is more site settings. | 18:30 |
| aelkner | replaceafill, are breadcrumbs in a state of flux? | 18:31 |
| th1a | These are things that have to be resolved. | 18:31 |
| th1a | I'd say skip it since we don't know the root. | 18:31 |
| replaceafill | aelkner ? | 18:31 |
| th1a | They work for persons | 18:31 |
| aelkner | replaceafill, 'after we have most parts working' | 18:32 |
| aelkner | was that about nreadcrumbs? | 18:32 |
| replaceafill | aelkner yes | 18:32 |
| th1a | Most parts of the manage/setting views. | 18:32 |
| replaceafill | yes | 18:32 |
| th1a | Is I think the context. | 18:32 |
| th1a | OK, moving on... | 18:32 |
| th1a | formatting drop downs. | 18:32 |
| th1a | Not too much on the subject on p. 22. | 18:33 |
| replaceafill | ok | 18:33 |
| replaceafill | no :( | 18:33 |
| th1a | Did we already change the CSS for these at all? | 18:33 |
| th1a | What am I looking at now? | 18:33 |
| * replaceafill looks | 18:34 | |
| th1a | I guess this is the first time we've done drop downs in flourish? | 18:34 |
| replaceafill | th1a no | 18:34 |
| replaceafill | i mean, yes first time | 18:34 |
| replaceafill | no, we dont have styles yet | 18:34 |
| th1a | http://www.ubuntu.com/download/ubuntu/download | 18:34 |
| replaceafill | http://img706.imageshack.us/img706/7391/pantallazoiy.png | 18:36 |
| replaceafill | ff 3.5 and chrome differences | 18:36 |
| th1a | SO they aren't styling them very agressively. | 18:36 |
| th1a | Should we just add the "Focused input" CSS on p. 22? | 18:37 |
| replaceafill | just two styles in ubuntu.com: | 18:37 |
| replaceafill | width and vertical-align | 18:37 |
| replaceafill | th1a we have focused input | 18:38 |
| replaceafill | for textlines | 18:38 |
| th1a | Oh. | 18:38 |
| th1a | Sorry. | 18:38 |
| th1a | Looking at the wrong thing. | 18:38 |
| th1a | Text input/select fields on p. 22. | 18:38 |
| th1a | Just has font-size and and color. | 18:38 |
| th1a | So you could set that but, essentially, nothing to see here. | 18:38 |
| replaceafill | right | 18:39 |
| th1a | Now... | 18:39 |
| th1a | What's the style of the text here (ID, ethnicity, etc)? | 18:39 |
| replaceafill | simple link | 18:40 |
| replaceafill | <a> | 18:40 |
| replaceafill | color and text-decoration=none only | 18:40 |
| replaceafill | no font-size for example (because we removed it) | 18:41 |
| th1a | OK, well, essentially we don't want text just floating like this. Put it in a table, aelkner. | 18:41 |
| th1a | I suppose the point is this text doesn't really have a style, which should be a red flag. | 18:42 |
| replaceafill | th1a well, it's just <div><a>...</a></div> | 18:42 |
| th1a | It should always have some kind of specific role: paragraph, header, list item, etc. | 18:42 |
| th1a | Table text, form label, etc. | 18:43 |
| th1a | Following me, aelkner? | 18:43 |
| replaceafill | correct | 18:43 |
| th1a | There's no such thing as "just text." | 18:43 |
| th1a | And when in doubt: tables. | 18:44 |
| th1a | The buttons need to be styled. Shouldn't that be automatic? | 18:44 |
| replaceafill | again, they're just <input>s in a <form> | 18:45 |
| replaceafill | for them to be styled need to be placed in a: | 18:45 |
| replaceafill | <div class="buttons"> container | 18:45 |
| replaceafill | or <div class="controls"> | 18:45 |
| replaceafill | btw, we should get rid of one... | 18:45 |
| replaceafill | not one button, i mean one of those containers... | 18:46 |
| replaceafill | aelkner do we really need this in the templates: | 18:46 |
| th1a | Also, the table should have title and ID columns (title first), and neither of them need to be links. | 18:46 |
| replaceafill | <tal:dummy condition="view/update"> </tal:dummy> | 18:46 |
| th1a | btw aelkner just did what I told him to do. I'm not picking on him! | 18:47 |
| th1a | Even though it seems like it. | 18:47 |
| replaceafill | does reorder.html work? | 18:48 |
| th1a | It used to. | 18:48 |
| replaceafill | i set placeofbirth to 1 | 18:48 |
| replaceafill | and it gets back to 4 | 18:48 |
| aelkner | replaceafill, it never worked | 18:49 |
| replaceafill | ah!?!? | 18:49 |
| aelkner | that is, the number is off by one | 18:49 |
| th1a | ? | 18:49 |
| replaceafill | S: | 18:49 |
| aelkner | if you change to 3, it will put it in slot 2 | 18:49 |
| aelkner | that would be a bug from the old skin, not flourish | 18:49 |
| th1a | Why does it do that? | 18:49 |
| replaceafill | but http://69.164.203.135:6660/demographics/reorder.html | 18:50 |
| replaceafill | doesnt work at all | 18:50 |
| aelkner | zero-based vs one-based i'm guessing | 18:50 |
| replaceafill | if i set 4 to be 3, gets back to 3 | 18:50 |
| replaceafill | i mean, 4 | 18:50 |
| aelkner | what about setting 4 to 1 | 18:50 |
| aelkner | does it go to 2? | 18:50 |
| replaceafill | nothing goes nowhere | 18:51 |
| aelkner | or does setting it to 2 go to 1? | 18:51 |
| aelkner | ok, i'll look into it | 18:51 |
| aelkner | but you guys were asking question/making comments before | 18:51 |
| aelkner | one at a time | 18:51 |
| aelkner | th1a, what text needs to go in a table? | 18:51 |
| aelkner | replaceafill, +1 on deciding between controls and buttons | 18:51 |
| aelkner | we can wait for yvl to discuss this | 18:52 |
| aelkner | th1a, i'll put the buttons in one of the containers for now | 18:52 |
| th1a | Containers? | 18:52 |
| replaceafill | aelkner use buttons | 18:53 |
| th1a | The table should have Order | Title | ID | 18:53 |
| aelkner | <div class="buttons"> is the container div | 18:53 |
| aelkner | replaceafill, ok | 18:53 |
| aelkner | replaceafill, did you rename ExpandedPAge to NoSidebarPage yet? | 18:53 |
| replaceafill | nope | 18:53 |
| aelkner | it sounded like you were in the middle of doing it before | 18:54 |
| replaceafill | i was in the middle... :) | 18:54 |
| replaceafill | i'm still in the middle... | 18:54 |
| replaceafill | question: | 18:54 |
| aelkner | th1a, Order | Title | ID as headings or just the order of the columns? | 18:54 |
| aelkner | and what about the checkbox? | 18:55 |
| replaceafill | th1a we're asumming javascript enabled in the browser, correct? | 18:55 |
| th1a | Yes. | 18:55 |
| replaceafill | why do we need the two buttons in reorder.html? | 18:55 |
| th1a | :-D | 18:55 |
| aelkner | th1a, what about my question? | 18:56 |
| aelkner | questions | 18:57 |
| aelkner | replaceafill, we don't | 18:57 |
| th1a | Oh... hm... | 18:57 |
| aelkner | since the reorder happens upon changing the dropdown | 18:57 |
| th1a | I haven't been thinking about the delete aspect of this. | 18:57 |
| th1a | Do we have to have two buttons? | 18:58 |
| th1a | If you reorder and hit delete, does nothing happen? | 18:59 |
| th1a | And if you check to delete and hit apply nothing happens? | 18:59 |
| replaceafill | yeah, delete is broken too | 18:59 |
| th1a | Can we have one submit button that does both? | 19:00 |
| th1a | (and works) | 19:00 |
| replaceafill | shouldn't Delete be on a "Manage fields" view or something? | 19:01 |
| replaceafill | not on "Reorder fields" | 19:01 |
| replaceafill | maybe "Rearrange fields" | 19:01 |
| th1a | Well, yes. | 19:02 |
| th1a | I was kind of overlooking that whole aspect. | 19:02 |
| th1a | But we can change the link and page title. | 19:02 |
| replaceafill | th1a what about Delete buttons in each row? | 19:02 |
| replaceafill | like in relationship views | 19:02 |
| replaceafill | to delete | 19:02 |
| th1a | Wait. | 19:02 |
| replaceafill | sorry | 19:03 |
| aelkner | i loaded a couple fixes into the instance | 19:03 |
| th1a | Maybe I need to actually try this form too. | 19:03 |
| aelkner | 1) the buttons are in a buttons div | 19:03 |
| aelkner | 2) the reorder form action make reordering work again | 19:03 |
| aelkner | still one-based vs zero-based | 19:03 |
| th1a | Wait... what does "Apply" do? | 19:03 |
| aelkner | delete crashes | 19:03 |
| th1a | (in theory) | 19:03 |
| th1a | ? | 19:03 |
| aelkner | Apply does noting | 19:04 |
| replaceafill | th1a in theory | 19:04 |
| th1a | Ah! | 19:04 |
| replaceafill | apply works for no javascript scenario | 19:04 |
| replaceafill | (i think) | 19:04 |
| replaceafill | you select the values in the dropdown | 19:04 |
| aelkner | ah | 19:04 |
| replaceafill | and dont get redirected | 19:04 |
| aelkner | maybe | 19:04 |
| th1a | Apply is "Done" | 19:04 |
| replaceafill | hhmm | 19:04 |
| aelkner | replaceafill, i think you're right about apply | 19:04 |
| replaceafill | Apply should redirect to the same page | 19:05 |
| replaceafill | Done redirects to another one, right? | 19:05 |
| aelkner | we must assume javascript for now on, so that button has no use | 19:05 |
| aelkner | Done could go back to container view | 19:05 |
| aelkner | i meant Apply had no use, just in case the thread got confusing here | 19:06 |
| th1a | In theory, this is changing the order in the database as soon as you change the drop-down? | 19:06 |
| replaceafill | th1a correct | 19:06 |
| aelkner | in practice | 19:06 |
| aelkner | :) | 19:06 |
| replaceafill | :D | 19:06 |
| th1a | Could it not do that? | 19:06 |
| replaceafill | then you'd need Apply ;) | 19:06 |
| aelkner | i went to the doctor and said 'it hurts when i do this' | 19:06 |
| aelkner | know what he said? | 19:06 |
| th1a | OK... | 19:07 |
| aelkner | th1a, so no submit on change and keep apply? | 19:07 |
| th1a | backing up... | 19:07 |
| th1a | Wait. | 19:07 |
| th1a | OK. | 19:07 |
| aelkner | though | 19:07 |
| th1a | Trying to remember some principles here. | 19:07 |
| th1a | OK. Delete. | 19:07 |
| aelkner | yeah, let's deal with delete | 19:07 |
| th1a | Here's the thing I'm forgetting. | 19:07 |
| th1a | We don't want to do delete this way. | 19:07 |
| th1a | So we switch to a delete icon in the table rows. | 19:08 |
| th1a | Reorder live (as now) | 19:08 |
| replaceafill | yay "minus" buttons! | 19:08 |
| th1a | and have a done button. | 19:08 |
| th1a | Done link! | 19:08 |
| th1a | That's it. | 19:08 |
| aelkner | i figured we were heading there | 19:08 |
| aelkner | to repeat | 19:09 |
| th1a | So Order | Title | ID | Delete | 19:09 |
| aelkner | go on | 19:09 |
| th1a | Those are the columns. | 19:09 |
| th1a | Delete has icons. | 19:09 |
| aelkner | like relationship views, right? | 19:09 |
| th1a | Y. | 19:09 |
| aelkner | delete icons | 19:10 |
| aelkner | sorry, thought that was a ?, not a Y | 19:10 |
| aelkner | a question about delete (here or in relationship views) | 19:10 |
| aelkner | yvl always clamours for confirmation step | 19:11 |
| aelkner | in the past, at least | 19:11 |
| aelkner | what's your feeling? | 19:11 |
| th1a | Don't do it yet. | 19:11 |
| th1a | We can do it with jquery if we want. | 19:11 |
| aelkner | if we could come up (later) with a modal for confirmation | 19:11 |
| th1a | YEs. | 19:12 |
| aelkner | that wold be good | 19:12 |
| aelkner | anyway, directly delete for now | 19:12 |
| aelkner | and directly reorder | 19:12 |
| th1a | Yes. | 19:12 |
| aelkner | and Done is the only button, taking user back to table | 19:12 |
| th1a | Link, not button. | 19:12 |
| aelkner | link | 19:12 |
| aelkner | about the four columns | 19:13 |
| th1a | The relationship management views are your guide style-wise. | 19:13 |
| aelkner | sure | 19:13 |
| aelkner | title and id, neither are links? | 19:13 |
| aelkner | no need to be able to edit from here? | 19:13 |
| th1a | I don't think it is helpful in this case. | 19:13 |
| aelkner | ok | 19:14 |
| replaceafill | aelkner please do me a favor | 19:14 |
| aelkner | yes | 19:14 |
| aelkner | pep8? | 19:14 |
| replaceafill | use <table class="demographics-reoder-table"> | 19:14 |
| replaceafill | and <th class="id"> | 19:14 |
| replaceafill | <th class="delete"> | 19:14 |
| aelkner | what's with the minus signs, btw? | 19:15 |
| aelkner | replaceafill, do you realize that if we don't keep consistent with using _ or - | 19:15 |
| aelkner | then people like me waste needless time choosing the wrong ne | 19:15 |
| aelkner | one | 19:15 |
| aelkner | from time to time | 19:15 |
| aelkner | things like that do matter, you know | 19:16 |
| aelkner | developer productivity and all | 19:16 |
| replaceafill | good point | 19:16 |
| aelkner | i'd choose _ over minus because it is used for python varaibles | 19:17 |
| aelkner | so it's consistent even across longuages | 19:17 |
| aelkner | but at least let's choose one | 19:17 |
| aelkner | we can talk about wich one with yvl tomorrow | 19:17 |
| replaceafill | i'd do too | 19:17 |
| replaceafill | go with '_' | 19:17 |
| aelkner | ok, we can agree on that for now, great | 19:17 |
| aelkner | so you want me to add that class to the table | 19:18 |
| replaceafill | yes | 19:18 |
| replaceafill | and classes for the <th> | 19:18 |
| aelkner | what's up with that, styling column widths? | 19:18 |
| replaceafill | yep | 19:18 |
| aelkner | where's the css? | 19:18 |
| aelkner | table.css? | 19:18 |
| replaceafill | we don't have it yet | 19:18 |
| aelkner | ah | 19:18 |
| replaceafill | it'll be there if we think we need it | 19:19 |
| aelkner | ok | 19:19 |
| replaceafill | just put the styles for now | 19:19 |
| replaceafill | thanks | 19:19 |
| replaceafill | sorry, the classes not the styles | 19:19 |
| aelkner | table: demographics_reoder_table | 19:19 |
| aelkner | th, column1: ? | 19:19 |
| aelkner | th, column2: ? | 19:19 |
| aelkner | you tell me | 19:19 |
| replaceafill | what are the columns again? | 19:20 |
| aelkner | Order | Title | ID | Delete | 19:20 |
| replaceafill | order, title, id, action | 19:20 |
| replaceafill | and we should use Remove instead of Delete | 19:21 |
| replaceafill | right th1a? | 19:21 |
| th1a | Ah.. is that what we did before? | 19:22 |
| replaceafill | yes | 19:22 |
| th1a | Well, in that case we weren't literally deleting the person. | 19:22 |
| th1a | So it was more accurate. | 19:22 |
| replaceafill | ah ok | 19:22 |
| th1a | If you remove Jeff Elkner as a contact you aren't deleting him. | 19:22 |
| aelkner | right | 19:23 |
| th1a | In this case you are deleting the field. | 19:23 |
| replaceafill | so Delete means gone forever | 19:23 |
| aelkner | yes | 19:23 |
| replaceafill | got it | 19:23 |
| th1a | More to the point, you're deleting the object. | 19:23 |
| replaceafill | we should use different icons... | 19:23 |
| th1a | You don't delete a relationship. | 19:23 |
| aelkner | what about th class names? | 19:23 |
| th1a | It isn't a temporal distinction, but one of type. | 19:23 |
| replaceafill | aelkner i typed them above | 19:23 |
| th1a | Type not permanence. | 19:24 |
| aelkner | removing a relationship | 19:24 |
| aelkner | unlinking | 19:24 |
| replaceafill | aelkner order, title, id, action | 19:24 |
| aelkner | i can name classes that generically? | 19:24 |
| aelkner | just because you will use css rules? | 19:24 |
| replaceafill | aelkner you're setting a class on the table | 19:25 |
| replaceafill | we're setting classes on almost all the tables | 19:25 |
| aelkner | demographics_reoder_table title{ | 19:26 |
| aelkner | some css | 19:26 |
| aelkner | } | 19:26 |
| replaceafill | dont forget the dots starting the class names ;) | 19:26 |
| replaceafill | .demographics_reorder_table .title | 19:26 |
| replaceafill | but yes | 19:26 |
| aelkner | i see | 19:27 |
| aelkner | so gong forward, as long as we have some container class for our local css situation | 19:27 |
| aelkner | like demographics_reorder_table | 19:27 |
| aelkner | we are free to use generic (i.e. short and easy to use) class names | 19:27 |
| aelkner | for anything element contained therein | 19:28 |
| aelkner | since the css rules will sort it out | 19:28 |
| replaceafill | it depends on how the table looks | 19:28 |
| aelkner | go on | 19:28 |
| replaceafill | if we can let it to the browser... | 19:28 |
| replaceafill | for example | 19:29 |
| replaceafill | in some of the relationships views | 19:29 |
| replaceafill | contacts | 19:29 |
| replaceafill | have first name, last name, relationship, action | 19:29 |
| replaceafill | and the table below just first name, last name, action | 19:29 |
| replaceafill | they looked funny with no styles | 19:29 |
| replaceafill | so, i set widths for some columns | 19:30 |
| replaceafill | they still look funny, but less :) | 19:30 |
| aelkner | well, i anticipated the need for column styles for widths in tables in general | 19:31 |
| replaceafill | it's difficult to do it generally | 19:31 |
| aelkner | but i was wondering about the coding pattern | 19:31 |
| replaceafill | since tables have so little space and different columns | 19:31 |
| replaceafill | your table won't be at all like: first_name, last_name, action | 19:32 |
| replaceafill | th1a my tasks for the rest of the day: rename the expanded page, change our styles to use "_" instead of "-", review menesis merge request (which includes hiding empty rows in person accordions) and the password form | 19:34 |
| th1a | Sounds good, replaceafill. | 19:35 |
| menesis | my merge is a few very simple changes, I just didn't want to interfere | 19:35 |
| replaceafill | menesis not at all | 19:35 |
| replaceafill | i'll set it up so th1a see it | 19:35 |
| th1a | aelkner: Basically, your next task is the same thing for "Resource attributes" | 19:35 |
| th1a | I never want to hear the term "resource demographics" again! ;-) | 19:36 |
| menesis | I saw you talking about password form, but I just did that a few minutes earlier :) | 19:36 |
| aelkner | th1a, :) | 19:36 |
| replaceafill | menesis thanks :) | 19:36 |
| aelkner | Resource custom attributes? | 19:36 |
| aelkner | just throwing it out there | 19:36 |
| replaceafill | resource demographics is banned link "action buttons" :D | 19:37 |
| replaceafill | *like | 19:37 |
| aelkner | action links, right | 19:37 |
| replaceafill | aelkner i guess you missed my comment: | 19:40 |
| replaceafill | do we really need <tal:dummy condition="view/update"> </tal:dummy> in the new templates? | 19:40 |
| aelkner | um, maybe not, which one? | 19:43 |
| replaceafill | f_demographics_reorder.pt | 19:43 |
| aelkner | i mean, when update needs to be alled? | 19:43 |
| aelkner | is it always called now? | 19:43 |
| replaceafill | flourish pages do that for you | 19:43 |
| aelkner | ok, i'll remember that, thanks | 19:43 |
| aelkner | th1a, if i'm changing the title link to now be two columns, title and id, neither links | 19:51 |
| aelkner | wouldn't it help for the tale to have headings? | 19:51 |
| aelkner | table | 19:51 |
| th1a | Yes, it should. | 19:51 |
| aelkner | ok | 19:52 |
| th1a | It has gotten more complicated. | 19:52 |
| replaceafill | th1a question | 19:58 |
| replaceafill | do you want the sidebar rendered in the reorder view | 19:58 |
| replaceafill | http://69.164.203.135:6660/demographics/reorder.html | 19:58 |
| th1a | No. | 19:58 |
| replaceafill | they should only be available in /demographics, correct? | 19:59 |
| replaceafill | wtf?!?! i'm trying to check my email typing: manager/schooltool | 20:00 |
| replaceafill | :D | 20:00 |
| th1a | yes | 20:01 |
| replaceafill | aelkner you should limit the demographics_fields and demographics_actions managers to the /demographics view | 20:01 |
| replaceafill | just adding: view=".demographics.FlourishDemographicsView" | 20:01 |
| replaceafill | to the viewlet directive | 20:02 |
| replaceafill | changes pushed to trunk: "Changed ExpandedPage class name to NoSidebarPage. Also conditioned sidebar rendering" | 20:07 |
| *** menesis has quit IRC | 20:09 | |
| replaceafill | th1a new task for me: make the error messages in forms to look the same without extra markup | 20:20 |
| th1a | sounds good replaceafill | 20:27 |
| replaceafill | change pushed to trunk: "Removed label from contact edit form and styled contact forms errors" | 20:33 |
| * replaceafill realizes jquery and zc.table use - for classes... | 20:59 | |
| * th1a goes to see if he can get some new glasses. | 20:59 | |
| replaceafill | for tomorrow meeting: should we change our css class names to use _ instead of - | 21:00 |
| *** menesis has joined #schooltool | 21:01 | |
| *** ignas has quit IRC | 21:25 | |
| aelkner | th1a, new reorder view load in demo server | 22:02 |
| aelkner | btw, the change order thing works perfectly | 22:02 |
| aelkner | so i guess the whole zero-base/one-based discussion was way off base :) | 22:03 |
| aelkner | perhaps i misunderstood the behaviour at a time when it wasn't working for other reasons | 22:03 |
| aelkner | but it works fine now | 22:03 |
| replaceafill | aelkner remember to set the sidebars only for the /demographics view | 22:09 |
| replaceafill | nice work on the reorder.html btw :) | 22:09 |
| aelkner | thanks | 22:11 |
| aelkner | how do i set the sidebars only for the one view? | 22:11 |
| aelkner | i didn't see that in the person-container case | 22:11 |
| replaceafill | i told you before.. :/ | 22:11 |
| replaceafill | just adding: view=".demographics.FlourishDemographicsView" | 22:12 |
| aelkner | in the manager, right? | 22:12 |
| replaceafill | to the demographics_fields and demographics_actions managers | 22:12 |
| aelkner | ok | 22:12 |
| aelkner | replaceafill, now it has the blank grey sidebar | 22:14 |
| replaceafill | have you sync WITH trunk? | 22:14 |
| aelkner | not yet | 22:14 |
| replaceafill | it should be gone when you sync | 22:15 |
| aelkner | loaded in the server | 22:19 |
| replaceafill | th1a http://69.164.203.135:6660/persons/alvaro | 22:28 |
| replaceafill | i applied the part of menesis patch that hides empty fields in accordions | 22:28 |
| aelkner | do i need to sync again? | 22:29 |
| replaceafill | aelkner no, this is not in trunk yet | 22:29 |
| aelkner | i pushed my changes to trunk | 22:33 |
| replaceafill | th1a we need to fix the bound contact page (contact information on persons) | 22:37 |
| * replaceafill goes to get lunch | 22:40 | |
| * th1a is back. | 22:41 | |
| th1a | 20/25 vision. | 22:41 |
| replaceafill | ah! | 22:41 |
| * replaceafill stops from going for lunch | 22:41 | |
| replaceafill | th1a a couple of questions | 22:41 |
| replaceafill | should "password edit" be modal? | 22:42 |
| th1a | Nice. | 22:42 |
| th1a | No... we though it would be weird since it logs you out, right? | 22:42 |
| replaceafill | it depends | 22:42 |
| replaceafill | not if you're the manager | 22:43 |
| replaceafill | which brings me to my second question | 22:43 |
| replaceafill | maybe we need a "change my own password" form and a "reset the password for this user" form | 22:43 |
| th1a | Ah, yes. | 22:43 |
| replaceafill | change my own password should ask you for your current password | 22:43 |
| th1a | Indeed. | 22:43 |
| replaceafill | and the two new passwdx | 22:43 |
| *** menesis has quit IRC | 22:44 | |
| replaceafill | reset the password for this user should be for admins only | 22:44 |
| replaceafill | requiring just the 2 new pass | 22:44 |
| replaceafill | change my own password should warn you about the log out | 22:45 |
| th1a | Yes. | 22:45 |
| th1a | You can have that project replaceafill. ;-) | 22:45 |
| replaceafill | :D | 22:45 |
| replaceafill | label for the "reset the password for this user" form? | 22:46 |
| replaceafill | in Settings in the person index? | 22:46 |
| replaceafill | or Actions? ;) | 22:46 |
| th1a | I think it can have the same label but different behaviours. | 22:48 |
| th1a | But the form should be... informative. | 22:48 |
| replaceafill | same label? like: | 22:49 |
| replaceafill | Settings | 22:49 |
| replaceafill | - Preferences | 22:49 |
| replaceafill | - Password | 22:49 |
| replaceafill | - Password | 22:49 |
| replaceafill | ? | 22:49 |
| replaceafill | ah! | 22:49 |
| replaceafill | sorry | 22:49 |
| replaceafill | i meant the link for the form | 22:49 |
| replaceafill | :( | 22:49 |
| * replaceafill just read his own question... | 22:49 | |
| replaceafill | i mean the title for the link to the reset password form | 22:50 |
| th1a | I know. | 22:50 |
| th1a | You just have to not duplicate it if the active user is looking at his own page. | 22:50 |
| th1a | Right? | 22:50 |
| replaceafill | ah, smart link | 22:51 |
| replaceafill | ok | 22:51 |
| th1a | I think you can do that kind of logic with some marbles and tinkertoys. | 22:51 |
| replaceafill | :D | 22:51 |
| th1a | I can probably get those bolted onto the server. | 22:51 |
| replaceafill | ¬¬ | 22:51 |
| replaceafill | last unnanounced question | 22:52 |
| replaceafill | i merged menesis fix about hiding empty fields in accordions | 22:52 |
| replaceafill | http://69.164.203.135:6660/persons/alvaro/ | 22:52 |
| th1a | Yes, looks good. | 22:52 |
| replaceafill | it helped me to find out that we need to fix contact info for persons | 22:52 |
| replaceafill | we have contact info on contacts | 22:53 |
| replaceafill | but check Sally Cunninham | 22:53 |
| replaceafill | in alvaros page | 22:53 |
| replaceafill | Sally is a person | 22:53 |
| replaceafill | who is also a contact | 22:53 |
| replaceafill | persons can have "bound" contact info | 22:53 |
| replaceafill | or "self" | 22:53 |
| th1a | Oh, right, yes, we forgot that! | 22:53 |
| th1a | Yes. | 22:53 |
| replaceafill | how do we get to that form_ | 22:54 |
| replaceafill | from the person view? | 22:54 |
| th1a | Hm... | 22:54 |
| th1a | Well, it could just be an action on manage contacts. | 22:55 |
| replaceafill | are we going to add an accordion for that? | 22:56 |
| th1a | Or it could be listed as an editable default contact if you wanted a shortcut. | 22:56 |
| th1a | No. | 22:56 |
| th1a | I mean, we could. | 22:56 |
| th1a | But in K-12 schools you rarely actually use it. | 22:56 |
| replaceafill | ah ok | 22:56 |
| th1a | Let's start with it as an action on manage contacts. | 22:57 |
| th1a | Add | 22:57 |
| th1a | - New Contact | 22:58 |
| th1a | - hm... | 22:58 |
| th1a | Hm... | 23:00 |
| replaceafill | ok, i'll add it to my queue, will go for lunch | 23:02 |
| * replaceafill goes away for 1h | 23:02 | |
| th1a | Add | 23:03 |
| th1a | -- $Person as Contact? | 23:03 |
| replaceafill | :| | 23:03 |
| replaceafill | seems strange | 23:03 |
| th1a | It is all a bit strange. | 23:03 |
| replaceafill | :) | 23:04 |
| th1a | I don't want to add an accordion that will be useless most of the time. | 23:04 |
| replaceafill | i agree | 23:04 |
| th1a | Although perhaps several of them will be for specific users. | 23:04 |
| replaceafill | what if we mix contact data with the General information data? | 23:05 |
| replaceafill | in the first accordion? | 23:05 |
| replaceafill | we hide emtpy fields now anyway | 23:05 |
| th1a | Yeah, but we don't want to encourage people to do contacts incorrectly either. | 23:05 |
| th1a | That's the catch. | 23:05 |
| th1a | tbh $PERSON as Contact might be best. | 23:06 |
| replaceafill | ah ok, i'll use that | 23:06 |
| th1a | I'd actually rather have it be a little confusing than lead people to use the application incorrectly. | 23:06 |
| replaceafill | actually, looking it all in the "manage contact" view "add $person as contact" makes sense | 23:08 |
| th1a | It is accurate. | 23:09 |
| * th1a goes to make dinner. | 23:13 | |
| *** replaceafill has quit IRC | 23:33 | |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!
