/irc-logs / mozilla / #accessibility / 2012-06-18 / end

Options:

  1. # Session Start: Mon Jun 18 00:00:01 2012
  2. # Session Ident: #accessibility
  3. # [00:00] * Quits: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se) (Quit: Leaving.)
  4. # [00:23] * Joins: davidb (davidb@moz-68BF56A6.dsl.bell.ca)
  5. # [00:23] * ChanServ sets mode: +qo davidb davidb
  6. # [00:25] * Quits: @davidb (davidb@moz-68BF56A6.dsl.bell.ca) (Quit: blast off!)
  7. # [00:31] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
  8. # [00:35] * Quits: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com) (Ping timeout)
  9. # [00:35] * Quits: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net) (Ping timeout)
  10. # [00:35] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  11. # [00:35] * Quits: logbot (logbot@moz-58CB32ED.glob.com.au) (Ping timeout)
  12. # [00:35] * Joins: capella_ (chatzilla@moz-DD0C7E4F.twcny.res.rr.com)
  13. # [00:35] * Joins: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com)
  14. # [00:35] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  15. # [00:35] * Joins: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net)
  16. # [00:35] * Quits: capella (chatzilla@moz-DD0C7E4F.twcny.res.rr.com) (Ping timeout)
  17. # [00:36] * capella_ is now known as capella
  18. # [00:36] * Joins: logbot (logbot@moz-58CB32ED.glob.com.au)
  19. # [00:36] * Quits: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com) (Ping timeout)
  20. # [00:37] * Joins: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com)
  21. # [00:48] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
  22. # [00:53] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
  23. # [00:54] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
  24. # [00:54] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
  25. # [02:58] * Quits: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com) (Ping timeout)
  26. # [02:59] * Joins: shorlander_ (shorlander@moz-853043D6.dhcp.insightbb.com)
  27. # [03:02] * khuey is now known as khuey|away
  28. # [03:03] * khuey|away is now known as khuey
  29. # [03:31] * Joins: mike5w3c (MikeS@moz-C3998BBE.xgsspn.imtp.tachikawa.spmode.ne.jp)
  30. # [04:31] * Quits: mike5w3c (MikeS@moz-C3998BBE.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
  31. # [04:31] * Joins: mike5w3c (MikeS@moz-736E4479.xgsspn.imtp.tachikawa.spmode.ne.jp)
  32. # [04:43] * khuey is now known as khuey|away
  33. # [04:43] * khuey|away is now known as khuey
  34. # [05:27] * Quits: mike5w3c (MikeS@moz-736E4479.xgsspn.imtp.tachikawa.spmode.ne.jp) (Quit: mike5w3c)
  35. # [07:00] * khuey is now known as khuey|away
  36. # [07:13] * Joins: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp)
  37. # [07:16] * Quits: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp) (No route to host)
  38. # [07:16] * Joins: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp)
  39. # [07:18] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Client exited)
  40. # [07:26] * Joins: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se)
  41. # [07:53] * Joins: surkov (surkov@539620DD.7F110E62.4C549139.IP)
  42. # [07:53] * ChanServ sets mode: +o surkov
  43. # [07:54] * Quits: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se) (Quit: Leaving.)
  44. # [08:14] * Quits: mike5w3c (MikeS@moz-B5EE3299.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
  45. # [08:16] * Quits: @surkov (surkov@539620DD.7F110E62.4C549139.IP) (Quit: surkov)
  46. # [08:25] * Joins: mike5w3c (MikeS@moz-6876B8D5.u05.hotspot.ne.jp)
  47. # [09:23] * Joins: MarcoZ (Daily@moz-7A1C740A.dip.t-dialin.net)
  48. # [09:23] * ChanServ sets mode: +o MarcoZ
  49. # [09:28] * Quits: mike5w3c (MikeS@moz-6876B8D5.u05.hotspot.ne.jp) (Quit: mike5w3c)
  50. # [10:15] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
  51. # [12:10] * Quits: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP) (Quit: Computer has gone to sleep.)
  52. # [12:11] * Quits: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net) (Quit: nhirata)
  53. # [12:16] <capella> firebot: black coffee
  54. # [12:16] <@firebot> capella: eww, i prefer milk with my coffee
  55. # [12:27] <capella> firebot: milk is cow juice
  56. # [12:27] <@firebot> capella: ok
  57. # [12:38] <@firebot> trev.saunders@gmail.com granted review for attachment 633867 on bug 757503.
  58. # [12:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
  59. # [12:41] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
  60. # [12:47] * Quits: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com) (Ping timeout)
  61. # [12:51] * Joins: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com)
  62. # [12:52] * Quits: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com) (Client exited)
  63. # [12:53] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
  64. # [12:53] * Joins: NeilAway (neil@moz-32AA0D01.in-addr.btopenworld.com)
  65. # [13:35] * Quits: victorporof (victorporo@2B22C4F3.20718EE9.79933D60.IP) (Quit: Linkinus - http://linkinus.com)
  66. # [13:35] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  67. # [13:39] * Joins: mike5w3c (MikeS@moz-D1E89A92.xgsspn.imtp.tachikawa.spmode.ne.jp)
  68. # [13:50] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
  69. # [14:01] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  70. # [14:06] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
  71. # [14:14] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
  72. # [14:15] <@firebot> vinceyang15@gmail.com requested review from trev.saunders@gmail .com for attachment 633994 on bug 757503.
  73. # [14:15] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
  74. # [14:30] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  75. # [14:31] * shorlander_ is now known as shorlander
  76. # [14:45] * khuey|away is now known as khuey
  77. # [15:16] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  78. # [15:16] * ChanServ sets mode: +qo davidb davidb
  79. # [15:16] <@MarcoZ> WordPress and BuddyPress CSS confuse me. Oh, wait, CSS in general confuses me!
  80. # [15:16] <icaaq> MarcoZ: lol
  81. # [15:28] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
  82. # [15:29] * Joins: victorporof (victorporo@C7255039.602BD6C.79933D60.IP)
  83. # [15:29] * Joins: mike5w3c_ (MikeS@moz-B94F10FE.xgsspn.imtp.tachikawa.spmode.ne.jp)
  84. # [15:29] * Quits: mike5w3c (MikeS@moz-D1E89A92.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
  85. # [15:29] * mike5w3c_ is now known as mike5w3c
  86. # [15:31] <@firebot> trev.saunders@gmail.com granted review for attachment 633994 on bug 757503.
  87. # [15:31] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
  88. # [15:31] <@davidb> heyo
  89. # [15:33] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  90. # [15:36] <@davidb> any surkov sightings?
  91. # [15:36] <capella> in and out all weekend
  92. # [15:36] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  93. # [15:36] <tbsaunde> phi davidb
  94. # [15:37] <@davidb> heyo
  95. # [15:37] <@davidb> capella: thanks
  96. # [15:37] <capella> davidb: is ok
  97. # [15:38] <capella> davidb: modifed bug 570322 which you previously approved ... wanna re-approve?
  98. # [15:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, ---, markcapella, ASSI, random fail of image map (area) accessible creation
  99. # [15:39] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
  100. # [15:40] <@davidb> i'd want to see a patch
  101. # [15:43] <@firebot> markcapella@twcny.rr.com requested review from dbolter@mozilla.com for attachment 634008 on bug 570322.
  102. # [15:43] <@MarcoZ> capella: How did the try run on that go?
  103. # [15:43] <capella> Great! various android issues as usual ... image problems all corrected
  104. # [15:44] <@MarcoZ> capella: Nice!
  105. # [15:44] <@MarcoZ> icaaq: Yeah my biggest problem at the moment are color codes. There are so many ways to code these in CSS that it's really confusing.
  106. # [15:45] * @MarcoZ would need a tool where he could input a color code and it would give me a plain clear name such as "black", "a whiter shade of pale" or so.
  107. # [15:45] <icaaq> MarcoZ: then less-variables would make it easier for you :)
  108. # [15:46] <@MarcoZ> Yeah.
  109. # [15:48] <icaaq> I just started a new project where we use LESS, it's a bit difficult to know all the optimizations one can do. but I'm getting there. :)
  110. # [15:48] * icaaq giggles every time he looks at home.less
  111. # [15:51] <@MarcoZ> LOL
  112. # [15:51] <@firebot> dbolter@mozilla.com granted review for attachment 634008 on bug 570322.
  113. # [15:51] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, ---, markcapella, ASSI, random fail of image map (area) accessible creation
  114. # [15:53] <capella> davidb: I ran tests on local windows ... but on the first TRY push they showed up as errors requiring adjustments
  115. # [15:53] <@davidb> interesting
  116. # [15:54] <capella> yah - i though it was localized to linux builds but even WIN broke on the TRY
  117. # [15:54] * khuey is now known as khuey|away
  118. # [15:55] <capella> this ones nice and clean across the board
  119. # [15:56] * Joins: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com)
  120. # [15:58] * Quits: scott_gonzalez (scott_gonz@moz-44AB6E75.nc.res.rr.com) (Quit: scott_gonzalez)
  121. # [16:00] <@MarcoZ> davidb: Was comment #63 the reason you cc'd to bug 689623? :D
  122. # [16:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=689623 nor, --, ---, tnikkel, ASSI, layout needs to provide information on which images are visible or likely to be visible
  123. # [16:00] <@firebot> vinceyang15@gmail.com requested review from trev.saunders@gmail .com for attachment 634010 on bug 757503.
  124. # [16:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
  125. # [16:02] <capella> tbsaunde: is bug 765371 still on your radar?
  126. # [16:02] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=765371 nor, --, ---, markcapella, ASSI, decomtaminate GetSelected-RowCount / ColumnCount / CellCount on accessible tables
  127. # [16:05] * clown is now known as clown_mtg
  128. # [16:05] <@davidb> MarcoZ: heh
  129. # [16:05] <tbsaunde> capella: commenting now ctually
  130. # [16:06] <capella> thanks!
  131. # [16:19] * Quits: icaaq (Adium@moz-CF0EF6A6.cust.bredbandsbolaget.se) (Quit: Leaving.)
  132. # [16:31] * Joins: icaaq (Adium@moz-E7557AE7.telia.com)
  133. # [16:38] * Joins: richardschwerdtfeger (richardsch@A08B5B24.9DD4DBBF.6A7A197.IP)
  134. # [16:48] * @MarcoZ is playing around with the Firefox dev tools.
  135. # [16:55] * Quits: logbot (logbot@moz-58CB32ED.glob.com.au) (Client exited)
  136. # [16:56] <@firebot> markcapella@twcny.rr.com changed the Target Milestone on bug 570322 from --- to mozilla16.
  137. # [16:56] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, mozilla16, markcapella, ASSI, random fail of image map (area) accessible creation
  138. # [17:00] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
  139. # [17:03] * khuey|away is now known as khuey
  140. # [17:04] <@MarcoZ> davidb: With a lot of NVDA object navigation, the fields are actually useable. But I think we could do more clever things with properly labelling the different text fields, so that tabbing becomes smoother.
  141. # [17:05] <@davidb> MarcoZ: rypple
  142. # [17:05] <@davidb> ?
  143. # [17:05] <@MarcoZ> davidb: Sorry, "inspect element" in Firefox.
  144. # [17:05] <@davidb> ohhh
  145. # [17:05] <@davidb> MarcoZ: file patches :)
  146. # [17:10] <@firebot> trev.saunders@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 634010 on bug 757503.
  147. # [17:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757503 nor, --, ---, nobody, NEW, decomtaminate GetColumnIndexAt/GetRowIndexAt/GetRowAndColumnIndicesAt on accessible tables
  148. # [17:11] * Joins: logbot (logbot@moz-58CB32ED.glob.com.au)
  149. # [17:11] <@MarcoZ> davidb: Other tools like the style editor completely illude me what their usability concept is concerned.
  150. # [17:11] <@MarcoZ> They basically didn't implement *any* of the stuff that was so nicely accessible in Firebug.
  151. # [17:12] * clown_mtg is now known as clown
  152. # [17:12] <@davidb> what do you get?
  153. # [17:14] <@MarcoZ> In the Style editor, when I open it first, i get something that has a long data:// URL as its accessible name, and no interaction whatsoever. I can then click on something like a list with NVDA's mouse emulation, but arrowing in there doesn't five me any speech, either.
  154. # [17:15] <@MarcoZ> I can then tab, get some different stylesheet names, as "heading1" and "link", but no idea what these are or what to do with them.
  155. # [17:15] <@MarcoZ> davidb: I PM'ed ya.
  156. # [17:21] * Quits: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP) (Quit: Computer has gone to sleep.)
  157. # [17:27] * Joins: drexler (chatzilla@moz-6F22CAC7.e7jvsoe-gw.cust.sover.net)
  158. # [17:41] * Joins: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net)
  159. # [17:46] * Quits: icaaq (Adium@moz-E7557AE7.telia.com) (Quit: Leaving.)
  160. # [17:55] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  161. # [17:57] * Quits: @firebot (firebot@moz-F8C1AF63.carolina.res.rr.com) (Ping timeout)
  162. # [17:58] * Joins: icaaq (Adium@moz-CF0EF6A6.cust.bredbandsbolaget.se)
  163. # [18:01] * Joins: firebot (firebot@moz-F8C1AF63.carolina.res.rr.com)
  164. # [18:01] * ChanServ sets mode: +o firebot
  165. # [18:02] * Joins: davidb|mobile (yaaic@31ABB91E.F7EBA23.C3C3A9D0.IP)
  166. # [18:06] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  167. # [18:06] * ChanServ sets mode: +o jprmc
  168. # [18:12] * Joins: aaronlev (aaronlev@moz-654DF715.bstnma.fios.verizon.net)
  169. # [18:18] <icaaq> have any of you looked at https://marketplace.mozilla.org/ yet
  170. # [18:24] <@eeejay> https://wiki.mozilla.org/Accessibility/Mobile/GestureInterface
  171. # [18:25] <aaronlev> !seen marcoz
  172. # [18:25] <@firebot> marcoz was last seen 70 minutes and 23 seconds ago, saying 'davidb: I PM'ed ya.' in #accessibility.
  173. # [18:26] <aaronlev> MarcoZ: are you gone for the day?
  174. # [18:26] <aaronlev> i'm looking for a blind user who knows a lot of different screen readers on Windows
  175. # [18:27] * Quits: davidb|mobile (yaaic@31ABB91E.F7EBA23.C3C3A9D0.IP) (Ping timeout)
  176. # [18:33] <@MarcoZ> aaronlev: Was in a meeting with eeejay. What's the question?
  177. # [18:34] <@MarcoZ> icaaq: Yes I have, and I even filed a bug.
  178. # [18:37] * khuey is now known as khuey|away
  179. # [18:37] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
  180. # [18:37] <aaronlev> MarcoZ: I PM'd you
  181. # [18:37] * Parts: aaronlev (aaronlev@moz-654DF715.bstnma.fios.verizon.net)
  182. # [18:40] <@davidb> tbsaunde: jlin is going to ask you about laptops
  183. # [18:42] <icaaq> MarcoZ: I just got access today, *looks not so nice* imho
  184. # [18:44] <tbsaunde> davidb: ok
  185. # [18:45] <@firebot> dbolter@mozilla.com granted review for attachment 633623 on bug 764203.
  186. # [18:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=764203 nor, --, ---, nobody, NEW, Don't notify pivot change if it has not actually changed
  187. # [18:47] <@davidb> tbsaunde: just waiting to see if the x230 is available first
  188. # [18:48] <@MarcoZ> icaaq: When I looked, it actually wasn't that bad with a screen reader. You'd have to look yourself or ask someone else sighted for stuff like keyboard focus visibility, though.
  189. # [18:49] * Joins: margle (margle@moz-75218163.dsl.mweb.co.za)
  190. # [18:50] <tbsaunde> davidb: ok, no hurry should be around all day
  191. # [18:50] <@davidb> ack
  192. # [18:54] <icaaq> MarcoZ: I think it has some a weird structure, for example the more-link is before the content, the navigation button is not connected to the navigation items (a search form between) and stuff like that.
  193. # [18:54] <icaaq> MarcoZ: -some^
  194. # [18:55] <@MarcoZ> icaaq: I may have to revisit. When I looked a few weeks ago, it looked pretty straight-forward in NVDA, but this may have changed.
  195. # [18:56] <icaaq> MarcoZ: yes, I'll be filing in some bugs tonight :)
  196. # [19:00] <@MarcoZ> CC me on them please icaaq!
  197. # [19:00] * @MarcoZ is going off for the night. See you all tomorrow!
  198. # [19:00] <icaaq> MarcoZ: Will do :)
  199. # [19:02] * Quits: @MarcoZ (Daily@moz-7A1C740A.dip.t-dialin.net) (Quit: MarcoZ)
  200. # [19:24] * Quits: margle (margle@moz-75218163.dsl.mweb.co.za) (Quit: Computer has gone to sleep.)
  201. # [19:29] * khuey|away is now known as khuey
  202. # [19:30] * Joins: hub (hub@A5087023.2354C43D.D8E68FF6.IP)
  203. # [19:30] * ChanServ sets mode: +o hub
  204. # [19:32] * Joins: hub_ (hub@A5087023.2354C43D.D8E68FF6.IP)
  205. # [19:39] <@firebot> hub@mozilla.com requested review from dbolter@mozilla.com for attachment 634093 on bug 755048.
  206. # [19:40] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=755048 nor, --, ---, josh, NEW, Fix build warnings under accessibility/
  207. # [19:45] * Joins: askalski (akuda@moz-6A36EC49.ip.abpl.pl)
  208. # [19:45] * ChanServ sets mode: +o askalski
  209. # [19:45] <@askalski> hi everybody!
  210. # [19:46] <@askalski> davidb, hi!
  211. # [19:46] <@davidb> heyo!
  212. # [19:46] * @davidb starts vidyo 1:1
  213. # [19:46] * davidb is now known as davidb|mtg
  214. # [19:50] * Joins: tofumatt (tofumatt@moz-9995AABD.hosts.kicksass.ca)
  215. # [19:54] * Joins: margle (margle@moz-75218163.dsl.mweb.co.za)
  216. # [19:58] * Quits: margle (margle@moz-75218163.dsl.mweb.co.za) (Quit: Textual IRC Client: http://www.textualapp.com/)
  217. # [20:02] * Joins: Mic (Instantbir@moz-108E19BC.superkabel.de)
  218. # [20:04] <@firebot> dbolter@mozilla.com granted review for attachment 633682 on bug 756296.
  219. # [20:04] <@firebot> dbolter@mozilla.com requested feedback from marco.zehe@googlemail.com for attachment 633682 on bug 756296.
  220. # [20:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=756296 nor, --, mozilla16, eitan, NEW, nsIAccessiblePivot move to coordinate method
  221. # [20:16] <@firebot> dbolter@mozilla.com granted review for attachment 634093 on bug 755048.
  222. # [20:16] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=755048 nor, --, ---, josh, NEW, Fix build warnings under accessibility/
  223. # [20:23] <@firebot> markcapella@twcny.rr.com requested review from trev.saunders@gmail .com for attachment 634116 on bug 765512.
  224. # [20:23] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=765512 nor, --, ---, markcapella, ASSI, decomtaminate GetSelected (Cell / Column / Row) Indices() on accessible tables
  225. # [20:28] * davidb|mtg is now known as davidb
  226. # [20:34] * Quits: Mic (Instantbir@moz-108E19BC.superkabel.de) (Quit: Instantbird 1.2a1pre -- http://www.instantbird.com)
  227. # [20:38] <@hub> tbsaunde: if you prefer I can change the "default" to be an explicit fall thru for the enums
  228. # [20:40] <tbsaunde> hub: I'm worried that might be even more confusing when things don't work
  229. # [20:40] <tbsaunde> since sometimes we'd complain and other times not
  230. # [20:41] <@hub> tbsaunde: the thing is today you have a compilation warning and nothing else. and it has been ignored.
  231. # [20:42] <@hub> tbsaunde: default suppress it but suppress the safeguard for new enum values
  232. # [20:42] <@hub> tbsaunde: on the other hand I can explicitely handle the enums doing nothing. no warning but new enum value will cause one
  233. # [20:43] * Joins: arky (arky@moz-7B0110AD.mv.mozilla.com)
  234. # [20:43] <tbsaunde> hub: so, if it was me I'd be really tempted to check the spec to see how the compiler is allowed to screw me if I just ignore the warning
  235. # [20:44] <tbsaunde> the thing is I'm not entirely convinced that we shouldn't handle more values of the enum
  236. # [20:45] <tbsaunde> but that's hard to figure out I'd have to go dive through layout a bit, doesn't seem to bother anyone, so it seems less important than other things I could spend time on
  237. # [20:46] * tbsaunde shrugs
  238. # [20:47] <@hub> tbsaunde: the compiler will do nothing. if the case isn't handled it just does not do anything
  239. # [20:47] <@hub> tbsaunde: like an empty default:
  240. # [20:47] * Quits: drexler (chatzilla@moz-6F22CAC7.e7jvsoe-gw.cust.sover.net) (Ping timeout)
  241. # [20:48] <@hub> tbsaunde: the warning is here to tell you " you might have missed some cases"
  242. # [20:48] <@hub> now it does not seem to be doing any harm the way it is
  243. # [20:49] <@hub> but too much warnings kill the warnings
  244. # [20:49] <@hub> warnings are usefull
  245. # [20:49] <@davidb> i very much agree
  246. # [20:49] <@davidb> in gcc this is just the warning -Wswitch does
  247. # [20:50] <@davidb> which is turned on by -Wall
  248. # [20:50] <@davidb> adding the default case tells the compiler "yeah i'm not worried about missing cases here"
  249. # [20:50] * @davidb moves on
  250. # [20:53] <tbsaunde> hub: sure, but if you don't read the warnings all the time its not a big deal to have some pointless ones
  251. # [20:53] <@hub> I read warning ALL THE TIME
  252. # [20:53] <@hub> ok, lunch time
  253. # [20:54] <tbsaunde> davidb: I know what the warning is I'm just on the fence if I think supressing it is a great idea
  254. # [20:54] <tbsaunde> hub: well, then don't ;) find something better to do than watch output scroll by
  255. # [20:55] <@davidb> tbsaunde: so you think the warning is something we should be reminded about because it points out something to be concerned about?
  256. # [20:56] <@davidb> that would be a good reason
  257. # [20:56] <@davidb> hub: so handling the cases explicitly might be good… as per tbsaunde
  258. # [20:57] <@davidb> then when we add to the enum… the warning can remind us in case there is a useful reason to switch on it
  259. # [20:57] <@davidb> either way, I vote to get rid of warnings.
  260. # [20:58] <@davidb> then warnings will mean something again.
  261. # [20:58] <@davidb> IMO
  262. # [20:59] <tbsaunde> davidb: imo they already mean something
  263. # [20:59] <@davidb> not to everyone
  264. # [21:00] <tbsaunde> but basically since last summer I've had more important things to with my time than look at them
  265. # [21:00] <@davidb> not while they are just noise for some people
  266. # [21:00] <@davidb> that's fair… so let's solve it quick and move on
  267. # [21:00] <tbsaunde> sounds like a personal problem ;p
  268. # [21:00] <@davidb> heh
  269. # [21:00] <tbsaunde> I think if we're trying to fix something we should get it right
  270. # [21:01] <@davidb> yes.
  271. # [21:01] <tbsaunde> and be sure it is right
  272. # [21:01] <@davidb> i'm fine with that
  273. # [21:01] <tbsaunde> so, I guess another way to put it is I'd rather have warnings than paper over things
  274. # [21:01] <@davidb> there is a cost, you'll admit right?
  275. # [21:02] <tbsaunde> and being sure something is correct is neither easy or fast
  276. # [21:02] <@davidb> it can be.
  277. # [21:02] <@davidb> if trivial
  278. # [21:02] <tbsaunde> a cost to what warnings? sure but pretty small
  279. # [21:02] <@davidb> i'm not sure it is as small as you think
  280. # [21:02] <tbsaunde> why?
  281. # [21:03] <@davidb> humans.
  282. # [21:03] <tbsaunde> hu???
  283. # [21:03] <@davidb> ask ms2ger and jdm
  284. # [21:03] <tbsaunde> I still don't understand your point
  285. # [21:04] <@davidb> there are people who want to compile the codebase and see no warnings.
  286. # [21:04] <@davidb> blocking them has a cost.
  287. # [21:04] <tbsaunde> nd they're wrong
  288. # [21:04] <@davidb> i understand your position, i'm talking about cost.
  289. # [21:05] <tbsaunde> making people not introduce warnings also has a cost
  290. # [21:05] <tbsaunde> and making people review warning fixes also has a cost
  291. # [21:05] <@davidb> yes indeed.
  292. # [21:06] <@davidb> so
  293. # [21:06] <@davidb> i think this boils down to different opinions on what each cost is.
  294. # [21:06] <@davidb> but admitting them all is worthwhile i think.
  295. # [21:08] <tbsaunde> so, I admitt people who believe in fixing every warning exist
  296. # [21:08] <tbsaunde> but I think they're wasting their time by watching for warnings all the time and wasting my time looking at their patches
  297. # [21:09] <@davidb> I guess it boils down to wanting to be able to notice warnings that are worth noticing.
  298. # [21:09] <@davidb> Does that make sense?
  299. # [21:10] <capella> fwiw - i think we over do warning around here ... "hey this might happen someoday" is not useful
  300. # [21:10] <capella> if 99.99% it never will
  301. # [21:10] <@davidb> capella: are you talking about compiler warnings?
  302. # [21:10] <tbsaunde> I suppose
  303. # [21:11] <capella> yeh - abd situationbal also
  304. # [21:11] <capella> (spelling)
  305. # [21:11] <@davidb> I dunno… the compiler warning thing isn't black and white for me.
  306. # [21:12] <tbsaunde> davidb: I guess where I differ is I don't think its that bad to have extra warnings if you just go through and read all of them say every couple months
  307. # [21:12] <@davidb> or before channel migration i guess
  308. # [21:13] <tbsaunde> I guess that would be fine
  309. # [21:13] <@davidb> I'm in danger of bringing up our #pragma discussion again… but let's not go there.
  310. # [21:13] <tbsaunde> though maybe a bit frequent for my taste, and if anything actually is bad I'd think taking it on aurora would be fine
  311. # [21:13] <@davidb> sure… but with awareness that we looked at it.
  312. # [21:14] <@davidb> that it was noticed.
  313. # [21:14] <@davidb> i must admit my first paid C++ work (in 1995) required 0 errors, 0 warnings for commits
  314. # [21:15] <capella> ack
  315. # [21:15] <@davidb> it has pros and cons
  316. # [21:15] <@davidb> the warnings were useful to me back then… at least some percentage of the time
  317. # [21:16] <capella> sometimes achieving results requires warnings to be accepted as suitable level of risk
  318. # [21:16] <tbsaunde> they are certainly sometimes useful
  319. # [21:16] <capella> (forgive if i intrude :) )
  320. # [21:16] <@davidb> heh the key is "to me" … because i am otherwise lazy
  321. # [21:16] <@davidb> capella: not at all
  322. # [21:16] <tbsaunde> the thing is its impossible to be completely correct in all cases everywhere
  323. # [21:17] <@davidb> i think i understand tbsaunde's position better now
  324. # [21:17] <@davidb> this sort of reminds me of the oranges debate… about turning off those tests vs leaving them
  325. # [21:18] * Parts: tofumatt (tofumatt@moz-9995AABD.hosts.kicksass.ca)
  326. # [21:18] <@davidb> and the idea to come up with 'known oranges' suppression etc
  327. # [21:18] <@davidb> very similar actually
  328. # [21:18] <capella> my techie side wants to have code as pure as possible ... business side requires results .... my libra tilts
  329. # [21:18] <@davidb> you should see a doctor about that
  330. # [21:18] <capella> :p
  331. # [21:19] <tbsaunde> its basically the same issue
  332. # [21:20] <capella> right ... if theres time to do it right, thats the answer
  333. # [21:20] <tbsaunde> accept tests failing is a little more scary than warnings since warnings mean rough "this might not be compiled the way you want which might lead to wrong results"
  334. # [21:21] <@davidb> tbsaunde: and we went with leaving the oranges going… but in this case they have to be noticed/starred… which is a difference.
  335. # [21:21] <@davidb> yeah
  336. # [21:21] <tbsaunde> and tests failing are "you program appears to not working as specified"
  337. # [21:21] <@davidb> true
  338. # [21:22] <capella> could be "have you designed your test properly"?
  339. # [21:22] <capella> lol - devils advocate
  340. # [21:22] <tbsaunde> true, that's certainly more effort than checking warnings every know and again, but simple test fails aren't to bad to star imo
  341. # [21:22] <@davidb> capella: it could be fragile test yeah
  342. # [21:22] <@davidb> tbsaunde: especially when we have philor heheh
  343. # [21:23] <tbsaunde> true
  344. # [21:23] <capella> philor will mock you!
  345. # [21:23] <@davidb> that's how he shows his love
  346. # [21:23] <tbsaunde> sure, but he'll get more anoyed by having to dig through logs to find out why android isn't working
  347. # [21:24] <capella> he told me that once :) if you get that wrong, I will mock you mercilessly
  348. # [21:24] <capella> android needs lots of love
  349. # [21:28] <@davidb> ok getting into other work now… i'll loop back if casual conversation turns up anything interesting about the warnings debate.
  350. # [21:29] <capella> i have a beginners question im too embarassed to ask ...
  351. # [21:29] <tbsaunde> davidb: yeah, Im going back to reviewing once I stop catching up on irc
  352. # [21:29] <@davidb> capella: this is a safe channel
  353. # [21:29] * Quits: @firebot (firebot@moz-F8C1AF63.carolina.res.rr.com) (Ping timeout)
  354. # [21:29] * Joins: margle (margle@moz-75218163.dsl.mweb.co.za)
  355. # [21:30] <capella> who services the call on line 258 ...
  356. # [21:30] <capella> http://mxr.mozilla.org/mozilla-central/search?string=GetSelectedColumnsArray&find=&findi=&;filter=^[^\0]*%24&hitlimit=&tree=mozilla-central
  357. # [21:30] <@davidb> can you bit.ly that?
  358. # [21:31] <capella> bit.ly == pastebin?
  359. # [21:32] <capella> it has to be an interface thing .idl ... why i can find the search ?
  360. # [21:32] <tbsaunde> capella: the impl on line 628
  361. # [21:32] <capella> c++ cant call a function with one vs two parms tho ??
  362. # [21:33] <tbsaunde> I assume the second arg is declared as optional, but haven't actually checked
  363. # [21:33] <capella> optional? c++? thats new to me
  364. # [21:34] <capella> i though itd be a mismatch error
  365. # [21:34] <capella> oh! i see it!!!
  366. # [21:34] <capella> function declaration thing
  367. # [21:34] <tbsaunde> capella: yes, C++ has optional args
  368. # [21:35] <tbsaunde> well, more default values
  369. # [21:35] <capella> default value kinda thing ... yes!
  370. # [21:36] <capella> i missed the "= nsnull" part yesterday night
  371. # [21:36] <capella> not sure i like that .... but will remember the trick
  372. # [21:39] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  373. # [21:43] <tbsaunde> khuey: ping, how much do you hate the use of VPATH in bug 762770?
  374. # [21:44] * Joins: firebot (firebot@moz-F8C1AF63.carolina.res.rr.com)
  375. # [21:44] * ChanServ sets mode: +o firebot
  376. # [21:45] <khuey> tbsaunde: why do you need it?
  377. # [21:46] <tbsaunde> khuey: I don't, surkov wants to put files under accessible/src/windows/uia/ instead of accessible/src/msaa/
  378. # [21:47] <tbsaunde> and aparently had linking trouble when he tried to just add another makefile
  379. # [21:47] <@hub> tbsaunde: I watch warning because it happens that sometime they show that my code is wrong... like calling the wrong method in Obj-C
  380. # [21:47] <khuey> I'd be interested to know what trouble he had
  381. # [21:47] <khuey> I'd like to avoid adding VPATH stuff
  382. # [21:47] <khuey> it makes the build slow
  383. # [21:47] <@hub> davidb: I was taking that changed approach to handle the enum values explicitely, definitely.
  384. # [21:48] <tbsaunde> khuey: well, I'm absolutely fine with that, but don't have much of a place to argue about build stuff with him from :)
  385. # [21:48] <tbsaunde> hub: I'm not sure I follow that, since *all* warnings may mean your code is wrong
  386. # [21:50] <@hub> tbsaunde: hence properly fixing the warning
  387. # [21:50] <@hub> hence me watching them
  388. # [21:50] <@hub> to be honest we should car about warnings as much as code style
  389. # [21:51] <@firebot> New Core - Disability Access APIs bug 765878 filed by dbolter@mozilla.com.
  390. # [21:51] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=765878 nor, --, ---, nobody, NEW, ARIA abstract role to MSAA role mapping is spec violation
  391. # [21:51] <tbsaunde> hub: if it actually is an issue yes at some point, but we already have plenty of filed bugs for issues that actually bother people
  392. # [21:51] <tbsaunde> so I find a bit odd people decide they'd rather care about things that maybe might cause a problem
  393. # [21:52] <tbsaunde> than things that clearly are causing people problems
  394. # [21:52] <tbsaunde> hub: and caring about warnings means spending time and going out of your way
  395. # [21:52] <capella> tbsaunde++
  396. # [21:53] <capella> i learned a new trick today :)
  397. # [21:53] <tbsaunde> heh
  398. # [21:54] <@davidb> i would like access to BMO
  399. # [21:58] <tbsaunde> khuey: so, I guess I'll cc you, andmaybe Alex can explain what trouble he had
  400. # [21:58] <khuey> ok
  401. # [21:58] <khuey> sgtm
  402. # [21:58] <@davidb> can you access bugzilla?
  403. # [21:59] <capella> who me?
  404. # [22:00] <@davidb> anyone?
  405. # [22:00] <capella> yes i can - didnt you just file a bug?
  406. # [22:00] <@davidb> i did, and now i can't access it
  407. # [22:00] <@davidb> nor most of toronto
  408. # [22:00] <capella> i see it
  409. # [22:00] <@davidb> possible all of toronto
  410. # [22:00] <@davidb> ok thanks
  411. # [22:02] * Quits: victorporof (victorporo@C7255039.602BD6C.79933D60.IP) (Ping timeout)
  412. # [22:03] <@davidb> back
  413. # [22:03] * Quits: arky (arky@moz-7B0110AD.mv.mozilla.com) (Quit: Leaving)
  414. # [22:03] <capella> gremlins
  415. # [22:04] * Joins: victorporof (victorporo@C7255039.602BD6C.79933D60.IP)
  416. # [22:10] * Joins: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se)
  417. # [22:13] <capella> talk on #developers about bmo in and outs
  418. # [22:13] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
  419. # [22:16] <icaaq> is bugzilla down?
  420. # [22:17] <capella> in and out
  421. # [22:17] <capella> being tracked by bug 765670 apparently, but I can't confirm that because I can't load bugzilla
  422. # [22:17] <capella> says the #developers crowd
  423. # [22:29] <@firebot> hub@mozilla.com granted checkin for attachment 634093 on bug 755048.
  424. # [22:29] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=755048 nor, --, ---, josh, NEW, Fix build warnings under accessibility/
  425. # [22:30] * Joins: mike5w3c_ (MikeS@moz-294BE65F.xgsspn.imtp.tachikawa.spmode.ne.jp)
  426. # [22:30] * Quits: mike5w3c (MikeS@moz-B94F10FE.xgsspn.imtp.tachikawa.spmode.ne.jp) (Ping timeout)
  427. # [22:30] * mike5w3c_ is now known as mike5w3c
  428. # [22:32] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  429. # [22:32] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
  430. # [22:48] <@hub> fyi, the "granted checkin" is "checkin+" that you should set when you commit a patch and [leave open] the bug for more
  431. # [22:53] <@hub> it is part of the latest change to the tree workflow.
  432. # [22:56] * Quits: margle (margle@moz-75218163.dsl.mweb.co.za) (Quit: Textual IRC Client: http://www.textualapp.com/)
  433. # [22:59] <capella> dang ... brb
  434. # [23:01] <@firebot> markcapella@twcny.rr.com granted checkin for attachment 634008 on bug 570322.
  435. # [23:01] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=570322 nor, --, mozilla16, markcapella, ASSI, random fail of image map (area) accessible creation
  436. # [23:01] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
  437. # [23:07] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  438. # [23:14] * Quits: kolombiken (Adium@moz-72AAF78F.bredband.comhem.se) (Quit: Leaving.)
  439. # [23:29] * Quits: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net) (Ping timeout)
  440. # [23:36] * Quits: @jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  441. # [23:39] * Parts: richardschwerdtfeger (richardsch@A08B5B24.9DD4DBBF.6A7A197.IP)
  442. # [23:43] * Joins: nhirata (nhirata.bu@moz-6D599AA6.hsd1.ca.comcast.net)
  443. # [23:45] * Quits: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  444. # [23:52] * Joins: jhk (jhk@8E6C34C1.A3F9767A.1C37C358.IP)
  445. # Session Close: Tue Jun 19 00:00:00 2012

The end :)