The default bug view has changed. See this FAQ.

[SeaMonkey 2.1, mochitest-browser-chrome] Double id='addBookmarkKb' detected in navigator.xul

RESOLVED FIXED in seamonkey2.1b1

Status

SeaMonkey
Bookmarks & History
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: Robert Kaiser, Assigned: Robert Kaiser)

Tracking

(Blocks: 1 bug, {intermittent-failure})

Trunk
seamonkey2.1b1
intermittent-failure
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [sm-perma][cc-orange])

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
This probably comes from places bookmarks:

Double id='addBookmarkKb' detected in chrome://navigator/content/navigator.xul:
  Tree 0:
    key addBookmarkKb [39]
    keyset navKeys [24]
    window main-window [25]
    #document undefined [0]
  Tree 1:
    key addBookmarkKb [9]
    keyset navigationKeys [48]
    keyset navKeys [24]
    window main-window [25]
    #document undefined [0]
ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/suite/common/tests/chrome/test_idcheck.xul | check id: navigator.xul#addBookmarkKb


Thanks to Callek for finding that one!
(Assignee)

Comment 1

7 years ago
http://mxr.mozilla.org/comm-central/search?string=addBookmarkKb tells a pretty good story. I guess we should kill it from platformNavigationBindings.xul or navigatorOverlay.xul - but as it's only on the former in non-Linux, I guess that's what we should change.
(Assignee)

Comment 2

7 years ago
Created attachment 468166 [details] [diff] [review]
remove platform-specific keys

We have accel,shift in use for tabs already now, so this is a conflict as well. The best thing is to remove the platform-specific bindings, from what I'm seeing.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #468166 - Flags: review?(neil)

Comment 3

7 years ago
Comment on attachment 468166 [details] [diff] [review]
remove platform-specific keys

Fortunately the Ctrl+Shift+D shortcut is no longer reserved by GTK. (Sadly you're now trying to use the reserved Ctrl+Alt+D shortcut. Doesn't anyone read the Error Console any more? I really need to review more of those Places patches...)
Attachment #468166 - Flags: review?(neil) → review+
(Assignee)

Updated

7 years ago
Blocks: 452942
(Assignee)

Comment 4

7 years ago
Pushed as http://hg.mozilla.org/comm-central/rev/64ad57c8da27

(In reply to comment #3)
> (Sadly
> you're now trying to use the reserved Ctrl+Alt+D shortcut. Doesn't anyone read
> the Error Console any more? I really need to review more of those Places
> patches...)

I read it but didn't find a different combination to use for something I wasn't sure if it would even stick. And actually, it might be a good idea to just remove that option when the fast bookmarking button lands. Though I wonder if FF has a shortcut for that one...
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
Flags: in-testsuite-

Comment 5

7 years ago
(In reply to comment #4)
> And actually, it might be a good idea to just
> remove that option when the fast bookmarking button lands.
The fast bookmarking button doesn't help keyboard users, who are the people interested in the shortcut...
(Assignee)

Comment 6

7 years ago
(In reply to comment #5)
> (In reply to comment #4)
> > And actually, it might be a good idea to just
> > remove that option when the fast bookmarking button lands.
> The fast bookmarking button doesn't help keyboard users, who are the people
> interested in the shortcut...

Well, the question is if they need shortcuts for both "add bookmark without being prompted" and "add bookmark with an add bookmark panel showing".
I'd say keyboard shortcut for "Add Bookmark" should trigger fast path, second invoke of keyboard shortcut should prompt (also allowing remove).
(Assignee)

Comment 8

7 years ago
(In reply to comment #7)
> I'd say keyboard shortcut for "Add Bookmark" should trigger fast path, second
> invoke of keyboard shortcut should prompt (also allowing remove).

This basically means putting the shortcut on the icon and removing the one on the "..." menu item. I also think the other menu item for simple adding should go away. Still, let's put this on a different bug.
Keywords: intermittent-failure
Whiteboard: [sm-perma][orange] → [sm-perma]

Updated

4 years ago
Whiteboard: [sm-perma] → [sm-perma][cc-orange]
You need to log in before you can comment on or make changes to this bug.