Closed ChristinWhite closed 6 years ago
Same happens to me, but I can circumvent it by removing the size
property from the config. If I put it back, regardless of whether it's a single integer or an object with width and height, it throws the same error.
UPDATE: looks like it only happens when I assign an object to size
. @christopherdwhite, it might help you pinpoint which configuration is the root for your issues if you clear out the overlay
object and then re-add each property to see which one causes the error to re-occur.
Sorry for not addressing this earlier, I somehow completely missed the notification. That said, make sure the size is an object as that is required in the new config to allow setting both height and width. Example config of my own:
overlay: {
hasShadow: true,
hideOnBlur: true,
position: 'center',
tray: true,
size: {
width: 0.4,
height: 0.4
}
Note that 0.4 on both are also the default values
If this issue still persists after correcting your config please re-open it. Otherwise it will be closed for now since as far as I can tell it doesn't appear to be an issue with the plugin itself.
My apologies @Favna, this fell off my radar but I just changed my configuration to a size object with width and height and I'm no longer getting an error. Looks like @cobrabr had nailed it. Thanks both of you guys!
When I change my config and set size
to be an object, I get the following error when I open Hyper:
If I move it back to a number, it works fine.
@cobrabr can you confirm that your Hyper properly updated the plugin to the latest version by opening the package.json
of the plugin folder wherever those are stored on MacOS? It should be 1.2.1
It looks like I had both hyper-overlay
and hyperterm-overlay
installed (honestly, I hadn't even noticed the change in the name :blush:).
I removed the old one and reinstalled hyper-overlay
-- the package.json
under .hyper_plugins
says latest
for hyper-overlay
, and the package.json
inside hyper-overlay
's folder says 1.2.1.
I had to make it a new name due to how NPM register works and I don't really feel like appealing for taking over the old project for it being abandoned (which is an option in NPM's publishing rules).
Good you found out why then.
Oh, it's still failing... I'm trying to re-pinpoint which configuration option is actually throwing things off.
It looks like it's the hotkeys
option -- have it set to ['Super+`'], which worked fine with the old version, but seems to be failing with the new one.
Sorry for the spam, but I think it's finally OK: looking at some other issues, I found that hotkeys
is also an object now, with open
and close
as properties. I've changed my config to that, and it's now working.
I'm really excited to see you've taken on this project, it definitely had some issues but was so great to use!
I switched my config from hyperterm-overlay to hyper-overlay and left my prior settings in place but I'm getting the following error any time I open Hyper, reload the window or bring up the overlay with the keyboard shortcut.
Hyperterm-overlay does not throw an error.
Unless I'm missing something it looks like the config hasn't changed except possibly the hotkey set to two different options. I updated my configuration with open/close but am still getting the error.
It does not appear to get far enough to move and resize the window.
Here are my settings:
This happens both in Hyper 2.0.0 (Stable) and Canary; with and without any other plugins. macOS 10.13.5 (17F77). Here's my full config
If it would be helpful for troubleshooting I'm happy to jump on your Discord.