Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Option for showing open window previews on click action. #465

Closed
Amplificator opened this issue Apr 7, 2017 · 19 comments
Closed

Option for showing open window previews on click action. #465

Amplificator opened this issue Apr 7, 2017 · 19 comments

Comments

@Amplificator
Copy link

Now that we have the "Show open windows previews" when we right click on an icon, it would be great if we could have it go straight to the "All Windows" section to immediately show the window previous.
And if we could set that as the left mouse click action that would be awesome :)

@franglais125
Copy link
Contributor

franglais125 commented Apr 7, 2017

I think this is a duplicate of #400

@Amplificator
Copy link
Author

Indeed, for the most part. Difference seems to be that I requested that it could be chosen to happen on left-click.

@franglais125
Copy link
Contributor

Yes, it's getting a bit hard, as right-click, left-click and hover are possible options. Maybe a duplicate of this one #418 in that case?

If you want left click, check out this branch: https://github.com/franglais125/dash-to-dock/tree/click_window_preview

I'm still trying to make it nicer, but it works already.

@Amplificator
Copy link
Author

Amplificator commented Apr 7, 2017

I get an error on that branch.
Removed the old plugin, restarted Gnome, cloned the branch, restarted gnome again.

I get this:

GLib.FileError: Failed to open file '/home/martin/.local/share/gnome-shell/extensions/[email protected]/schemas/gschemas.compiled': open() failed: No such file or directory

Stack trace:
  getSettings@/home/martin/.local/share/gnome-shell/extensions/[email protected]/convenience.js:64
  Settings<._init@/home/martin/.local/share/gnome-shell/extensions/[email protected]/prefs.js:58
  wrapper@resource:///org/gnome/gjs/modules/lang.js:178
  _Base.prototype._construct@resource:///org/gnome/gjs/modules/lang.js:110
  Class.prototype._construct/newClass@resource:///org/gnome/gjs/modules/lang.js:213
  buildPrefsWidget@/home/martin/.local/share/gnome-shell/extensions/[email protected]/prefs.js:675
  Application<._selectExtension@resource:///org/gnome/shell/extensionPrefs/main.js:90
  wrapper@resource:///org/gnome/gjs/modules/lang.js:178
  Application<._onCommandLine@resource:///org/gnome/shell/extensionPrefs/main.js:239
  wrapper@resource:///org/gnome/gjs/modules/lang.js:178
  main@resource:///org/gnome/shell/extensionPrefs/main.js:377
  @<main>:1

@franglais125
Copy link
Contributor

Did you install with make && make install?

What you are having is essentially a missing option in the schemas, which means it wasn't recompiled.

@Amplificator
Copy link
Author

Yeah, I forgot that, already fixed that though, but the window preview doesn't happen here.. the behavior is the same as when choosing to cycle between windows.

@Amplificator
Copy link
Author

Basically it should open the right-click menu and already have opened the "All Windows" tab.

@franglais125
Copy link
Contributor

Did you select the option in the settings?

screenshot from 2017-04-07 18-48-10

@Amplificator
Copy link
Author

Yes :)

@franglais125
Copy link
Contributor

I'm a bit puzzled... It should work for icons with more than 1 window.

screenshot from 2017-04-07 18-58-43

@Amplificator
Copy link
Author

It manages to show 1½ preview image and then it freezes up Gnome completely.

@franglais125
Copy link
Contributor

Well, sorry to hear that. :(

What version of Gnome-Shell do you use? And do you get any error messages that can help debug?

@Amplificator
Copy link
Author

Amplificator commented Apr 7, 2017

Gnome 3.22.3 running on Arch.

No errors, it just locks up :/

@franglais125
Copy link
Contributor

@Amplificator here is another branch, with a different implementation https://github.com/franglais125/dash-to-dock/tree/click_window_preview_fromPopupMenu

This one actually opens the popupMenu (same as right click), but forces the window preview section to be open.

@Amplificator
Copy link
Author

@franglais125 that branch works flawlessly, thank you :)

Would you mind making a pull request to have it merged to the official plugin?

@franglais125
Copy link
Contributor

@Amplificator I actually don't like the latest branch I linked, as it feels more hacky: I'm opening the right-click menu and setting a variable to keep it forcibly open.

The first solution was cleaner, while introducing more code. I now logged in into a X session and I can reproduce the freeze. I had no problems under Wayland!

In the end, I'll try to come up with a better solution, from the point of view of the code, and that doesn't crash your shell ;)

@franglais125
Copy link
Contributor

@Amplificator would you mind trying again the first branch? I pushed a fix which hopefully addresses your issue.
https://github.com/franglais125/dash-to-dock/tree/click_window_preview

@Amplificator
Copy link
Author

Amplificator commented Apr 8, 2017

@franglais125 This fix seems to work :)

I actually like this version better because it only shows previews and none of the buttons like the 2nd branch you made did.

Can it be made to not show above the main Gnome bar? When not transparent it hides whatever is on the bar. If it started just below it I think it would look better.

screenshot

Don't mind the semi-transparentness of it, that is not an issue but something I did to take the screenshot.

@franglais125
Copy link
Contributor

From #504:

This was merged into master [0772814]..[3c9f7f3].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants