Review of "Alternate Menu for Hplip2" version 24

Details Page Preview

control your hp printers by calling the device manager hp-toolbox, also some useful links Motivation: the hp-systray doesn't work reliably under gnome shell you need to have installed hplip in order to use this Choice of using a printer icon or a hp_logo.png if it's installed in the same place as mine on Ubuntu you could use symbolic links to fake the path. This is a replacement for the old "Alternate Menu for Hplip" which doesn't work under the new Gome-Shell I have cleaned it up a bit and it has a few new menu's but it is still basically the same thing. Added even more menus all most all system settings ones. New: the menu will be too big if your resolution is way too low like 800x600 I have implemented a compact option in the settings, if you use it the menu will become a two colomn compact format and will fit even on such a low resolution. Dutch language support added by Vistaus (Heimen Stoffels) https://github.com/Vistaus In the past you needed to restart gnome-shell after upgrades so either reboot, log out and back in again, or if your using X11 then alt+F2 then run r; But that should be good now in most cases.

Extension Homepage
https://github.com/grizzlysmit/hplip-menu2-grizzlysmit.smit.id.au

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Previous Reviews on this Version

JustPerfection rejected
1. You cannot call functions and methods in global scope (line 33 `CompactMenu.js`): [EGO Review Guidelines: Initialization](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-initialization-for-static-resources) 2. You shouldn't remove line 408-409 `extension.js`: [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 3. You cannot import `Gtk` to the GNOME Shell process (line 19 `CompactMenu.js`): [EGO Review Guidelines: import](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#do-not-import-gtk-libraries-in-gnome-shell)
grizzly@smit.id.au posted a review
on 3. I take your point but is import [DirectionType, PolicyType] from 'gi://Gtk'; OK as I only use constants out off these enums or I could just create my own constants inside my Object ??
grizzly@smit.id.au posted a review
on 3. I take your point but is import [DirectionType, PolicyType] from 'gi://Gtk'; OK as I only use constants out off these enums or I could just create my own constants inside my Object ??
grizzly@smit.id.au posted a review
Never mind I'll put my own constants from the docs it looks like they wont change from system to system.
JustPerfection posted a review
You can use: https://gjs-docs.gnome.org/st14~14/st.directiontype and https://gjs-docs.gnome.org/st14~14/st.policytype