fluid-work IRC Logs-2012-06-01

[07:44:32 CDT(-0500)] <anastasiac> good morning cindyli. I had a question about one of the alt text fixes you made yesterday afternoon

[07:44:44 CDT(-0500)] <cindyli> ok, anastasiac

[07:45:08 CDT(-0500)] <cindyli> r u working from home today, anastasiac

[07:45:18 CDT(-0500)] <anastasiac> it's about the folder combo box arrow (yes, working from home)

[07:45:32 CDT(-0500)] <anastasiac> did you test your fix for this one?

[07:45:34 CDT(-0500)] <cindyli> oh, tabindex −1?

[07:45:55 CDT(-0500)] <anastasiac> yes, but also whether or not the title is read by the screen reader

[07:45:57 CDT(-0500)] <cindyli> i tested the saved states but not unsaved one

[07:46:09 CDT(-0500)] <anastasiac> when I tested, I found that the screen reader didn't read the title

[07:46:13 CDT(-0500)] <anastasiac> because there is link text there

[07:46:46 CDT(-0500)] <anastasiac> (this was yesterday)

[07:47:50 CDT(-0500)] <cindyli> right, unless you adjust the screen reader setting to read the title, or the longest of all the available info

[07:48:20 CDT(-0500)] <cindyli> JAWS has this setting, don't know where in voiceOver though

[07:48:57 CDT(-0500)] <anastasiac> ah, interesting. I wonder if it's ok to rely on that setting, given that it's not the default...

[07:49:04 CDT(-0500)] <anastasiac> also, why the tab index=-1?

[07:52:58 CDT(-0500)] <cindyli> regarding tabindex "-1", with created items or if the user has to login or register to perform save, the "save" button is disabled and not allowed to perform, so there's no need for the user to tab onto it. what do you think?

[07:53:04 CDT(-0500)] <cindyli> anastasiac: ^

[07:53:57 CDT(-0500)] <anastasiac> but isn't the tabindex=-1 on the drop down arrow, and not the save button?

[07:55:48 CDT(-0500)] <anastasiac> cindyli: ^

[07:56:02 CDT(-0500)] <cindyli> anastasiac: since the savebutton is disabled at that point, it's already not tabbable

[07:56:58 CDT(-0500)] <anastasiac> hm. ok, I guess I see the logic

[07:58:08 CDT(-0500)] <cindyli> cool

[07:58:46 CDT(-0500)] <anastasiac> michelled, I sent you an email about the one thing left in the 545 ticket that's not done. Other than that, I think cindyli's FLOE-26 branch is ready.

[08:00:46 CDT(-0500)] <michelled> anastasiac, cindyli: that's great!

[08:01:42 CDT(-0500)] <michelled> anastasiac, cindyli: did one of you work on a patch that added alt text to the content generated from the authoring tool? I seem to remember that it was in the authoring-a11y branch

[08:01:50 CDT(-0500)] <cindyli> one sec, anastasiac, michelled. I haven't tested a couple of fixes in my FLOE-26 'cuz I cannot get the published items showed up on browse all page

[08:01:56 CDT(-0500)] <michelled> I think we need to pull that out into it's own branch so it can get in now

[08:03:57 CDT(-0500)] <cindyli> and, this item in 545 ticket: "Site-wide: Input fields should be associated with proper labels." haven't checked up site wide

[08:04:17 CDT(-0500)] <anastasiac> michelled, I believe we fixed the auth tool to ensure that images got alt text. I don't think the fix was in its own branch, or that there's a specific patch

[08:04:54 CDT(-0500)] <michelled> anastasiac: it looks to me that there is no alt text for images in the current staging - am I missing something?

[08:05:46 CDT(-0500)] <anastasiac> sorry, michelled, I wasn't clear: I believe the fix is in the authoring-a11y branch i.e. not in its own branch

[08:06:15 CDT(-0500)] <michelled> right. I think we need to pull it out into its own branch

[08:06:40 CDT(-0500)] <michelled> because that fix must get in whereas we aren't sure we'll get the authoing-a11y branch in

[08:06:50 CDT(-0500)] <anastasiac> ok, michelled, I can look into that next

[08:06:55 CDT(-0500)] <michelled> thanks!

[08:07:19 CDT(-0500)] <michelled> anastasiac: I'm going to open a separate assembla ticket for the Facebook and google plus links

[08:07:58 CDT(-0500)] <Justin_o> jhung: good morning.. any luck with getting the unit tests running

[08:17:32 CDT(-0500)] <jhung> justin_o: I haven't gotten to it yet. On the phone.

[08:17:56 CDT(-0500)] <Justin_o> jhung: okay no problem.. let me know when you are free. have a couple more questions for you

[08:31:36 CDT(-0500)] <michelled> cindyli, anastasiac: which screen readers and browsers have you been testing your fixes with?

[08:31:55 CDT(-0500)] <cindyli> voiceOver on safari

[08:32:06 CDT(-0500)] <anastasiac> michelled, I mostly used Jaws with IE9 (since that't the vm where I have jaws), with a little bit of voiceover

[08:32:43 CDT(-0500)] <anastasiac> michelled, cindyli: are you able to preview stuff in the auth tool using the staging branch code? I'm getting an error

[08:32:46 CDT(-0500)] <michelled> ok, cool - good to know

[08:33:26 CDT(-0500)] <cindyli> anastasiac: i'm getting an error too

[08:33:32 CDT(-0500)] <cindyli> get_draft() takes exactly 2 arguments (1 given)

[08:34:16 CDT(-0500)] <anastasiac> right

[08:34:25 CDT(-0500)] <anastasiac> that' sucks

[08:37:29 CDT(-0500)] <anastasiac> michelled, it seems staging adds alt text to images already. I'm going to double-check the code, but I just tested my "pre-fix" state, and the alt text is there

[08:37:59 CDT(-0500)] <cindyli> anastasiac: another way to get into a preview is, rather than using (http://localhost:8000/authoring/1/preview), use (http://localhost:8000/authoring/1-united-nations/view), in the 2nd URL, replace "1-united-nations" with your item title. These previews might be slightly different though

[08:38:17 CDT(-0500)] <cindyli> anastasiac: michelled went on coffee trip

[08:38:34 CDT(-0500)] <anastasiac> ah, right, the coffee run. thanks for reminding me, cindyli

[08:38:41 CDT(-0500)] <cindyli> np

[08:40:38 CDT(-0500)] <jhung> justin_o: going to try testing now.

[08:52:41 CDT(-0500)] <michelled> anastasiac: that's good to know - I'll have to double check what I was doing last night to get a resource with an image without alt text

[08:56:10 CDT(-0500)] <Justin_o> jhung: thanks.. I running the 100+ page exports again and still can't reproduce DECA-61. I'm in the middle of a type-3 export now and if it works i'll close it off

[08:56:27 CDT(-0500)] <Justin_o> Type 2 took over 30 minutes.. so i imagine this will be a while

[08:59:48 CDT(-0500)] <anastasiac> michelled, I'm still testing - it looks like it might be different based on how the image is provided

[09:00:44 CDT(-0500)] <michelled> ah, interesting

[09:08:51 CDT(-0500)] <anastasiac> michelled, if you're able to reproduce the lack of alt text, could you let me know how? I can't

[09:21:12 CDT(-0500)] <cindyli> michelled: in ticket 545 to improve alt text, there's this item: One example is the 'results per page' and 'Sort By' selects on the search results page. They do have text that labels but because they are not associated correctly screen readers are unable to give the user context of what the drop down is for.

[09:21:38 CDT(-0500)] <cindyli> how do we wanna change it? it sounds ok to me via screen readers.

[09:23:03 CDT(-0500)] <Justin_o> jhung: i've closed DECA-61 as can't reproduce.. i'm testing DECA-58 now

[09:23:30 CDT(-0500)] <jhung> Great news Justin_o:

[09:23:32 CDT(-0500)] <michelled> cindyli: did you try NVDA and JAWS?

[09:23:43 CDT(-0500)] <michelled> cindyli: I think Voice Over read it ok

[09:23:44 CDT(-0500)] <cindyli> tried JAWS and voiceOver, not NVDA

[09:24:02 CDT(-0500)] <jhung> Still downloading packages justin_o. In the meantime, reviewing the videos.

[09:26:17 CDT(-0500)] <Justin_o> jhung: ah.. you miss the super fast internet we have here at the office eh

[09:26:46 CDT(-0500)] <michelled> anastasiac: it does seem fine to me now - not sure what was going on yesterday. if we don't fill in the 'description' when inserting an image then there is no alt text but both NVDA and Voice Over do something reasonable

[09:26:53 CDT(-0500)] <michelled> anastasiac: thanks for looking into that

[09:27:13 CDT(-0500)] <michelled> cindyli: can you try it with NVDA?

[09:27:37 CDT(-0500)] <cindyli> just tried, michelled, doesn't sound good with NVDA. working on it

[09:27:48 CDT(-0500)] <michelled> thx!

[09:27:50 CDT(-0500)] <cindyli> np

[09:27:55 CDT(-0500)] <jhung> justin_o: yeah. Also doesn't help that it stops on a Y/n prompt. :/

[09:28:25 CDT(-0500)] <Justin_o> jhung: do you know if DECA-207, DECA-203, DECA-261 can be closed?

[09:28:43 CDT(-0500)] <Justin_o> jhung: yah.. i hate that… i have to sit and watch it to make sure it doesn't hold up because of that

[09:29:54 CDT(-0500)] <jhung> looking those jiras up...

[09:30:27 CDT(-0500)] <jhung> Deca-207 is supposed to be implemented, but hasn't been tested thoroughly.

[09:31:08 CDT(-0500)] <michelled> anastasiac: are you still working with cindyli on the alt text fixes?

[09:31:18 CDT(-0500)] <jhung> DECA-203 I think will be a "nice to have". Still worth having in Jira, but probably not get around to it...

[09:31:41 CDT(-0500)] <anastasiac> michelled, I'm on an IMS call right now; I think cindyli said she needed to test some fixes, but other than that, there's nothing left to do

[09:31:58 CDT(-0500)] <jhung> justin_o: I changed DECA-203 to minor and as a "New Feature"

[09:33:18 CDT(-0500)] <michelled> ah, cool - anastasiac. let me know when you're done with IMS

[09:33:48 CDT(-0500)] <Justin_o> jhung: thanks..

[09:34:16 CDT(-0500)] <jhung> DECA-261 has been completed justin_o. I'll close that one now.

[09:34:24 CDT(-0500)] <Justin_o> jhung: I tried DECA-58, and it still seems to be skipping a page of your supplied images.. although the one it skips is one of the only ones that is all text.. so it is a bit strange

[09:34:30 CDT(-0500)] <Justin_o> jhung: thanks

[09:35:01 CDT(-0500)] <Justin_o> jhung: can you also add more info to DECA-121 i'm not really sure what it means

[09:35:48 CDT(-0500)] <jhung> thomas said that the new ocropus may fix DECA-58. Does our TIFF conversion trick fix this one too?

[09:37:07 CDT(-0500)] <jhung> Justin_o: I have no idea what DECA-121 is. Close it because it's not specific enough.

[09:39:15 CDT(-0500)] <Justin_o> jhung: okay.. i'll close DECA-121 and DECA-58 isn't fixed by the tiff conversion as it still happens when run through the server now.. well only one page is skipped…

[09:39:32 CDT(-0500)] <Justin_o> jhung: we should also retest the need for tiff conversion with the new genpdf we are using

[09:39:45 CDT(-0500)] <jhung> justin_o: sounds good.

[09:39:56 CDT(-0500)] <jhung> it would be good not to have to use ImageMagick if we can.

[09:40:32 CDT(-0500)] <Justin_o> jhung: agreed.. it would be nice to knock off another dependency

[09:44:14 CDT(-0500)] <jhung> justin_o: Skype in a few minutes?

[09:44:38 CDT(-0500)] <Justin_o> jhung: sure.. i'll login now.. call me when you're ready

[10:05:14 CDT(-0500)] <anastasiac> michelled, I'm off my call now

[10:06:01 CDT(-0500)] <michelled> anastasiac: do you still have cloud 4 all stuff to do or are you looking for something to do for Floe?

[10:06:29 CDT(-0500)] <anastasiac> michelled, I'm waiting on feedback from colinclark for cloud4all stuff, so I'm looking for Floe work in the meantime

[10:06:45 CDT(-0500)] <colinclark> i'll read your use cases shortly, anastasiac

[10:06:50 CDT(-0500)] <anastasiac> np, colinclark

[10:06:51 CDT(-0500)] <colinclark> when do you leave today?

[10:07:07 CDT(-0500)] <anastasiac> 2pm ish, but I'll be making up the time over the weekend

[10:07:29 CDT(-0500)] <michelled> ok, cool anastasiac - I think we should test the alt text branch in ISKME's target browsers

[10:07:36 CDT(-0500)] <michelled> anastasiac: I'm looking at IE8

[10:07:44 CDT(-0500)] <michelled> anastasiac: can you try FF on windows?

[10:08:07 CDT(-0500)] <anastasiac> michelled, will do. The latest is cindy's FLOE-26 branch, right?

[10:08:14 CDT(-0500)] <michelled> yep

[10:08:17 CDT(-0500)] <anastasiac> k

[10:17:12 CDT(-0500)] <anastasiac> cindyli, did you add a title to the search field in the main header?

[10:17:45 CDT(-0500)] <anastasiac> I'm not seeing one

[10:27:54 CDT(-0500)] <cindyli> right, anastasiac, i missed that

[10:28:08 CDT(-0500)] <anastasiac> ok, I wondered if I merged wrong

[10:44:58 CDT(-0500)] <anastasiac> michelled, cindyli, sorry about the technical difficulties. I'm testing the fixes in the alt text branch in FF on windows with Jaws

[10:45:12 CDT(-0500)] <cindyli> thanks, anastasiac

[10:45:33 CDT(-0500)] <anastasiac> cindyli, if you're going to fix the missing title attar in the header, you might want to also look at the search field in the left sidebar of the search results page: no title attar there either

[10:45:53 CDT(-0500)] <cindyli> ok, anastasiac

[10:46:09 CDT(-0500)] <anastasiac> sorry, cindyli, that's on the My OER page

[10:46:13 CDT(-0500)] <anastasiac> not seach results

[10:46:43 CDT(-0500)] <cindyli> ok, i will search all the "search" fields across.

[12:01:57 CDT(-0500)] <anastasiac> michelled, cindyli: I emailed you with results of FF+Jaws testing of the alt text issues

[12:02:16 CDT(-0500)] <cindyli> ok, thanks, anastasiac

[12:08:50 CDT(-0500)] <michelled> thx

[12:11:00 CDT(-0500)] <anastasiac> michelled, should I focus on more alt-text testing, and other screen reader testing, or move on to actually fixing one of the issues you've filed tickets form?

[12:11:07 CDT(-0500)] <anastasiac> *for

[12:18:51 CDT(-0500)] <michelled> anastasiac: I think you should start looking at one of the keyboard issues

[12:18:55 CDT(-0500)] <michelled> maybe this one? https://www.assembla.com/spaces/iskme/tickets/546-bookmark-and-share-popup-does-not-allow-users-to-navigate-past-it-when-using-a-keyboard

[12:33:39 CDT(-0500)] <anastasiac> will do, michelled

[14:05:59 CDT(-0500)] <cindyli> michelled: more commits into FLOE-26 @ https://github.com/cindyli/OER-Commons/tree/FLOE-26

[14:06:12 CDT(-0500)] <michelled> thanks cindyli

[14:06:15 CDT(-0500)] <cindyli> np

[14:23:59 CDT(-0500)] <michelled> cindyli: I find in chrome I can't get focus into the reference dialog - were you able to with the browser you are testing with?

[14:24:50 CDT(-0500)] <cindyli> where's the reference dialog, michelled

[14:28:16 CDT(-0500)] <michelled> it shows up when you add a reference in the editing tool

[14:28:20 CDT(-0500)] <michelled> you made a fix to it

[14:28:22 CDT(-0500)] <michelled> textarea name="reference_text