| mattuk1972 | info graphs are now fixed | 09:02 |
|---|---|---|
| frankban | mattuk1972: hi, the growl notification must appear over the charm panel? isn't it better if it appears on the top right part of the page? | 11:03 |
| mattuk1972 | eh? | 11:04 |
| frankban | mattuk1972: :-) | 11:20 |
| frankban | mattuk1972: I am working on the growl notifications | 11:21 |
| frankban | mattuk1972: https://docs.google.com/a/canonical.com/file/d/0B6l8lFdCRvtqRjY0S3NIV3N4bmM/edit | 11:21 |
| mattuk1972 | i thought it was best there as it don't really obscure and controls | 11:21 |
| mattuk1972 | doesn't really obscure any controls | 11:22 |
| frankban | mattuk1972: you mean a click on a notification is really a click on the layer below? | 11:23 |
| mattuk1972 | no - the notification if it appears where I've suggested won't hinder the user clicking on the search or charm store controls | 11:24 |
| mattuk1972 | maybe I'm wrong | 11:24 |
| mattuk1972 | any thoughts jovan2 | 11:25 |
| jovan2 | mattuk1972: not sure what you're asking | 11:27 |
| mattuk1972 | can u see the thread above? | 11:27 |
| mattuk1972 | the position of the notification | 11:28 |
| mattuk1972 | frank ban is wondering if it should appear right at the top of the page rather than at the top of the canvas | 11:28 |
| jovan2 | mattuk1972: what do ubuntu notifications do? we should be consistent where appropriate. Also. notifications will appear when the user is simply monitoring their environment, most likely, rather than browsing the charm store, so I'm not sure how much of an issue is the charm search field being obscured momentarily. | 11:33 |
| mattuk1972 | ok so its another bunch of wireframes you need to redo | 11:33 |
| frankban | mattuk1972, jovan2: in my current branch (the first for growl notification, others will follow), the notifications appear in the window top right, and they can be closed on click. This way the position is consistent if you resize the page. | 11:36 |
| jovan2 | mattuk1972, frankban, good point, we need to consider window re-sizing. | 11:37 |
| jovan2 | frankban: you say closed on click. Is it not better to display more of the details of the notification on click? | 11:39 |
| frankban | jovan2: it can be done in a future branch, but consider that sometimes we have just a title and a message, not a real url where to go. | 11:40 |
| jovan2 | frankban: is the message always going to be some sort of error message wrt a unit for a service? In which case opening up the unit details for the service would be a good idea. | 11:42 |
| frankban | jovan2: however, we could add a "view details" link (or something like that) when we have that info. I'd like to postpone this for another branch, if I can get this one landed it can be a base to start requirements discussion. | 11:43 |
| jovan2 | frankban: I think we should go with the spec for now and evaluate when in use. | 11:50 |
| frankban | jovan2: so, notifications over the env view? | 11:51 |
| jovan2 | frankban, yes | 11:51 |
| frankban | jovan2: there is a problem: the notifications div is outside of the env view (and must be outside, because notifications should be preserved if you navigate the app). so their position is absolute, and their z-index is 9999. but the absolute position of the env view top right corner is relative to the window size. so, I can investigate further but at first sight it seems difficult to draw notifications using CSS o | 11:55 |
| frankban | nly | 11:55 |
| jovan2 | frankban: so you need to make it work outside/above of the env area? I think I can live with that. | 12:01 |
| === mattuk1972__ is now known as mattuk1972 | ||
| frankban | tveronezi: hi | 13:58 |
| tveronezi | frankban: hi! | 13:59 |
| frankban | tveronezi: could you please review https://codereview.appspot.com/6851058 ? | 13:59 |
| tveronezi | sure! | 13:59 |
| frankban | tveronezi: thanks! | 13:59 |
| tveronezi | frankban: its done, dude. | 14:40 |
| frankban | thanks tveronezi | 14:47 |
| tveronezi | Hi benji. Man, do you have a minute for a quick brainstorm? I have a question about one of the requirements in https://bugs.launchpad.net/juju-gui/+bug/1078898 | 14:49 |
| _mup_ | Bug #1078898: Makefile does not support static file deployment <juju-gui:Triaged> < https://launchpad.net/bugs/1078898 > | 14:49 |
| benji | tveronezi: sure, juju-ui hangout? | 14:49 |
| tveronezi | ok. | 14:49 |
| frankban | tveronezi: replied to your comment. | 14:52 |
| frankban | benji: if you have time, could you please review https://codereview.appspot.com/6851058/ ? | 15:03 |
| benji | frankban: sure | 15:03 |
| frankban | benji: thank you | 15:03 |
| frankban | benji: abort! abort! Makyo did my second review. thanks Makyo. | 15:19 |
| benji | frankban: heh, thanks | 15:19 |
| Makyo | Oops, sorry, didn't see this until after | 15:22 |
| frankban | jovan2, mattuk1972: growl notifications are now in staging. The alert icon background is not transparent, but that's a known problem of our staging sprite. | 15:48 |
| Makyo | Call today? | 16:00 |
| Makyo | benji, frankban, bcsaller, hazmat, etc ^^^ | 16:01 |
| benji | Makyo: absolutely | 16:01 |
| benji | mattuk1972 jovan2 Makyo tveronezi frankban benji bcsaller hazmat: call in 1 minute | 16:02 |
| bcsaller | more like -1 minute | 16:02 |
| jovan2 | benji: apologies but I'm in a Landscape meeting with Mark | 16:03 |
| benji | jovan2: no worries | 16:03 |
| * tveronezi lunch | 16:49 | |
| * tveronezi back | 17:46 | |
| * tveronezi brb | 19:00 | |
| * tveronezi back | 19:20 | |
| * tveronezi brb | 21:52 | |
| === rog is now known as Guest2621 | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!