search

ByzantineFailure / BPM-for-Discord

BPM for Discord's Desktop App. Includes one-click installers, update notifications, and custom script support.
GNU Affero General Public License v3.0
17 stars 8 forks source link

BPM settings don't show in upcoming full screen settings. #72

Closed Jibodeah closed 7 years ago

Jibodeah commented 7 years ago

This is more an upcoming problem rather than a current one. There's an upcoming revamp of the settings screen which is already live in Canary. It looks like this:

image

You'll note the lack of a 'BPM' section in the navbar on the left! This makes the BPM settings unreachable.

...Incidentally adding a shortcut to the BPM settings on the right click menu that appears when you right click the settings cog would make it more consistent with the other sections.

ByzantineFailure commented 7 years ago

Aw fuck. Thanks for the heads up, looks like I have some work to do.

ByzantineFailure commented 7 years ago

@Jibodeah https://github.com/ByzantineFailure/BPM-for-Discord/pull/73

When these changes drop I'll merge and release this PR. Thanks for the heads up.

Jibodeah commented 7 years ago

Good to know! ...and no problem. Guess this is what us Canary users are for anyway.

ByzantineFailure commented 7 years ago

Yep! ❤️

bb010g commented 7 years ago

This is a bigger issue now that it's the default in stable.

ByzantineFailure commented 7 years ago

@bb010g merged in!