search

nutti / Screencast-Keys

Blender Add-on: Screencast Keys
GNU General Public License v3.0
1.27k stars 112 forks source link

Choice between Sidebar panel and Viewport Overlay #35

Closed Michaelknubben closed 4 years ago

Michaelknubben commented 4 years ago

Description about the feature
As we all know, the Sidebar can get overloaded with hard to read tabs, and Blender now has an easily accessible feature for Viewport Overlays. I think this fits there, and would love to have the option to display it in the Viewport Overlay dropdown. Personally I don't see a reason for all of the features from the current panel to be displayed there, I would set the colour, font size etc. in the addon options.

Are you willing to contribute about this feature. (Yes/No)
Yes: but I'm not a developer. I would be open to making mockups and testing the feature though.

Screenshots [Optional]
It is good to understand the features if you attach the screenshotsfile. The panel as it is now: image Where I'd want to see it instead: image

Additional comments [Optional]

I'm a teacher since very recently, so I'd like to thank you for keeping this addon up to date!

nutti commented 4 years ago

@Michaelknubben

Thanks for your request. It seems interesting.

If possible, I would like to know the add-ons which realizes this improvement.

nutti commented 4 years ago

@Michaelknubben

OK, now I got how to add UI to viewport overlay.

github_20201006

You can also this feature from below link. https://github.com/nutti/Screencast-Keys/archive/spike_35.zip

Do you have any good idea for improving this UI?

Michaelknubben commented 4 years ago

Great, glad you figured it out! Honestly? I'd just get rid of all features except for the toggle (so turning it on or off), and have all other features available in the addon preferences. Possibly you could also list the colour after it, so it'd be like Wireframe's Toggle and Slider. Another option is to have a cogwheel icon behind it that opens the addon preferences directly? I just really want to advise against listing all of this stuff in that menu.

Michaelknubben commented 4 years ago

Oh, and if you do list anything else, I think it'd be good if you'd make it invisible until the toggle is on. An example of this behaviour is in Mesh Check (BGL Edition), a free addon.

nutti commented 4 years ago

@Michaelknubben

Thanks for your feedback. I almost implemented this feature now.

Finally, I introduced new overlay menu for Screencast Keys.

github_20201018

I think it is good for separating from exists overlay menus. This feature is now available in testing branch. Could you try this?

https://github.com/nutti/Screencast-Keys/archive/spike_35.zip

Michaelknubben commented 4 years ago

Like I said before, I really do think the settings belong in the addon preferences, and I see no reason to create a new menu. If every addon does that, we're right back to 2.7x's problems with the T panel! You're ofcourse free to disagree and steer your own course, but I'll probably use Pie Menu Editor to create my own button somewhere and take it out of the sidepanel.

nutti commented 4 years ago

Your advise was introducing a overlay menu on header for Screencast keys. Does my understanding mistake?

If you have any mock up UI, it helps me a lot.

nutti commented 4 years ago

@Michaelknubben

Perhaps, you think the UI as follows.

github_20201019

This new UI does not introduce any new Menu. You can switch on/off previous Panel on Sidebar and new UI.

Michaelknubben commented 4 years ago

Exactly, that's what I meant! Sorry for the confusion, I'll make sure to include a mockup next time.

nutti commented 4 years ago

This feature is now merged to master branch.

@Michaelknubben

Thanks for your help!

Michaelknubben commented 3 years ago

Hey Nutti, I tested this (in 2.91) and found an issue: afbeelding This is with 'Overlay' enabled in the settings, which only adds a blank space at the bottom, but I see no button!

I've tried restarting, but no change.

Is there something I can do to help you fix this?

nutti commented 3 years ago

@Michaelknubben

Sorry, I haven't noticed your issue. I can find this issue from the another issue. The bug has already fixed, and you can try the unstable version which merges the fixed patch.

If you have any further issue about this feature, please create new issue. Thanks.