nutti / Screencast-Keys

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

Feature Request: Options to re-size Shadow/Outline thickness #19

Closed 0Curtain0 closed 4 years ago

0Curtain0 commented 4 years ago

Description about the feature

Are you willing to contribute about this feature. (Yes/No)
I can provide feedback.

Screenshots [Optional]
It is good to understand the features if you attach the screenshotsfile.

Additional comments [Optional]

nutti commented 4 years ago

@0Curtain0

Thanks for the requests. Adding this feature looks good to me.

nutti commented 4 years ago

@0Curtain0

I found that shadow size is not changeable in current Blender API. But, I think adding shadow seems good to me.

github_20200501

github_20200501-2

Another alternative option you can add, if possible, is a backcolor/highlight box option for all key strokes.

Also, I'll show you a backcolor/highlight box version.

github_20200501-3

What do you think above looks? If it is good for you, I will apply this change.

0Curtain0 commented 4 years ago

@nutti Both very good, and the backcolor/highlight is very nice and visible. The backcolor/highlight version will have changeable alpha channel too?


Also, what do you think of giving x7 a separate color option? Or separate color option for Select ('view3d.select') ?

nutti commented 4 years ago

@0Curtain0

The backcolor/highlight version will have changeable alpha channel too?

Sorry, this option is not good for my add-on due to the add-on's specification. When the text overlap between "Window" region and "Toolbar" region, we could not handle these situation if we have alpha channel. Instead, I will provide disable option to hide the background.

Also, what do you think of giving x7 a separate color option? Or separate color option for Select ('view3d.select') ?

It seems good option. But, I'm afraid that too many option confuses the user. What do you think about it?

0Curtain0 commented 4 years ago

@nutti

Sorry, this option is not good for my add-on due to the add-on's specification. When the text overlap between "Window" region and "Toolbar" region, we could not handle these situation if we have alpha channel. Instead, I will provide disable option to hide the background.

That's fine, and no worries.

It seems good option. But, I'm afraid that too many option confuses the user. What do you think about it?

It's not urgent. Maybe it can be made a separate issue and then come back to it in a few months or whenever the next Blender python API update happens. That should be enough time to make things more intuitive.

When I get time, I will see if I can contribute any ideas to improve the UI of the Screencast-Keys side panel.

I'm currently learning python, but my eyes aren't experienced enough to look through and comprehend Blender python code just yet.

nutti commented 4 years ago

@0Curtain0

When I get time, I will see if I can contribute any ideas to improve the UI of the Screencast-Keys side panel.

Great news! Thanks. I committed current version which enables to add shadow/background of text.

I think your contribution should be different issues, so closes this issue for now. This project is contribution welcomed. We are waiting your contribution in future!