search

asus-linux-drivers / asus-numberpad-driver

Maintained feature-rich linux driver for NumberPad(2.0) on Asus laptops. NumberPad(2.0) is illuminated numeric keypad integrated to touchpad which appears when is done tap on top right corner of touchpad for atleast 1s by default (configurable) or slide gesture from top right/left corner to the center, the left shows calc app aswell (configurable).
GNU General Public License v2.0
257 stars 19 forks source link

Calculator open multiple instances #195

Closed encrustace closed 1 month ago

encrustace commented 1 month ago

Short Description

If the calculator is already opened, it open another one.

Expected behaviour, it should toggle the app, if calculator already opened then close it and if closed then open it.

New dependencies

No response

Additional Info

No response

ldrahnik commented 1 month ago

@encrustace Hi, this driver send Calc key and associated script for Calc key do toggling. I need detailed description to understand you - I mean which calculator, is used script from this driver or your own, use case step by step etc.

encrustace commented 1 month ago

System: Arch Linux with Gnome Desktop Used install.sh script to install. I am using default script provided by this setup. Using gnome calculator from flatpak.

ldrahnik commented 1 month ago

@encrustace And this script in your environment do not do toggling? https://github.com/asus-linux-drivers/asus-numberpad-driver/blob/master/scripts/gnome_calculator_toggle.sh

encrustace commented 1 month ago

@ldrahnik It does work when I installed gnome-calculator from pacman library, but was not working with flatpak one, I guess it does not have support of flatpak version, I can close the issue, or can keep it open to track if you plan to give support of flatpak gnome app.

ldrahnik commented 1 month ago

@encrustace Try the latest master branch.

encrustace commented 1 month ago

@ldrahnik I tried but there was an issue, I have created a PR-197 Please have a look.

ldrahnik commented 1 month ago

@encrustace Please, try the latest master. In case still does not work, reopen PR, I will check it and merge it. Sorry, I committed in a hurry not working version.

encrustace commented 1 month ago

@ldrahnik Yes, now it is working like butter, Thanks