| *** ignas has quit IRC | 00:22 | |
| *** aks has joined #schooltool | 06:13 | |
| *** aks has joined #schooltool | 06:13 | |
| *** ignas has joined #schooltool | 09:39 | |
| *** menesis has joined #schooltool | 10:15 | |
| *** ignas has quit IRC | 11:47 | |
| *** ignas has joined #schooltool | 12:29 | |
| *** menesis has quit IRC | 12:30 | |
| *** menesis has joined #schooltool | 13:16 | |
| *** aks has quit IRC | 14:34 | |
| *** replaceafill has joined #schooltool | 16:05 | |
| *** th1a has joined #schooltool | 16:10 | |
| *** menesis has quit IRC | 16:12 | |
| *** menesis has joined #schooltool | 16:17 | |
| th1a | dontfearthereaper | 16:21 |
|---|---|---|
| th1a | oops | 16:21 |
| * th1a changes his gmail password... | 16:24 | |
| *** replaceafill has quit IRC | 16:28 | |
| *** replaceafill has joined #schooltool | 16:29 | |
| aelkner | th1a, just don't make it blueoystercolt :) | 16:31 |
| th1a | hi replaceafill, aelkner, menesis. | 16:31 |
| replaceafill | good morning/afternoon | 16:31 |
| menesis | hi | 16:31 |
| aelkner | morning all | 16:31 |
| th1a | aelkner: I think it would be fine with your typo. | 16:31 |
| aelkner | how do you spell it? | 16:32 |
| aelkner | oh, cult | 16:32 |
| th1a | ;-) | 16:32 |
| th1a | OK... | 16:33 |
| th1a | Where are we? | 16:34 |
| * th1a is trying to get his head back in the game after a week in Maine and 36 innings of base ball on Long Island. | 16:34 | |
| th1a | replaceafill? | 16:34 |
| replaceafill | i'm working on the Application Control views | 16:35 |
| replaceafill | zodb, translations domains and generations | 16:35 |
| replaceafill | i also have to fix the student form in Cambodia | 16:36 |
| replaceafill | i forgot to include some fields when i moved them to use the demographics fields | 16:36 |
| replaceafill | parents data | 16:37 |
| th1a | Ah, ok. That's the bug Chandara was referring to? | 16:39 |
| replaceafill | yes | 16:40 |
| th1a | OK. | 16:41 |
| th1a | We should be wrapping up the oddball views soon then? | 16:42 |
| replaceafill | th1a yes | 16:42 |
| replaceafill | today i think | 16:42 |
| th1a | Good. | 16:42 |
| th1a | I just slowed you down a little improving the email views. | 16:43 |
| replaceafill | ah, i need to work on the "Errors" view too | 16:43 |
| th1a | OK. | 16:43 |
| replaceafill | it's at /++etc++site/default/RootErrorReportingUtility | 16:43 |
| th1a | Yes. | 16:44 |
| replaceafill | th1a that's it from me :) | 16:45 |
| th1a | OK. | 16:45 |
| th1a | Here's a question for the audience. | 16:46 |
| th1a | Does anyone remember why "teaching levels" are in years? | 16:46 |
| aelkner | replaceafill should know that one | 16:46 |
| replaceafill | aelkner ? | 16:46 |
| aelkner | :) | 16:46 |
| aelkner | i thought we did that for cambodia | 16:46 |
| replaceafill | cambodia use them :) | 16:47 |
| aelkner | any given year can decide to have differnet levelsw | 16:47 |
| replaceafill | yes | 16:47 |
| aelkner | like if the school makes a policy change, right? | 16:47 |
| aelkner | SLA only had freshmen when it started | 16:48 |
| aelkner | the next year it had two levels | 16:48 |
| th1a | Nah... | 16:48 |
| th1a | Doing them by year is overkill. | 16:48 |
| aelkner | th1a, i must warn you, making data model changes at this point will be costly | 16:49 |
| th1a | We should probably table that view until we see if we have time to fix the whole thing. | 16:49 |
| aelkner | right, until the release | 16:49 |
| th1a | I don't think it would be that costly. | 16:50 |
| th1a | Because we don't have to respect the data at this point. | 16:50 |
| aelkner | there's a lot of code that depends on the data model | 16:50 |
| th1a | What? | 16:50 |
| aelkner | i don't think you are taking the cost of changing code into consideration | 16:50 |
| th1a | Do levels do anything? | 16:50 |
| replaceafill | not much related to levels though | 16:51 |
| menesis | levels do nothing | 16:51 |
| menesis | except in cambodia | 16:51 |
| th1a | If we made a simple mistake on a system that does nothing, we should just fix it. | 16:51 |
| aelkner | well, there are still lines of code out there in the levell package | 16:51 |
| menesis | just skip levels now | 16:52 |
| aelkner | considering the limited time we have left before the release | 16:52 |
| aelkner | and the many tasks that we haven't even discussed but that are vital | 16:53 |
| aelkner | i think it would be a mistake to add data model work, even if it were a small three day task | 16:53 |
| th1a | The more immediate point is that I don't want to write the levels views right now, and we'll decide later. | 16:54 |
| aelkner | +1 for skipping levels | 16:54 |
| th1a | It is simply a question of how much time we have. | 16:55 |
| th1a | Moving on... | 16:55 |
| th1a | aelkner, where are you at this point. | 16:55 |
| aelkner | ok, so i worked on the schoolyear add/edit widget discriminator errors | 16:56 |
| aelkner | those errors that used to be at the top, but needed to be next to the widget | 16:56 |
| aelkner | overlapping year, first day after last day, etc. | 16:56 |
| aelkner | that was tricky, but i finally got it to work | 16:57 |
| aelkner | then i coded the activate school year view | 16:57 |
| aelkner | i didn't do it as a modal because i thought maybe you would reconsider after seeing it | 16:57 |
| aelkner | i mean, if a school has been around for long enough, then listing all the years in a modal would be ugly | 16:58 |
| th1a | ok. | 16:58 |
| th1a | That's fine. | 16:58 |
| aelkner | then i created the /terms view | 16:58 |
| aelkner | we should go over these views after the meeting to discuss details | 16:59 |
| th1a | ok. | 16:59 |
| aelkner | that's it for me | 16:59 |
| replaceafill | aelkner in /terms those <th> should be in <thead> correct? | 17:00 |
| aelkner | shucks, i forgot that part | 17:00 |
| aelkner | anyway, we can go over this after th1a drops the bag of gravel | 17:01 |
| * th1a dropd the bag of gravel. | 17:02 | |
| th1a | drops | 17:02 |
| aelkner | change in instance: | 17:03 |
| aelkner | http://69.164.203.135:36660/terms | 17:03 |
| aelkner | th1a, we don't have many views for terms yet, so i had to change an older version of the DATA.fs | 17:04 |
| aelkner | because evolution prevents us from modifying any of our demo data with old schooltool anymore | 17:04 |
| aelkner | i hope you don't miss any data you set up in my instance | 17:04 |
| th1a | i didn't. | 17:05 |
| aelkner | ok | 17:05 |
| th1a | Can we do subheads by year? | 17:05 |
| aelkner | definitely | 17:06 |
| aelkner | no accordions? | 17:06 |
| aelkner | it would be easy to add them | 17:06 |
| th1a | no. | 17:06 |
| aelkner | ok | 17:06 |
| aelkner | so, no year column, subheads for year, colspan=3 so that it centers all the way across? | 17:06 |
| th1a | Yes. | 17:07 |
| aelkner | ok, i'll need a half hour or so for that, could you play around with years in the meantime? | 17:08 |
| th1a | yes | 17:09 |
| aelkner | no link for the year i guess, right? | 17:10 |
| aelkner | i mean if it's in a <th>, do we want it to be a link? | 17:10 |
| th1a | no | 17:11 |
| th1a | The error message in the year form is a bit misaligned. | 17:13 |
| th1a | If you have a school year that ends before it begins. | 17:13 |
| replaceafill | i guess it's because the [...] button of the calendar widget | 17:14 |
| replaceafill | we're getting rid of it, right? | 17:15 |
| replaceafill | when we change to jquery datepicker | 17:15 |
| th1a | Well, I guess we should be doing that now. | 17:15 |
| aelkner | http://69.164.203.135:36660/terms | 17:18 |
| aelkner | should we have a 'there a no terms' message for years with no terms? | 17:19 |
| aelkner | also, year date range next to ttle in parens? | 17:19 |
| aelkner | i see you added a test year | 17:19 |
| th1a | Yes. | 17:20 |
| th1a | I want to limit years to 12 characters. | 17:20 |
| aelkner | to both? | 17:20 |
| aelkner | i forgot, you did ask me to limit years to 12 | 17:21 |
| th1a | Probably "This year has no terms defined." would be good. | 17:22 |
| aelkner | th1a, http://69.164.203.135:36660/terms | 17:33 |
| th1a | OK. | 17:36 |
| th1a | Looks good. | 17:36 |
| aelkner | replaceafill, what's the best way to limit the size of a TextLine field? | 17:44 |
| aelkner | i can't find anything like max_chars in zope.schema | 17:44 |
| replaceafill | max_length | 17:46 |
| replaceafill | if i remember correctly | 17:46 |
| aelkner | yeah, i think i need to change the field | 17:48 |
| aelkner | fields['title'].max_length = 12 | 17:48 |
| aelkner | that didn't work | 17:50 |
| replaceafill | aelkner try setting .maxlength on the widget | 17:56 |
| aelkner | still nothing | 17:57 |
| aelkner | what is the <input> element attribute for maxlength? | 17:57 |
| aelkner | is it with or without the underscore? | 17:58 |
| replaceafill | check z3c.form.browser.interfaces. IHTMLTextInputWidget | 17:58 |
| aelkner | ah, i changed the wrong interface | 17:59 |
| aelkner | :) | 17:59 |
| * replaceafill goes to get breakfast | 18:00 | |
| * aelkner does the same | 18:15 | |
| th1a | replaceafill, aelkner, you back? | 18:45 |
| replaceafill | th1a yes | 18:45 |
| th1a | It looks like I have to go get a new tire. | 18:45 |
| aelkner | th1a, i'm back | 18:50 |
| aelkner | using pdb to figure out why maxlength doesn't get rendered | 18:50 |
| replaceafill | aelkner did you set it in updateWidgets in your form? | 18:51 |
| aelkner | in other words, i have stuff to do til you get back | 18:51 |
| aelkner | no, didn't think to do that | 18:51 |
| aelkner | have we done that before? | 18:52 |
| replaceafill | try it | 18:52 |
| aelkner | trying... | 18:52 |
| * th1a goes to get a new tire. | 18:57 | |
| aelkner | replaceafill, that works! | 18:59 |
| replaceafill | :) | 18:59 |
| aelkner | th1a, have you left yet? | 18:59 |
| aelkner | np, i can do breadcrumbs for now | 19:04 |
| aelkner | replaceafill, http://69.164.203.135:36660/schoolyears/2010-2011/@@edit.html | 19:06 |
| aelkner | what am i doing wrong to not get the message coming out? | 19:06 |
| aelkner | after the red *, i mean | 19:06 |
| replaceafill | what should i do? | 19:06 |
| replaceafill | ah | 19:07 |
| replaceafill | you're setting label on the form | 19:07 |
| replaceafill | you need legend | 19:07 |
| replaceafill | or the fieldset logic | 19:07 |
| aelkner | css logic? | 19:07 |
| replaceafill | no | 19:07 |
| replaceafill | your form needs to have a fieldset | 19:07 |
| replaceafill | with a legend | 19:07 |
| aelkner | the macro, you mean, right? | 19:07 |
| aelkner | oh, sorry, no macro | 19:08 |
| replaceafill | are you using the z3c-schooltool-form.pt template? | 19:08 |
| replaceafill | the one you added legend to? | 19:08 |
| aelkner | no, add schoolyear is more complicated | 19:08 |
| aelkner | so i cloned | 19:08 |
| replaceafill | no!!!!!! | 19:09 |
| replaceafill | :D | 19:09 |
| replaceafill | (kidding) | 19:09 |
| aelkner | it does use the macro | 19:09 |
| aelkner | but it has other stuff, too | 19:09 |
| aelkner | yeah, i get your sarcasm | 19:09 |
| replaceafill | well, your fieldset needs a legend | 19:09 |
| replaceafill | :D | 19:09 |
| aelkner | i'm not going there now :) | 19:09 |
| aelkner | it's the macro that needs the legend | 19:09 |
| replaceafill | why dont you insert the legend part | 19:09 |
| aelkner | i have two fieldsets | 19:09 |
| replaceafill | ah | 19:10 |
| aelkner | the one the macro renders, and the one my template renders | 19:10 |
| aelkner | could it be a problem with having two fieldsets? | 19:10 |
| replaceafill | i only see one fieldset... | 19:10 |
| aelkner | replaceafill, sorry, i'm going nuts hear | 19:10 |
| aelkner | confusing add with edit views | 19:10 |
| replaceafill | :D | 19:10 |
| aelkner | and taking you down in the process :) | 19:10 |
| replaceafill | yes, this is the edit form with just one fieldset | 19:11 |
| aelkner | sorry for the confusion, anyway, my view needs a label, that's all | 19:12 |
| replaceafill | :) | 19:12 |
| aelkner | crap, it already has one | 19:13 |
| aelkner | replaceafill, the macro renders the label as <h3> | 19:14 |
| replaceafill | aelkner remember you included a legend attribute in the template? | 19:15 |
| replaceafill | why don't you use that? | 19:15 |
| aelkner | ah, legend | 19:15 |
| aelkner | thanks | 19:15 |
| replaceafill | :) | 19:15 |
| aelkner | replaceafill, much better: | 19:19 |
| aelkner | http://69.164.203.135:36660/schoolyears/2010-2011/@@edit.html | 19:19 |
| replaceafill | definitely :) | 19:19 |
| aelkner | so by changing the widget and not the interface, we don't need to evolve | 19:19 |
| *** menesis has quit IRC | 19:19 | |
| aelkner | because maxlength only stops the user from making the field larger | 19:20 |
| aelkner | and we don't want to punish the users who already have larger titles | 19:20 |
| aelkner | though i doubt there are many that do | 19:20 |
| aelkner | so it's a win-win | 19:21 |
| replaceafill | are we changing Apply to Submit? | 19:21 |
| aelkner | th1a, has asked me to | 19:21 |
| replaceafill | ah ok | 19:21 |
| aelkner | i think that is something he wants program wide | 19:21 |
| replaceafill | good to know | 19:21 |
| aelkner | i think he is right that people understand submit better than apply | 19:22 |
| aelkner | apply implies applying for membership or something | 19:22 |
| aelkner | it | 19:22 |
| aelkner | it's more of a techie database term, like apply the change to the data | 19:22 |
| replaceafill | right | 19:23 |
| aelkner | not everyone thinks that way | 19:23 |
| aelkner | he may have gotten the idea from the book on forms | 19:23 |
| aelkner | i bought it, but haven't read it yet | 19:23 |
| aelkner | i guess i figured that just by buying it, i would absorb all its energy :) | 19:24 |
| replaceafill | :| | 19:24 |
| aelkner | what the heck does :| mean?! | 19:26 |
| aelkner | i think you made up one there | 19:26 |
| aelkner | i checked the reference and everything | 19:27 |
| replaceafill | :) | 19:27 |
| aelkner | is that like, 'no expression' | 19:27 |
| aelkner | are you being coy? :) | 19:27 |
| replaceafill | to me :| is "ah!??!" | 19:28 |
| * replaceafill goes to take a shower | 19:28 | |
| aelkner | you know, ah is the same number of letters! | 19:29 |
| replaceafill | :D | 19:29 |
| *** ignas has quit IRC | 19:48 | |
| *** menesis has joined #schooltool | 20:15 | |
| * th1a is back. | 21:10 | |
| replaceafill | th1a http://69.164.203.135:6660/control | 21:19 |
| th1a | Zope version: Meaningless? | 21:20 |
| replaceafill | yes :) | 21:20 |
| replaceafill | it was deprecated | 21:20 |
| th1a | Can we get rid of that line? | 21:20 |
| replaceafill | def getZopeVersion(): | 21:20 |
| replaceafill | """Return a string containing the descriptive version of the | 21:20 |
| replaceafill | current zope installation. | 21:20 |
| replaceafill | 21:20 | |
| replaceafill | Deprecated: the concept of a Zope version went away in the | 21:20 |
| replaceafill | Zope Toolkit. It is unlikely this gives sensible results in | 21:20 |
| replaceafill | many situations. | 21:20 |
| replaceafill | """ | 21:20 |
| replaceafill | yes | 21:20 |
| th1a | OK. | 21:21 |
| replaceafill | gone | 21:21 |
| replaceafill | btw, those paths may split to multiple lines: http://img851.imageshack.us/img851/6014/1311013384725.png | 21:23 |
| replaceafill | i think that's ok, right? | 21:24 |
| th1a | Yes. | 21:24 |
| th1a | This is not exactly a critical screen, anyhow. | 21:24 |
| replaceafill | cool, will add the action butt.... links ;) | 21:24 |
| aelkner | th1a, can we talk about /settings and /manage? | 21:50 |
| th1a | I just put it at the top of my todo list for this week. | 21:51 |
| th1a | So yes. | 21:51 |
| aelkner | cool | 21:51 |
| aelkner | well, i don | 21:51 |
| aelkner | i don't have anything to offer up, but it would be nice to get some of these links for /schooyear, /terms, etc | 21:52 |
| aelkner | into either of the two views | 21:52 |
| aelkner | what's you thinking at the highest level about the distinction | 21:52 |
| aelkner | between the two views, what they deal with, etc.? | 21:52 |
| th1a | It is pretty much just server admin stuff in Application and school management stuff in School | 21:56 |
| th1a | I think. | 21:56 |
| th1a | I'm mainly concerned with the visual organization of each page. | 21:56 |
| th1a | That I'm not sure of. | 21:56 |
| aelkner | so far we have http://69.164.203.135:36660/settings | 21:57 |
| aelkner | and it has a blank page in the middle with some a linkset on the side with links in it | 21:58 |
| aelkner | how would the middle of the page even look? | 21:58 |
| th1a | Just pretend that does not exist. | 21:59 |
| aelkner | can we talk about what i can add and where? | 22:00 |
| th1a | We're not really at that point yet. | 22:01 |
| th1a | Actually adding the links is pretty trivial. | 22:01 |
| th1a | What are you worried about? | 22:02 |
| th1a | Just getting to the pages you're working on? | 22:02 |
| aelkner | not worried, just would like to not have to add two new bookmarks every time i deal | 22:02 |
| aelkner | with a new /something | 22:02 |
| aelkner | if i could just click on Application of School to get to the link, that would be nice for me, but not important | 22:02 |
| aelkner | what would you like me to do next? | 22:03 |
| th1a | OK... looking at years again... | 22:03 |
| th1a | Can you make the already active year selected by default? | 22:04 |
| aelkner | sure | 22:06 |
| aelkner | keep going | 22:06 |
| th1a | Sort the years in descending order. | 22:06 |
| th1a | What about that new calendar widget? | 22:07 |
| aelkner | oh, in the /schooyears view | 22:07 |
| aelkner | what about the widget? | 22:07 |
| th1a | replaceafill brought it up... should we be switching that now? | 22:07 |
| replaceafill | th1a i think yvl wanted to work on that | 22:07 |
| th1a | OK. | 22:08 |
| replaceafill | taking into account translations | 22:08 |
| replaceafill | files | 22:08 |
| th1a | And that won't need the extra button? | 22:08 |
| replaceafill | i dont think so | 22:08 |
| replaceafill | you just focus the input field | 22:08 |
| replaceafill | and the calendar comes up | 22:08 |
| th1a | Also "There were some errors." should not be our generic error message anymore. | 22:08 |
| replaceafill | i told aelkner that | 22:08 |
| th1a | replaceafill: That's what I thought. | 22:08 |
| th1a | Hint for year title form: "Limited to 12 characters or less." | 22:09 |
| th1a | Changes for new school year text: | 22:12 |
| th1a | Actually... | 22:13 |
| aelkner | replaceafill, what did you tell me? | 22:13 |
| replaceafill | about the "There were some errors" message | 22:13 |
| th1a | I guess the problem is I don't understand this form. | 22:14 |
| aelkner | replaceafill, tell me again | 22:14 |
| replaceafill | :/ | 22:14 |
| th1a | Can I only copy members of all groups and not just one? | 22:14 |
| aelkner | replaceafill, or tell me at least why you told me that you told me | 22:14 |
| replaceafill | aelkner remember errors messages are displayed next to widgets now? | 22:15 |
| aelkner | yeah, you hacked the form to hide the list of errors | 22:16 |
| aelkner | so why did you not get rid of that message whlie you were at it | 22:16 |
| aelkner | if that's what we want | 22:16 |
| th1a | It just needs to be updated to the better wording. | 22:16 |
| replaceafill | aelkner it's not possible to use the same view/status for all the forms ;) | 22:17 |
| aelkner | oh | 22:17 |
| aelkner | what do think we should do then? | 22:17 |
| replaceafill | use a super magic baseclass with: | 22:17 |
| replaceafill | formErrorsMessage = _('Please correct the marked fields below.') | 22:17 |
| aelkner | i grepped that message, and it only appears in contact | 22:17 |
| aelkner | so obviously, it comes from z3c.form | 22:18 |
| aelkner | replaceafill, we already have base classes, so don't call them super magic please | 22:18 |
| aelkner | that makes e think i need to create a new class or something | 22:18 |
| aelkner | we just need to change what we already have | 22:19 |
| replaceafill | th1a you still processing years, correct? | 22:20 |
| replaceafill | (i dont want to interrupt) | 22:20 |
| th1a | No... resolve what you're talking about. | 22:20 |
| replaceafill | aelkner i agree, formErrorsMessage should be moved to skin.flourish.form.Form maybe... | 22:22 |
| aelkner | th1a, that's ok, i can look into that | 22:22 |
| replaceafill | th1a ok, question | 22:22 |
| replaceafill | http://img26.imageshack.us/img26/2267/pantallazoqy.png | 22:22 |
| replaceafill | this is the current ZODB Control view | 22:22 |
| replaceafill | http://69.164.203.135:6660/control/ZODBControl.html | 22:22 |
| replaceafill | that's my attempt | 22:23 |
| replaceafill | but i dont like the input field and the button | 22:23 |
| replaceafill | + the checkbox | 22:23 |
| th1a | Oh... | 22:23 |
| th1a | Let's assume that since we can't undo, we don't need to keep X days. | 22:24 |
| replaceafill | ah | 22:24 |
| replaceafill | 0 by default | 22:24 |
| th1a | And then put the pack button in the table. | 22:24 |
| th1a | 0 always. | 22:24 |
| th1a | You only need that if you can undo. | 22:24 |
| replaceafill | th1a like an Action | 22:24 |
| th1a | Like the delete buttons. | 22:25 |
| replaceafill | yes | 22:25 |
| th1a | In the table row. | 22:25 |
| th1a | (not the sidebar) | 22:25 |
| replaceafill | yes | 22:25 |
| th1a | And a done button. | 22:25 |
| replaceafill | ah yes! Done | 22:25 |
| replaceafill | also | 22:25 |
| replaceafill | the view can spit success/error messages | 22:25 |
| replaceafill | for example | 22:25 |
| replaceafill | insert foo for the input | 22:25 |
| replaceafill | and submit | 22:25 |
| replaceafill | in the core skin we don't show the messages! :) | 22:26 |
| th1a | So... what do we need to do? | 22:26 |
| replaceafill | i guess one just assumes that if the size of the file changes the packing was successful :) | 22:27 |
| replaceafill | but i can set the message above the table | 22:27 |
| replaceafill | if that's ok? | 22:27 |
| th1a | Yes. | 22:28 |
| replaceafill | kk | 22:28 |
| th1a | aelkner: Can I only copy members of all groups and not just one when making a new year? | 22:29 |
| aelkner | it would seem so | 22:30 |
| aelkner | i wouldn't recommend adding too many features of that nature when we | 22:31 |
| aelkner | have so many things yet to be handled | 22:31 |
| *** menesis has quit IRC | 22:31 | |
| aelkner | like gradebook, journal, intervention | 22:31 |
| aelkner | or am i mistaken that that stuff needs to be addressed in time foe the release | 22:32 |
| th1a | No, but we need to fix this. | 22:32 |
| th1a | I'm not suggesting adding fuctionality. | 22:32 |
| th1a | Just making this view make sense. | 22:32 |
| aelkner | oh, so why do you think you can only copy all groups? | 22:33 |
| th1a | replaceafill: Is that right -- copy members only applies to all? | 22:34 |
| th1a | I'm just trying to interpret what this view is telling me. | 22:34 |
| aelkner | these is a checkbox next to each group | 22:34 |
| th1a | Why don't I see that. | 22:34 |
| th1a | ? | 22:34 |
| aelkner | with site managers not being available for copy | 22:34 |
| th1a | http://69.164.203.135:36660/schoolyears/add.html | 22:34 |
| th1a | Are we looking at the same thing? | 22:35 |
| aelkner | yes, under Copy Members | 22:35 |
| * replaceafill goes to see the view | 22:35 | |
| replaceafill | th1a no | 22:35 |
| th1a | Oh... why can't I copy members of built in groups? | 22:35 |
| replaceafill | it's just that the groups have no members | 22:35 |
| replaceafill | you could try that in my instance | 22:35 |
| replaceafill | let me see | 22:35 |
| aelkner | th1a, ah, i see it is missing on the demo serve for some reason | 22:35 |
| th1a | This view is kind of magical. | 22:36 |
| replaceafill | aelkner this is in trunk, right? | 22:36 |
| replaceafill | http://69.164.203.135:6660/schoolyears/add.html | 22:36 |
| replaceafill | there | 22:36 |
| replaceafill | if the group doesnt have members, there's no option to copy them! | 22:37 |
| replaceafill | maybe a disabled checkbox would be better... | 22:37 |
| replaceafill | or a message | 22:37 |
| aelkner | yeah, i just saw that in the template | 22:37 |
| th1a | No... it is only failing in the manager looking at empty database case. | 22:37 |
| th1a | It is the right behavior. | 22:38 |
| replaceafill | th1a you design it :P | 22:38 |
| th1a | Slightly wrong design. | 22:38 |
| replaceafill | :D | 22:38 |
| * replaceafill goes back to /control | 22:38 | |
| th1a | aelkner: | 22:40 |
| th1a | Get rid of the select all row. | 22:40 |
| th1a | is courses a separate table? | 22:41 |
| aelkner | it's a separate checkox, no table, only the checkbox for copying all courses | 22:46 |
| aelkner | the checkbox is not present if there are no courses | 22:46 |
| th1a | It is formatted as a table row though. | 22:46 |
| aelkner | the table contains everything | 22:47 |
| aelkner | <td> elements are there for each row | 22:47 |
| aelkner | it's not the cleanest view in the world | 22:47 |
| aelkner | sorry, <tr> elements for each row | 22:48 |
| aelkner | conditional on if active year has the type of stuff for that row | 22:48 |
| th1a | Yes, ok make it a separate table. | 22:48 |
| th1a | Get rid of the select all row. | 22:49 |
| th1a | Give the group table headers: | 22:49 |
| th1a | Group Name | Copy Group | Copy Members | 22:49 |
| aelkner | what about courses? | 22:51 |
| th1a | Then just as a question, "Copy courses from active year?" | 22:51 |
| th1a | Yes | 22:51 |
| th1a | (with checkbox) | 22:52 |
| aelkner | yes to what? | 22:52 |
| th1a | Under groups. | 22:52 |
| th1a | Yes to copying courses? | 22:52 |
| aelkner | that wasn't what i was asking | 22:52 |
| aelkner | we already copy courses | 22:52 |
| aelkner | you said something about a table for that? | 22:52 |
| th1a | Just make it a question with a yes checkbox. | 22:53 |
| th1a | Like in the UWG. | 22:53 |
| th1a | Under the group table. | 22:53 |
| aelkner | ah. no change to list each course, good | 22:53 |
| th1a | No... | 22:53 |
| th1a | It is just confusing everything to try to jam it into the same table. | 22:53 |
| aelkner | why do you want to get rid of select all groups? | 22:54 |
| th1a | It complexifies the form more than it helps. | 22:54 |
| aelkner | wouldn't it be a gotcha to not at least preselect all courses for copy? | 22:54 |
| aelkner | gourps | 22:54 |
| aelkner | groups is what i meant | 22:54 |
| th1a | I guess that's not a bad idea. | 22:55 |
| aelkner | then who needs select all | 22:55 |
| th1a | yes | 22:55 |
| aelkner | ok, i have plenty to do now | 22:56 |
| th1a | Good, because I have to go pick up the damn tire. | 22:56 |
| th1a | Later guys. | 22:56 |
| aelkner | cya | 22:56 |
| * replaceafill goes to get lunch | 22:57 | |
| *** dadeng has joined #schooltool | 23:49 | |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!
