| *** openstackgerrit_ has quit IRC | 00:23 | |
| *** openstackgerrit_ has joined #storyboard | 00:24 | |
| *** openstackgerrit_ has quit IRC | 00:48 | |
| *** openstackgerrit_ has joined #storyboard | 00:49 | |
| *** krotscheck has quit IRC | 03:30 | |
| *** krotscheck has joined #storyboard | 03:31 | |
| *** thrace_ has quit IRC | 05:56 | |
| *** krotscheck has quit IRC | 06:01 | |
| *** thrace_ has joined #storyboard | 06:02 | |
| *** krotscheck has joined #storyboard | 06:09 | |
| *** openstackgerrit_ has quit IRC | 06:17 | |
| *** openstackgerrit_ has joined #storyboard | 06:18 | |
| *** mrmartin has joined #storyboard | 06:47 | |
| *** thrace_ has quit IRC | 07:10 | |
| *** thrace_ has joined #storyboard | 07:21 | |
| *** jtomasek has joined #storyboard | 07:27 | |
| *** thrace_ has quit IRC | 07:47 | |
| *** mrmartin has quit IRC | 07:50 | |
| *** mrmartin has joined #storyboard | 07:52 | |
| *** thrace_ has joined #storyboard | 07:54 | |
| *** mrmartin has quit IRC | 07:56 | |
| Zara | ooh, I'll look at those today | 10:13 |
|---|---|---|
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack-infra/storyboard: Replace deprecated LOG.warn with LOG.warning https://review.openstack.org/291649 | 10:25 |
| Zara | morning, storyboard! | 10:41 |
| SotK | morning! | 10:41 |
| Zara | hm, that applies to old migrations; not sure we apply that kind of thing retrospectively but guessing we do? | 10:41 |
| pedroalvarez | that patch won't change the migrations, only how the migrations report warnings when running | 10:44 |
| Zara | ohhhh, okay | 10:45 |
| -openstackstatus- NOTICE: Gerrit is going to be restarted due to bad performance | 10:55 | |
| *** ChanServ changes topic to "Gerrit is going to be restarted due to bad performance" | 10:55 | |
| *** mrmartin has joined #storyboard | 10:57 | |
| *** ChanServ changes topic to "StoryBoard - A task tracker for OpenStack development needs :: http://storyboard.openstack.org/ :: https://wiki.openstack.org/wiki/StoryBoard" | 11:01 | |
| -openstackstatus- NOTICE: Gerrit has been restarted successfully | 11:01 | |
| Zara | okay, so that weird 500 when you add a bunch of tasks to a story, I'm wondering if it happens when you add a task from a different project | 11:06 |
| Zara | *when creating a new story | 11:06 |
| Zara | I just created a story with 3 storyboard-webclient, 1 storyboard, and the storyboard task wasn't created (and gave me a 500). | 11:07 |
| Zara | this is the one where, when you click save, it creates a story but gives a 500 error, doesn't move from the modal page, and clicking save again gives a 400 because there's already a story | 11:07 |
| Zara | it's happened to me a couple of times and it's going to be very annoying | 11:08 |
| Zara | I also found a bug with the new story modal that opens when you click 'create story' from a card modal, but I'm not sure what triggered it or how frequently it'll occur | 11:09 |
| Zara | I filled out all fields; 'save' was still greyed out and not interactive | 11:10 |
| *** mrmartin has quit IRC | 11:18 | |
| * Zara adds ALL the task notes | 11:21 | |
| Zara | I am so happy | 11:21 |
| *** mrmartin has joined #storyboard | 11:21 | |
| * SotK triggered that "save is still greyed out" bug yesterday too | 11:25 | |
| Zara | interesting, do you know if it's from all 'new story' modals? | 11:26 |
| Zara | or just the one from card modal | 11:26 |
| SotK | They all use the same code, so I'd imagine all of them | 11:27 |
| SotK | I got it using the button in the header bar | 11:27 |
| Zara | oh, okay, so yeah, definitely both | 11:27 |
| Zara | my working hypothesis is that the selected project didn't register for some reason | 11:27 |
| Zara | for one of the tasks | 11:27 |
| Zara | because that kind of happened when I made another story | 11:28 |
| Zara | but I put the project in again and selected it from the dropdown and it worked | 11:28 |
| Zara | okay, yeah, reproduced, it's kinda like that | 11:30 |
| Zara | if you type a project, then make some tasks and things afterwards, then go back and select the project for the earlier task from the dropdown, sometimes it still stays confused | 11:31 |
| Zara | then the workaround is to delete that task and make it again | 11:31 |
| *** mrmartin has quit IRC | 11:31 | |
| Zara | but sometimes it does work, so still a bit strange | 11:32 |
| *** jtomasek has quit IRC | 11:34 | |
| openstackgerrit | Merged openstack-infra/storyboard-webclient: Add a tooltip when hovering on the "Task notes" button https://review.openstack.org/291391 | 11:38 |
| *** jtomasek has joined #storyboard | 11:39 | |
| *** mrmartin has joined #storyboard | 11:42 | |
| * Zara nitpicks, kinda | 11:45 | |
| Zara | it's not even a complaint, really, more an 'I want a pony' | 11:45 |
| SotK | d'you mean instead of the button color? | 11:45 |
| Zara | alongside the button colour | 11:46 |
| Zara | yay, an ambiguous pony | 11:46 |
| Zara | I tried quickly making a followup patch, then got in a tangle of <ng-class...> vs <i class...> | 11:46 |
| Zara | and that just stopped all the buttons from displaying | 11:47 |
| Zara | I like it, anyway. | 11:48 |
| *** mrmartin has quit IRC | 11:48 | |
| Zara | what I evennnntually envision is | 11:48 |
| * SotK has a poke around | 11:48 | |
| Zara | fa-sticky-o for no notes | 11:48 |
| Zara | fa-stick for notes | 11:48 |
| Zara | and btn-primary for *new* notes | 11:49 |
| Zara | but that's wayyyy more complicated | 11:49 |
| SotK | if we end up allowing multiple notes, it'd be cool to show the number on the button too | 11:49 |
| Zara | I'm actually not sure 'multiple notes' is that important any more, where the field for notes is infinitely long (or close enough. | 11:50 |
| Zara | I would like collaborative realtime editing on note descriptions, mainly for my own amusement | 11:50 |
| Zara | s/note descriptions/notes | 11:51 |
| Zara | (sometimes I think of them as descriptions becuase the code is effective = story descriptions) | 11:51 |
| Zara | *effectively | 11:51 |
| Zara | I cannot type today | 11:51 |
| Zara | 'fa-stick', what is that. | 11:51 |
| Zara | let's represent everything in storyboard with a stick | 11:51 |
| Zara | *shrug* | 11:51 |
| Zara | I made a story for task notes, anyway https://storyboard.openstack.org/#!/story/2000511 | 11:52 |
| SotK | Zara: how's this look? http://storyboard.sotk.co.uk/#!/story/2000322 | 11:55 |
| SotK | (note: you can't save task details since the db is currently in a weird state) | 11:56 |
| Zara | ahhh | 11:56 |
| Zara | I was wondering about that | 11:56 |
| Zara | er, it currently looks the same as before when I start editing a task | 11:56 |
| Zara | that is, fa-sticky-o on btn-primary | 11:56 |
| Zara | but yeah, fa-sticky-o the rest of the time is nice | 11:57 |
| SotK | that is fa-sticky-note on btn-primary, btn-primary is inverting the text colour I think | 11:57 |
| Zara | oh, right | 11:57 |
| Zara | hm. | 11:57 |
| Zara | in that case I guess I want fa-sticky-note-o on both | 11:58 |
| Zara | sorry for the confusion :/ | 11:58 |
| Zara | and also I should prrrrrobably put the compliments before the criticisms, thanks for putting up with me xD | 11:58 |
| Zara | I do the whole, 'well that bit works, so I just won't mention it' thing when I'm lazy. | 11:59 |
| * SotK is trying sticky-note-o now | 12:01 | |
| * SotK doesn't mind :) | 12:01 | |
| Zara | heh :) | 12:02 |
| SotK | http://storyboard.sotk.co.uk/#!/story/2000322 now uses fa-sticky-note-o | 12:11 |
| Zara | gahhh, so that just makes the icon the same colour as the button | 12:12 |
| Zara | so actually, what I wanted originally... was fa-sticky-note but not inversed on btn-primary | 12:13 |
| Zara | klfdjasfkla | 12:13 |
| Zara | white post-it empty. black post-it full. gahhhh | 12:15 |
| Zara | oh, also found a bug | 12:15 |
| Zara | so after you cancel editing, button acts as though there are notes | 12:15 |
| Zara | until a refresh | 12:15 |
| Zara | so I'm not sure if that's my modal code being bad | 12:16 |
| Zara | or if it's to do with the ng-class for the button | 12:16 |
| SotK | it'll be because it edits task.link in its representation, but doesn't push the changes to the server | 12:16 |
| Zara | ah. it's confusing for users. :/ | 12:17 |
| Zara | they'll assume, like I did, that they didn't cancel and 'saved' by mistake | 12:17 |
| Zara | or saved because of a bug | 12:18 |
| Zara | so maybe it should refresh the page... though that's kinda slow | 12:18 |
| * Zara sadly notes it on gerrit | 12:20 | |
| * SotK solved it in the card modal by using a second variable as the ng-model, then making the save button set the description to the value of the second variable before saving (and the cancel button reset the second variable to the original description) | 12:22 | |
| *** jtomasek has quit IRC | 12:25 | |
| *** jtomasek has joined #storyboard | 12:27 | |
| SotK | Zara: is http://storyboard.sotk.co.uk/#!/story/2000322 what you were going for? | 12:40 |
| Zara | yes xD | 12:41 |
| Zara | minus the 'not reacting right to cancel', anyway | 12:41 |
| Zara | then in the future we only make it go red when someone adds a new note. | 12:41 |
| Zara | so you have 'empty', 'full', 'look at this!' | 12:42 |
| SotK | that seems sensible to me | 12:44 |
| Zara | :) actually coding the new note bit will probably be a bit horrible, so I've no timeline for that \o/ | 12:45 |
| Zara | also I have not yet started on my card modal section for adding notes | 12:45 |
| Zara | I should do that | 12:45 |
| Zara | oh we also need recent events templates for task notes | 12:55 |
| Zara | every change requires 5 more changes to be good | 12:55 |
| Zara | this is why I hate adding new features and prefer fixing existing things, haha. | 12:56 |
| *** fay_ has quit IRC | 13:27 | |
| *** fay_ has joined #storyboard | 13:27 | |
| *** fay_ has quit IRC | 13:42 | |
| *** fay_ has joined #storyboard | 13:57 | |
| Zara | after a while bashing my head agains the modal, realised it needed <label> tags | 14:09 |
| Zara | *against | 14:09 |
| Zara | now to make the notes actually editable | 14:10 |
| Zara | (and how is it 2:10) | 14:10 |
| persia | Blame timezones. It can be anywhere from 3:10am Friday to 4:10am Saturday, if you change that one variable. | 14:11 |
| Zara | :) | 14:12 |
| persia | Err, rather 3:12 to 2:12 (-11 to +12) at this time of year. | 14:19 |
| Zara | I did not notice the inaccuracy. :) | 14:20 |
| Zara | hm, wonder if I can update task notes as part of updateTask, or if it should be a separate function (in the card detail controller) | 14:49 |
| Zara | that's currently just used for assignee | 14:49 |
| Zara | (I'll push what I have so far to gerrit, anyway.) | 14:54 |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Display task notes from board https://review.openstack.org/291755 | 14:54 |
| Zara | can you tell I'm procrastinating from the horrible bit. | 14:55 |
| SotK | you can probably do it as part of updateTask | 14:58 |
| Zara | cool, I figured it was a similar sort of thing, so it shooould be okay... but ehh | 15:00 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Highlight the notes button if a task has notes https://review.openstack.org/291404 | 15:19 |
| Zara | thanks, will take a look now! I like how `color: @black !important;` reads like a jab at my review comments. | 15:20 |
| SotK | :D | 15:23 |
| Zara | arrrgh, now it doesn't save any changes in the view until refresh | 15:24 |
| SotK | ah, oops | 15:24 |
| Zara | ahahaha | 15:24 |
| Zara | I also discovered a way my modal is rubbish | 15:24 |
| Zara | you can't completely delete notes | 15:24 |
| Zara | I'll add that to the story | 15:25 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Highlight the notes button if a task has notes https://review.openstack.org/291404 | 15:26 |
| Zara | \o/ | 15:34 |
| Zara | looks great, thanks | 15:34 |
| SotK | \o/ | 15:36 |
| Zara | :) | 15:37 |
| openstackgerrit | Merged openstack-infra/storyboard-webclient: Highlight the notes button if a task has notes https://review.openstack.org/291404 | 15:46 |
| Zara | \o/ again | 15:48 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Stop updating the board in-place when moving lanes https://review.openstack.org/291786 | 16:02 |
| Zara | ah, thanks | 16:05 |
| * Zara screws up an ng-click so that her textbox closes whenever someone clicks in it to edit thigns | 16:06 | |
| Zara | *things | 16:06 |
| SotK | :D | 16:07 |
| SotK | $event.stopPropagation() might be your friend | 16:07 |
| Zara | that looks terrifying, whatever it is | 16:09 |
| Zara | I actually think I might be able to get away with 'showTaskForm="true"' | 16:10 |
| Zara | or mayyybe not ;_; | 16:10 |
| * Zara concedes defeat | 16:13 | |
| Zara | hm, googling suggests that it's to stop a child event bubbling up to a parent event? | 16:14 |
| Zara | I thiiiink I have the reverse | 16:14 |
| Zara | http://paste.openstack.org/show/490191/ | 16:21 |
| Zara | also suspect I've messed up the indentation in multiple places | 16:22 |
| Zara | but that's what I have so far | 16:22 |
| Zara | can anyone help? I tried putting upper ng-click in it's own <div> (and even <i>!) but it didn't help | 16:34 |
| Zara | I've probably got something weird in there | 16:34 |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Edit task notes from board https://review.openstack.org/291755 | 17:27 |
| *** krotscheck has quit IRC | 17:34 | |
| *** b3rnard0 is now known as b3rnard0_away | 17:35 | |
| *** krotscheck has joined #storyboard | 17:37 | |
| Zara | I'm already using task comments so much :') | 17:37 |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Edit task notes from board https://review.openstack.org/291755 | 17:42 |
| pleia2 | ooh, task notes, nice :) | 17:42 |
| Zara | :) I've wanted these for moooooonths | 17:43 |
| Zara | I'm so happy we've got round to it! :D | 17:43 |
| Zara | they may be buggy, very new. | 17:43 |
| Zara | (so I'd advise against writing loads and loads of stuff in them if it's not backed up anywhere, until they've had a bit of time to settle, juuust in case.) | 17:45 |
| * pleia2 nods | 17:45 | |
| Zara | but early signs are good, anyway! \o/ | 17:45 |
| pedroalvarez | black sticky note is black | 17:50 |
| pedroalvarez | :S | 17:50 |
| Zara | I don't think the newest version has been built yet | 17:50 |
| pedroalvarez | it has in my instance | 17:50 |
| pedroalvarez | my primary green and the black sticky note don't go well together.. :) | 17:51 |
| pedroalvarez | s/primary green/primary color(green)/ | 17:51 |
| Zara | ahh, right, hm. | 17:51 |
| pedroalvarez | it's ok though :) | 17:52 |
| pedroalvarez | I have task notes :D | 17:52 |
| persia | Are they annotated in a semantic way that means one with primary colour (green) can recolor them from black? | 17:53 |
| Zara | pedroalvarez: I guess we could change the note colour back to white if it bothers people, but the end goal is to only use the primary colour for new notes-- but that could take a while to do. if it's annoying for your users then I'm fine with changing it back, even if I prefer the black note. :P | 17:55 |
| Zara | persia: people can't recolour them at the moment, no | 17:55 |
| Zara | there's no timeline for the fixes that would make that possible, either. | 17:55 |
| Zara | I'm working on this until SotK sends me a wall of automatic boards and worklists patches. | 17:55 |
| * persia is reminded of http://thecodelesscode.com/case/95 | 17:56 | |
| Zara | well it's the case where the icon has a black outline, but is currently all black, so it's harder to see on a dark branding colour | 17:57 |
| Zara | whereas in white, it's distinguishable | 17:57 |
| Zara | but stands out less from 'empty' notes in a big list | 17:57 |
| Zara | (and once people know what the icon is, they don't need to work it out again (plus there's a tooltip).) so I'm not too fussed whatever, if there's a big backlash I'll change it until we have time to make it fancier. | 17:58 |
| persia | Understood. In my dream world, there is an "iconoutline" (or whatever capitlaisation one likes) property whose color can be adjusted to be sufficiently distinguishable. It may be that this references an palette element, so that one can maintain a consistent palette when altering presentation, or it may be that it just hardcodes a colour, so that when changing colours, one has to do much dancing. | 17:59 |
| persia | However, as time passes, I find more and more data that seems to indicate that neither my perceptions nor those reported by others indicate that this is my dream world :) | 18:00 |
| Zara | you can probably already play with it in the css if you really want, just not from the ui. | 18:01 |
| Zara | we just try to provide sensible defaults | 18:02 |
| persia | As you ought :) | 18:02 |
| Zara | (this is a reason there is no colour-coding on the kanban; too high a chance of clashing/blending with branding) | 18:02 |
| Zara | (plus one of my original thoughts was that if someone's btn-primary was light, the notes would be hard to distinguish from empty ones, whereas there's no danger of that happening. though I don't know if/when it switches from black to white, so maybe I was wrong to assume that could happen. | 18:08 |
| Zara | *that happening if the colour is always the inverse of the empty colour | 18:09 |
| * persia is enlightened | 18:09 | |
| Zara | :) | 18:11 |
| SotK | I'd imagine if someone sets a light `background-color` for btn-primary they'll also set a dark `color` | 18:12 |
| Zara | heh, well I'll change it back if people hate it. :) It doesn't look *nice* atm, so I won't fight. I figured it was easier to distinguish. | 18:16 |
| *** b3rnard0_away is now known as b3rnard0 | 18:39 | |
| *** yarkot_ has joined #storyboard | 21:17 | |
| *** yarkot_ has quit IRC | 21:19 | |
| persia | I just happened to be working a bug in LP that had a task link: https://bugs.launchpad.net/ubuntu/+source/freeciv/+bug/743265 | 21:41 |
| openstack | Launchpad bug 743265 in freeciv (Ubuntu Natty) "When using unity, the diplomacy and cities menus are not shown." [Undecided,Fix released] | 21:41 |
| persia | It seems that the overloaded display field is assignee: when LP knows that another tracker is responsible for the bug, it doesn't offer the ability to assign someone in LP, but rather directs people to the right tracker for that. It doesn't work for GNA, but for other trackers, LP can sometimes scrape status. | 21:42 |
| *** b3rnard0 has left #storyboard | 21:56 | |
| *** jtomasek has quit IRC | 22:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!