search

jonathanKingston / sea-containers

Sidebar for managing container tabs
https://addons.mozilla.org/en-GB/firefox/addon/sea-containers/
Mozilla Public License 2.0
32 stars 6 forks source link

cannot drag and move/re-order open tabs inside the same container #5

Closed ammarnajjar closed 7 years ago

ammarnajjar commented 7 years ago

One cannot drag and move/re-order open tabs inside the same container. I would provide debug info, but I don't know how, sorry.

jonathanKingston commented 7 years ago

Hey @ammarnajjar!

Ctrl+Alt+J should load up the browser console which should give a tonne of debugging.

Also about:debugging gives an extension specific view of that.

I have only tested dragging on Ubuntu, it's somewhat buggy in not permitting dragging to the first position etc.

ammarnajjar commented 7 years ago

Thanks @jonathanKingston for the tip. Hope that the following console output is enough:

this[module] is undefined  nsBrowserGlue.js:174
    observe resource://app/components/nsBrowserGlue.js:174:9
    initializeWindow resource:///modules/sessionstore/SessionStore.jsm:1162:9
    onBeforeBrowserWindowShown/< resource:///modules/sessionstore/SessionStore.jsm:1307:9
    process resource://gre/modules/Promise-backend.js:922:23
    walkerLoop resource://gre/modules/Promise-backend.js:806:7
    bound walkerLoop self-hosted:944:17
    bound bound walkerLoop self-hosted:944:17
Key event not available on some keyboard layouts: key=“r” modifiers=“accel,alt” id=“key_toggleReaderMode”  browser.xul
Key event not available on some keyboard layouts: key=“i” modifiers=“accel,alt,shift” id=“key_browserToolbox”  browser.xul
Error in parsing value for ‘max-width’.  Declaration dropped.  browser.xul
Unknown property ‘enable-background’.  Declaration dropped.  sea-containers.svg
Property contained reference to invalid variable.  Error in parsing value for ‘background-image’.  Falling back to ‘initial’.  containers.css:91:1942
Unknown property ‘enable-background’.  Declaration dropped.  usercontext.svg
Unknown property ‘enable-background’.  Declaration dropped.  usercontext.svg
Unknown property ‘enable-background’.  Declaration dropped.  usercontext.svg
Unknown property ‘enable-background’.  Declaration dropped.  usercontext.svg
Property contained reference to invalid variable.  Error in parsing value for ‘fill’.  Falling back to ‘inherit’.  containers.css:96:2104

A screenshot can also be found here

jonathanKingston commented 7 years ago

@ammarnajjar I don't think it is any of that. However I can confirm in refactoring the code I broke it for all platforms sorry. Working on it.

jonathanKingston commented 7 years ago

@ammarnajjar this will go out in the next release, I will likely be doing some follow up fixes for the other mentioned issues. Thanks for alerting me of that!