node-process-windows icon indicating copy to clipboard operation
node-process-windows copied to clipboard

getActiveWindow function doesn't do anything.

Open mike-eason opened this issue 7 years ago • 5 comments

The getActiveWindow doesn't return anything, have you forgotten to commit something?

/**
 * Get information about the currently active window
 *
 * @param {function} callback
 */
function getActiveWindow(callback) {
    callback = callback || noop;

    if (!isWindows) {
        callback("Non-windows platforms are currently not supported");
    }
}

Thanks, Mike

mike-eason avatar Apr 11 '17 10:04 mike-eason

+1

kaue avatar Aug 26 '17 15:08 kaue

+1

andresullua avatar Feb 02 '18 15:02 andresullua

@bryphe If it's not implemented its an enhancement not a bug right?

kaue avatar Feb 02 '18 18:02 kaue

+1

SpreadfilmsSamuel avatar Sep 12 '18 08:09 SpreadfilmsSamuel

I created a fork which fixes this among other bugs here . I've opened a pull request, but I'm not sure the author maintains this repo anymore.

adamsoutar avatar Dec 03 '18 08:12 adamsoutar