make draft comments focusable
Created attachment 80856 [details] Patch
Comment on attachment 80856 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=80856&action=review This is fine, but consider the comments below. BTW, I love j/k. :) > Websites/bugs.webkit.org/code-review.js:1280 > - if (node.hasClass(className)) { > + if (focus_type == FOCUS_TYPE.COMMENT && (node.hasClass('frozenComment') || node.hasClass('previousComment')) || > + focus_type == FOCUS_TYPE.DIFF_BLOCK && node.hasClass('DiffBlock')) { I'd just pass in a function that you call at this branch. That stops this code from having to know about all the different types.
Committed r77363: <http://trac.webkit.org/changeset/77363>
Comment on attachment 80856 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=80856&action=review >> Websites/bugs.webkit.org/code-review.js:1280 >> + focus_type == FOCUS_TYPE.DIFF_BLOCK && node.hasClass('DiffBlock')) { > > I'd just pass in a function that you call at this branch. That stops this code from having to know about all the different types. Done. That's much better.