PFA updated patch for RM#3295. The patch is to fix the issue in firefox
Kindly review. Please try in firefox also.
Thanks and Regards,
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"
On Wed, May 9, 2018 at 6:58 PM, Dave Page <dp...@pgadmin.org> wrote:
> Thanks, applied.
> On Wed, May 9, 2018 at 2:10 PM, Khushboo Vashi <
> khushboo.va...@enterprisedb.com> wrote:
>> Please find the attached patch to fix RM #3295: Shortcut key conflict
>> After opening the debugger, the shift+alt+o short-cut opens the Object
>> menu as well as perform the step over to the debugger. This issue has been
>> fixed in this patch.
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
diff --git a/web/pgadmin/tools/debugger/static/js/direct.js b/web/pgadmin/tools/debugger/static/js/direct.js
index b9cb2ec..42b2408 100644
@@ -1850,8 +1850,13 @@ define([
// Register the callback when user set/clear the breakpoint on gutter area.
self.editor.on('gutterClick', self.onBreakPoint.bind(self), self);
- // Set focus to the debugger container
+ /* Set focus to the debugger container
+ * Focus does not work in firefox without tabindex attr
+ * so, setting focus to parent of $container which is #container