henrikruscon / hyper-tabs-enhanced

Enhanced Tabs Plugin for Hyper
MIT License
192 stars 20 forks source link

Support opacity for background tabs #7

Open controversial opened 7 years ago

controversial commented 7 years ago

I'm using a hyper config where my window is transparent and vibrant:

screen shot 2017-02-20 at 12 29 04 pm

hyper-tabs-enhanced sets the background tab to opaque black, instead of a darker but still transparent look that I would expect. Perhaps for transparent black tabs, it should just increase background tab opacity?

henrikruscon commented 7 years ago

Can I get your config so I can try your setup? ✌️

controversial commented 7 years ago

sure :)

henrikruscon commented 7 years ago
skarmavbild 2017-02-20 kl 18 48 00 skarmavbild 2017-02-20 kl 18 49 34

It is transparent.

controversial commented 7 years ago

Ah, so it is. I think it's darker than I'd like, though, could there be a config option for "degree of darkening?" Maybe I'll work on a PR?

henrikruscon commented 7 years ago

If there's a way to detect vibrancy and then based on that darken the color less, that would be ideal.

controversial commented 7 years ago

https://github.com/matheuss/hyper-simple-vibrancy/blob/master/index.js#L1 sets the vibrancy inside the onwindow method, I'm not sure whether it's possible to access browserWindow outside of this.

henrikruscon commented 7 years ago

It is but I can't see that it holds the vibrancy variable. I'm gonna look into it more

controversial commented 7 years ago

If there's a setVibrancy method I'd assume there was a corresponding getVibrancy method

henrikruscon commented 7 years ago

Yeah, sadly there isn't. https://github.com/electron/electron/blob/master/docs/api/browser-window.md#winsetvibrancytype-macos

controversial commented 7 years ago

Strange. Maybe a config option is the best way to go, then

henrikruscon commented 7 years ago

I've got a lot of work so I won't be able to look into this for like 3 days ✌️

controversial commented 7 years ago

That's fine, there's no rush On Tue, Feb 21, 2017 at 8:37 AM Henrik notifications@github.com wrote:

I've got a lot of work so I won't be able to look into this for like 3 days ✌️

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/henrikdahl/hyper-tabs-enhanced/issues/7#issuecomment-281346261, or mute the thread https://github.com/notifications/unsubscribe-auth/AJ5Yr_kNiwSgDc7KXL-322ds9pUWfyq2ks5reuirgaJpZM4MGdLq .

outaTiME commented 7 years ago

Same when using hyper-transparent-dynamic

jbergstroem commented 7 years ago

Can confirm I get this as well. As much as I enjoy the unique features of this plugin I can't help but feel its too opinionated. It changes colors of my background, my inactive tabs and also interferes with icons. I guess its a combo plugin that sits well with the author and that's cool too.