darkxst / multiple-monitor-panels

Add panels including AppMenu to each additional monitor.
GNU General Public License v3.0
39 stars 27 forks source link

Multiple issues regarding workspaces switcher #8

Open gnustomp opened 12 years ago

gnustomp commented 12 years ago

.1. There is a 15px gap between between the edge of the screen and the workspaces switcher.

.2. It sometimes doesn't invert when displayed on the left.

It is displayed normally (incorrectly, with the rounded corners on the left) when the shell is restarted. It is corrected if you disable and re-enable the left option, and on startup.

.3. It doesn't get removed even if display-workspace is set to false.

.4. Clicking and dragging to change workspaces doesn't work on the displays with the extra panels.

darkxst commented 12 years ago
1 + 2, do these happen straight away after restarting gnome-shell,
or some time later after playing with settings?
3. this setting currently only takes effect on a restart of
gnome-shell, however I will fix that soon.On 03/10/12 15:34, gnustomp wrote:

      There is a 15px gap between between the edge of the screen
        and the workspaces switcher.

      It sometimes doesn't invert when displayed on the left.

      It doesn't get removed even if display-workspace is set to
        false.

    —
    Reply to this email directly or view
      it on GitHub.
gnustomp commented 12 years ago

The gap is always there, but the orientation of the switcher is only wrong when the shell is restarted (immediately).

darkxst commented 12 years ago
what resolutions are your monitors?On 03/10/12 19:38, gnustomp wrote:

  The gap is always there, but the orientation of the switcher is
    only wrong when the shell is restarted.

    —
    Reply to this email directly or view
      it on GitHub.
gnustomp commented 12 years ago

1920_1080 and 1680_1050. The uneven resolution causes the gap, as I tried with with 1680*1050 for both, and there was no gap. The orientation was still wrong on restart, though.

darkxst commented 12 years ago

this cc25d420f0ea77c427510da3488d36f26ce9a443 should fix the gap, and maybe orientation

gnustomp commented 12 years ago

The patch breaks thumbnails.js.

    JS ERROR: !!!   Exception was: TypeError: this._setBackgroundClass is not a function
    JS ERROR: !!!     message = '"this._setBackgroundClass is not a function"'
    JS ERROR: !!!     fileName = '"/home/user/.local/share/gnome-shell/extensions/extra-panels@darkxst.feathertop.org/thumbnails.js"'
    JS ERROR: !!!     lineNumber = '91'
    JS ERROR: !!!     stack = '"([object _private_Shell_GenericContainer],[object _private_Clutter_ActorBox],0)@/home/user/.local/share/gnome-shell/extensions/extra-panels@darkxst.feathertop.org/thumbnails.js:91
wrapper([object _private_Shell_GenericContainer],[object _private_Clutter_ActorBox],0)@/usr/share/gjs-1.0/lang.js:204
([object _private_Shell_GenericContainer],[object _private_Clutter_ActorBox],0)@/usr/share/gnome-shell/js/ui/main.js:119

-This might be helpful.-

Without this.setBackgroundClass(), the gap is fixed(!), but not the orientation. The orientation is wrong when the shell starts.

Workaround for orientation: disable then re-enable workspace-left.