UI: Updated Auto Keying Button Icon #105574

Merged
Harley Acheson merged 1 commits from Harley/blender:RecordRed into main 2024-01-08 23:17:05 +01:00
Member

Changing the icons used for auto keying to be more noticeable.


This patch replaces the icon used by the auto-keying button with a pair for off and on. The "on" version being a bit larger than we have now, and the "off" state uses an unfilled circle because that indicates a better "off" and has greater luminance change.

RecordButton6.gif

Changing the icons used for auto keying to be more noticeable. --- This patch replaces the icon used by the auto-keying button with a pair for off and on. The "on" version being a bit larger than we have now, and the "off" state uses an unfilled circle because that indicates a better "off" and has greater luminance change. ![RecordButton6.gif](/attachments/f17291d6-3413-46d6-96b4-3e9342e7efdf)
Member

I like "red" is record, that is kind of a universal "recording is going on" indicator.
The only change would be to keep the white outline so you don't change size just red fill.

I like "red" is record, that is kind of a universal "recording is going on" indicator. The only change would be to keep the white outline so you don't change size just red fill.
Author
Member

I like "red" is record, that is kind of a universal "recording is going on" indicator.

I do too. But there is a chance that the perception of it could differ by age with older people seeing it as normal because of audio and video recorders, while younger people could just see it as an "error" color.

Some red-green colorblindness could also alter this perception. Usually for them we push the color toward yellow into orange. Which could make sense for all since that is more of a warning color. Or maybe a bright orange-yellow?

The only change would be to keep the white outline so you don't change size just red fill.

The icons are created and stored as monochrome, just shades of white. But we are able to recolor these at will, so showing them as white on the dark theme while black on the light theme. So we can make it any color as long as it is one hue, not multiple at a time.

The change of size though is because I am just using our "radio button off" icon here, which isn't quite the right size. If this is something we like then we'd have to create a new icon in the set specifically for "record off"

> I like "red" is record, that is kind of a universal "recording is going on" indicator. I do too. But there is a **chance** that the perception of it could differ by age with older people seeing it as normal because of audio and video recorders, while younger people could just see it as an "error" color. Some red-green colorblindness could also alter this perception. Usually for them we push the color toward yellow into orange. Which could make sense for all since that is more of a warning color. Or maybe a bright orange-yellow? > The only change would be to keep the white outline so you don't change size just red fill. The icons are created and stored as monochrome, just shades of white. But we are able to recolor these at will, so showing them as white on the dark theme while black on the light theme. So we can make it any color as long as it is one hue, not multiple at a time. The change of size though is because I am just using our "radio button off" icon here, which isn't quite the right size. If this is something we like then we'd have to create a new icon in the set specifically for "record off"

I do too. But there is a chance that the perception of it could differ by age with older people seeing it as normal because of audio and video recorders, while younger people could just see it as an "error" color.

Audacity also has a 'red dot' recording button, and so does Reaper. These are always red, though, and not just when they are enabled.

Some red-green colorblindness could also alter this perception. Usually for them we push the color toward yellow into orange. Which could make sense for all since that is more of a warning color. Or maybe a bright orange-yellow?

I think either of these would work:

  • The current proposal, where the background colour changes, coupled with a lightness increase of that background.
  • An alternative, where the dot is always red, but rather dark/dim when it is disabled, and bright when it is enabled.

In both cases the red colour is not the only thing that conveys the information, so it should be fine for colourblind people.

The icons are created and stored as monochrome, just shades of white. But we are able to recolor these at will, so showing them as white on the dark theme while black on the light theme. So we can make it any color as long as it is one hue, not multiple at a time.

That's a shame.

The change of size though is because I am just using our "radio button off" icon here, which isn't quite the right size. If this is something we like then we'd have to create a new icon in the set specifically for "record off"

👍

> I do too. But there is a **chance** that the perception of it could differ by age with older people seeing it as normal because of audio and video recorders, while younger people could just see it as an "error" color. [Audacity](https://www.audacityteam.org/) also has a 'red dot' recording button, and so does [Reaper](https://www.reaper.fm/). These are always red, though, and not just when they are enabled. > Some red-green colorblindness could also alter this perception. Usually for them we push the color toward yellow into orange. Which could make sense for all since that is more of a warning color. Or maybe a bright orange-yellow? I think either of these would work: - The current proposal, where the background colour changes, coupled with a lightness increase of that background. - An alternative, where the dot is always red, but rather dark/dim when it is disabled, and bright when it is enabled. In both cases the red colour is not the only thing that conveys the information, so it should be fine for colourblind people. > The icons are created and stored as monochrome, just shades of white. But we are able to recolor these at will, so showing them as white on the dark theme while black on the light theme. So we can make it any color as long as it is one hue, not multiple at a time. That's a shame. > The change of size though is because I am just using our "radio button off" icon here, which isn't quite the right size. If this is something we like then we'd have to create a new icon in the set specifically for "record off" :+1:
Sybren A. Stüvel added this to the Animation & Rigging project 2023-03-30 16:18:56 +02:00
Sybren A. Stüvel added the
Module
Animation & Rigging
label 2023-03-30 16:19:00 +02:00
Harley Acheson force-pushed RecordRed from 24514f2d5e to 73aab156d3 2023-03-30 22:11:01 +02:00 Compare
Author
Member

@dr.sybren - That's a shame.

My comment about that was a bit wrong-ish. All our current icons are white so that we can re-color them for display. So we can turn that white source icon into one that is black or blue or green.

But... it works just fine if the source icon image has colors, even multiple colors or gradients, etc. Just doing so would not allow us to recolor it later. Of course that does not matter in this case. So the icons in this PR have both a white ring and a red interior. And it looks just fine in both dark and light themes.

> @dr.sybren - That's a shame. My comment about that was a bit wrong-ish. All our current icons are white so that we can re-color them for display. So we can turn that white source icon into one that is black or blue or green. But... it works just fine if the source icon image has colors, even multiple colors or gradients, etc. Just doing so would not allow us to recolor it later. Of course that does not matter in this case. So the icons in this PR have both a white ring and a red interior. And it looks just fine in both dark and light themes.
Author
Member

@blender-bot package

@blender-bot package
Member

Package build started. Download here when ready.

Package build started. [Download here](https://builder.blender.org/download/patch/PR105574) when ready.
Author
Member

Currently (not in PR yet) have a "disabled" state here that I quite like:

RecordButton3.gif

Currently (not in PR yet) have a "disabled" state here that I quite like: ![RecordButton3.gif](/attachments/ff793c88-09fb-494a-a47d-3fabe286eed7)
Member

My worry with it being that grey/faded it looks like it can't be clicked vs. off, if you have the white circle the same brightness as the play buttons, full white then it doesn't look disabled/unclickable maybe.

Because if you compare the drop down arrow next to it, it is the same light grey and it is disabled... unclickable.

My worry with it being that grey/faded it looks like it can't be clicked vs. off, if you have the white circle the same brightness as the play buttons, full white then it doesn't look disabled/unclickable maybe. Because if you compare the drop down arrow next to it, it is the same light grey and it is disabled... unclickable.
Brad Clark closed this pull request 2023-03-31 05:14:16 +02:00
Member

well.. clicked the wrong comment

well.. clicked the wrong comment
Brad Clark reopened this pull request 2023-03-31 05:14:39 +02:00
Author
Member

it looks like it can't be clicked vs. off...

Good point. I'll keep playing.

> it looks like it can't be clicked vs. off... Good point. I'll keep playing.
Author
Member

RecordButton4.gif

![RecordButton4.gif](/attachments/27c220f7-6def-4b5b-8bd0-27fba3918786)
Harley Acheson force-pushed RecordRed from 77480c4ee3 to 0772dda353 2023-04-01 23:12:45 +02:00 Compare
Author
Member

Some users will have a tough time with the red indication and blue background. So I updated the the icons to use only a single color, but you can set the color in the theme. That way anyone who hates the red can set it to white or yellow or anything else. I updated the original comment and the images there.

Some users will have a tough time with the red indication and blue background. So I updated the the icons to use only a single color, but you can set the color in the theme. That way anyone who hates the red can set it to white or yellow or anything else. I updated the original comment and the images there.
Member

My worry with it being that grey/faded it looks like it can't be clicked vs. off, if you have the white circle the same brightness as the play buttons, full white then it doesn't look disabled/unclickable maybe.

Because if you compare the drop down arrow next to it, it is the same light grey and it is disabled... unclickable.

we coukd maybe consider it as the other play forward, play backward, pause buttons.
they don't have a pressed blue state.
in that case we can use the white circle for disabled autokey, and white circle and red fill for auto key on.

edit:
i was wrong, they have pressed blue state. What i was looking at is the play button which changes from play to pause but doesn't stay in pressed blue state.

> My worry with it being that grey/faded it looks like it can't be clicked vs. off, if you have the white circle the same brightness as the play buttons, full white then it doesn't look disabled/unclickable maybe. > > Because if you compare the drop down arrow next to it, it is the same light grey and it is disabled... unclickable. > we coukd maybe consider it as the other play forward, play backward, pause buttons. ~~they don't have a pressed blue state.~~ in that case we can use the white circle for disabled autokey, and white circle and red fill for auto key on. edit: i was wrong, they have pressed blue state. What i was looking at is the play button which changes from play to pause but doesn't stay in pressed blue state.
Author
Member

I updated the animated gif capture in the first comment. Now using icons that exactly match the design language of other icons that have an on and off state:

image

I updated the animated gif capture in the first comment. Now using icons that exactly match the design language of other icons that have an on and off state: ![image](/attachments/4031315d-3155-4532-8ad9-e93680d0f459)

I like the button in your last comment 👍

I like the button in [your last comment]( https://projects.blender.org/blender/blender/pulls/105574#issuecomment-911723) :+1:
Member

I guess this wasn't finalized in any of the meetings, what is left to do?

I guess this wasn't finalized in any of the meetings, what is left to do?
Author
Member

@BClark - I guess this wasn't finalized in any of the meetings, what is left to do?

I think the last version is pretty nice (in the first comment), and allows users or theme designers to change the color if needed.

The next step should really be your module deciding if this is wanted or not. If yes I can then see what UI module thinks.

> @BClark - I guess this wasn't finalized in any of the meetings, what is left to do? I _think_ the last version is pretty nice (in the first comment), and allows users or theme designers to change the color if needed. The next step should really be your module deciding if this is wanted or not. If yes I can then see what UI module thinks.
Member

Yes having has this just mess me up once again when working, I really do want to see with a quick glance at the UI that I am in "record" mode with auto key on.

Yes having has this just mess me up once again when working, I really do want to see with a quick glance at the UI that I am in "record" mode with auto key on.
Member

I'm in favor of this.

Also, the auto-key button was already red pre-2.80:

blender_2.79_autokey_button.png

So in some ways this is just going back to how it was originally supposed to be. (Although it was always red, not just when toggled on.)

I suspect the reason it was changed in 2.80 was to make the UI stylistically consistent. But I think this is a clear case where clarity trumps style. Moreover, this patch still IMO keeps things stylistically consistent while in some ways being even more clear than pre-2.80, by making the button only colored when auto-key is turned on.

I'm in favor of this. Also, the auto-key button was already red pre-2.80: ![blender_2.79_autokey_button.png](/attachments/49c8f724-59a0-4a45-867b-7bd435e31f27) So in some ways this is just going back to how it was originally supposed to be. (Although it was *always* red, not just when toggled on.) I suspect the reason it was changed in 2.80 was to make the UI stylistically consistent. But I think this is a clear case where clarity trumps style. Moreover, this patch still IMO keeps things stylistically consistent while in some ways being even *more* clear than pre-2.80, by making the button only colored when auto-key is turned on.

This was discussed in this week's Animation & Rigging module meeting. The outcomes:

  • OK for the button design. @nathanvegdahl is happy with the blue background, which works well with his red-green colour blindness.
  • Uncertainty about the issue of the notification in the 3D Viewport overlapping with the navigation widget. This PR description has a screenshot of this issue, but no info about whether this is actually addressed as well in this PR. Now that I read through things again, I realise that this is actually handled in #105565. Might be worth it to link that from the PR description.
This was discussed in [this week's Animation & Rigging module meeting](https://devtalk.blender.org/t/2023-07-18-animation-rigging-module-meeting/30352). The outcomes: - OK for the button design. @nathanvegdahl is happy with the blue background, which works well with his red-green colour blindness. - Uncertainty about the issue of the notification in the 3D Viewport overlapping with the navigation widget. This PR description has a screenshot of this issue, but no info about whether this is actually addressed as well in this PR. Now that I read through things again, I realise that this is actually handled in #105565. Might be worth it to link that from the PR description.
Harley Acheson force-pushed RecordRed from 7b115721c5 to a4538f308b 2023-07-21 21:09:04 +02:00 Compare
Harley Acheson changed title from WIP: Evaluate Red Auto Keying Button to UI: Red Auto Keying Button 2023-07-21 21:09:59 +02:00
Harley Acheson requested review from Pablo Vazquez 2023-07-21 21:14:40 +02:00
Author
Member

@pablovazquez - This change is wanted by the Animation and Rigging module. Can you see any problems with doing so? Pre-2.8 did have a red "record" button and it was part of the plan post 2.8 to do so as well but never got done.

@pablovazquez - This change is wanted by the Animation and Rigging module. Can you see any problems with doing so? Pre-2.8 did have a red "record" button and it was part of the plan post 2.8 to do so as well but never got done.
First-time contributor

The red color could be too strong and distracting, at least the shade of it in the example that was shown
The current implementation with blue color is simple and elegant for my taste.

The red color could be too strong and distracting, at least the shade of it in the example that was shown The current implementation with blue color is simple and elegant for my taste.
Member

@Rockbard it should be strong and very visible as it is both a warning that it is on and a universal "recording" color that lets you know you are going to set keys.

@Rockbard it should be strong and very visible as it is both a warning that it is on and a universal "recording" color that lets you know you are going to set keys.
First-time contributor

Just an idea, how about we bring the classic camera recording sign which is blinking red dot when turned on? I think it will be more obvious to identify than Auto-keyframe status message.

Just an idea, how about we bring the classic camera recording sign which is blinking red dot when turned on? I think it will be more obvious to identify than Auto-keyframe status message.
Member

@Harley could you please update this patch? The first gif shows the icon in one way and in the second image looks larger. Not sure which one is the latest. Thanks!

@Harley could you please update this patch? The first gif shows the icon in one way and in the second image looks larger. Not sure which one is the latest. Thanks!
Harley Acheson force-pushed RecordRed from a4538f308b to 90badb06f3 2023-11-15 22:02:15 +01:00 Compare
Author
Member

@pablovazquez - could you please update this patch? The first gif shows the icon in one way and in the second image looks larger. Not sure which one is the latest. Thanks!

No problem. I updated the patch and replaced both images in the first comment.

> @pablovazquez - could you please update this patch? The first gif shows the icon in one way and in the second image looks larger. Not sure which one is the latest. Thanks! No problem. I updated the patch and replaced both images in the first comment.

Just an idea, how about we bring the classic camera recording sign which is blinking red dot when turned on? I think it will be more obvious to identify than Auto-keyframe status message.

I think that would be very obvious indeed, but also rather annoying to any animator who has auto-keying turned on permanently.

> Just an idea, how about we bring the classic camera recording sign which is blinking red dot when turned on? I think it will be more obvious to identify than Auto-keyframe status message. I think that would be very obvious indeed, but also rather annoying to any animator who has auto-keying turned on permanently.
First-time contributor

Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness.

Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness.
Author
Member

@Rockbard - Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness.

I think that looks quite bad to be honest, although I struggle with finding any red that looks good with our default blue highlight color.

The best thing is to run the PR and change the color in Themes / User Interface / Icon Colors to something that you like. Then paste a capture of the resulting record button here. Or - even easier - just take the image below and mock it up with a color you like and paste it back here.

Here are a few variations. The one on left is how it looks by default in the PR. Next is from the Properties categories, then a couple from the 3DView Cursor tool, then one taken from the 3D navigation gizmo.

image

> @Rockbard - Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness. I think that looks quite bad to be honest, although I struggle with finding any red that looks good with our default blue highlight color. The best thing is to run the PR and change the color in Themes / User Interface / Icon Colors to something that you like. Then paste a capture of the resulting record button here. Or - even easier - just take the image below and mock it up with a color you like and paste it back here. Here are a few variations. The one on left is how it looks by default in the PR. Next is from the Properties categories, then a couple from the 3DView Cursor tool, then one taken from the 3D navigation gizmo. ![image](/attachments/1600a23d-c3aa-4620-9d88-231dffedb52c)
5.6 KiB
First-time contributor

@Rockbard - Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness.

I think that looks quite bad to be honest, although I struggle with finding any red that looks good with our default blue highlight color.

The best thing is to run the PR and change the color in Themes / User Interface / Icon Colors to something that you like. Then paste a capture of the resulting record button here. Or - even easier - just take the image below and mock it up with a color you like and paste it back here.

Here are a few variations. The one on left is how it looks by default in the PR. Next is from the Properties categories, then a couple from the 3DView Cursor tool, then one taken from the 3D navigation gizmo.

image

Yeah, you are right actually, thats because the brightness value is quite similar to the bg color.

The color should stand out either by value or by saturation.

But still not to stand out too much from the overall scheme I think.

> > @Rockbard - Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness. > > I think that looks quite bad to be honest, although I struggle with finding any red that looks good with our default blue highlight color. > > The best thing is to run the PR and change the color in Themes / User Interface / Icon Colors to something that you like. Then paste a capture of the resulting record button here. Or - even easier - just take the image below and mock it up with a color you like and paste it back here. > > Here are a few variations. The one on left is how it looks by default in the PR. Next is from the Properties categories, then a couple from the 3DView Cursor tool, then one taken from the 3D navigation gizmo. > > ![image](/attachments/1600a23d-c3aa-4620-9d88-231dffedb52c) > > Yeah, you are right actually, thats because the brightness value is quite similar to the bg color. The color should stand out either by value or by saturation. But still not to stand out too much from the overall scheme I think.
First-time contributor

@Rockbard - Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness.

I think that looks quite bad to be honest, although I struggle with finding any red that looks good with our default blue highlight color.

The best thing is to run the PR and change the color in Themes / User Interface / Icon Colors to something that you like. Then paste a capture of the resulting record button here. Or - even easier - just take the image below and mock it up with a color you like and paste it back here.

Here are a few variations. The one on left is how it looks by default in the PR. Next is from the Properties categories, then a couple from the 3DView Cursor tool, then one taken from the 3D navigation gizmo.

image

The one on the left (the original) is the only one that doesn't hurt to look at. It's the best option for sure IMO

> > @Rockbard - Well if you going to use red, I suggest taking this shade of red from the sidebar icons to maintain the color scheme cohesiveness. > > I think that looks quite bad to be honest, although I struggle with finding any red that looks good with our default blue highlight color. > > The best thing is to run the PR and change the color in Themes / User Interface / Icon Colors to something that you like. Then paste a capture of the resulting record button here. Or - even easier - just take the image below and mock it up with a color you like and paste it back here. > > Here are a few variations. The one on left is how it looks by default in the PR. Next is from the Properties categories, then a couple from the 3DView Cursor tool, then one taken from the 3D navigation gizmo. > > ![image](/attachments/1600a23d-c3aa-4620-9d88-231dffedb52c) > > The one on the left (the original) is the only one that doesn't hurt to look at. It's the best option for sure IMO
Member

"Hurt to look at" is hard to really say without seeing it in the full interface at the correct size, isolated images are misleading and don't show the context of the change.

A full screen capture of it in the UI for size and color is best.

"Hurt to look at" is hard to really say without seeing it in the full interface at the correct size, isolated images are misleading and don't show the context of the change. A full screen capture of it in the UI for size and color is best.
Author
Member

Here's a capture of the entire interface with defaults and at 1X scale, with that option turned on:

image

Here's a capture of the entire interface with defaults and at 1X scale, with that option turned on: ![image](/attachments/f170c150-26ff-4c8d-ac12-4d514a327957)
251 KiB
Member

I'm not an animator but I get the impression that having the red dot being dark brown is not matching the association of a glowing record signal anyway.

Any of the bright orange-red examples seems to fit Blenders UI the most and still read as a record button. But the bad color blindness accessiblity is unfortunate.

Seems to be a choice between favoring accessibility or color association. Or leaving it as is for the best contrast?

I'm not an animator but I get the impression that having the red dot being dark brown is not matching the association of a glowing record signal anyway. Any of the bright orange-red examples seems to fit Blenders UI the most and still read as a record button. But the bad color blindness accessiblity is unfortunate. Seems to be a choice between favoring accessibility or color association. Or leaving it as is for the best contrast?
First-time contributor

Yep, the struggle is real :) Here is another take.

Yep, the struggle is real :) Here is another take.
Author
Member

Any of the bright orange-red examples seems to fit Blenders UI the most and still read as a record button. But the bad color blindness accessibility is unfortunate.

For vision accessibility for colorblindness what we are trying to avoid is requiring a differentiation of red and green, or between other colors for less common variations. The colorblind see colors, but have difficulty differentiating between some, for example red and green might look the same.

So the colorblind will see this as an off to on change, but they might see the red color differently than us. And here we have the added change in that the off state is an unfilled circle, where today the internal dot is the same color and brightness whether on or off. So this should be an improvement even for the rare monochromatism.

> Any of the bright orange-red examples seems to fit Blenders UI the most and still read as a record button. But the bad color blindness accessibility is unfortunate. For vision accessibility for colorblindness what we are trying to avoid is requiring a differentiation of red and green, or between other colors for less common variations. The colorblind see colors, but have difficulty differentiating between some, for example red and green might look the same. So the colorblind will see this as an off to on change, but they might see the red color differently than us. And here we have the added change in that the off state is an unfilled circle, where today the internal dot is the same color and brightness whether on or off. So this should be an improvement even for the rare monochromatism.
Author
Member

And if we continue to find all shades of red ugly with our blue, an option is to do only the icon change. Therefore have an on/off cycle like this, which is still some improvement.

RecordWhite.gif

And if we continue to find all shades of red ugly with our blue, an option is to do only the icon change. Therefore have an on/off cycle like this, which is still some improvement. ![RecordWhite.gif](/attachments/1be65305-8ca1-43b6-9495-13d7853ce99a)
Member

For vision accessibility for colorblindness what we are trying to avoid is requiring a differentiation of red and green, or between other colors for less common variations.

Color blind person here! That's certainly one of the considerations. Another consideration is luminance perception. For example, reds appear much darker to me than to people with normal color vision. As an example:

image

That red is so dark to me that at-a-glance it is difficult to distinguish from the dark grays in the rest of Blender's UI. The surrounding blue notwithstanding, it would look disabled to me.

Digression: it's also not quite right to say that I can't distinguish reds and greens, despite my color blindness being the red/green type. For example, with the except of the first on the left, in this image they're all more-or-less obviously shades of red to me:

image

"Red/green color blindness" is a misnomer, because color vision and color blindness isn't as simple as that. I actually have a harder time distinguishing equal blue-green mixes from gray, for example.

Anyway, digressions aside...

I'm not actually too worried here. Hopefully this won't offend other colorblind people, but as long as there is some clear visual distinction, I'm used to making do. I would prefer avoiding the (to me) very dark red option, but even then I can make do with the blue background as a distinguishing factor.

> For vision accessibility for colorblindness what we are trying to avoid is requiring a differentiation of red and green, or between other colors for less common variations. Color blind person here! That's certainly one of the considerations. Another consideration is luminance perception. For example, reds appear much darker to me than to people with normal color vision. As an example: > ![image](/attachments/f170c150-26ff-4c8d-ac12-4d514a327957) That red is so dark to me that at-a-glance it is difficult to distinguish from the dark grays in the rest of Blender's UI. The surrounding blue notwithstanding, it would look disabled to me. Digression: it's also not quite right to say that I can't distinguish reds and greens, despite my color blindness being the red/green type. For example, with the except of the first on the left, in this image they're all more-or-less obviously shades of red to me: > ![image](/attachments/1600a23d-c3aa-4620-9d88-231dffedb52c) "Red/green color blindness" is a misnomer, because color vision and color blindness isn't as simple as that. I actually have a harder time distinguishing equal blue-green mixes from gray, for example. Anyway, digressions aside... I'm not actually *too* worried here. Hopefully this won't offend other colorblind people, but as long as there is *some* clear visual distinction, I'm used to making do. I would prefer avoiding the (to me) very dark red option, but even then I can make do with the blue background as a distinguishing factor.
Member

I still think that with the current ui theme the red circle is not fitting in the general interface design. I personally find it really disturbing expecially in combination with the blue highlight background.

My suggestion is once again to make it work with general contrast. Exclude hue and saturation form the context and watch the button only in luminance difference. In this situation rgb 1.0.0 equals 0.0.1 and 0.1.0.

This also makes clear why imo the current interfaces always work since white is always at higher values than any other rgb option as long as you choose at maximum 2 over 3 channels.

Anyway seems to me that this change, even if largely debated, still stays controversial in its application. So until something very new pops up on the table, i feel like we are turning in rounds here.

This seems to me the best option amongst the other:
#105574 (comment)

image

I still think that with the current ui theme the red circle is not fitting in the general interface design. I personally find it really disturbing expecially in combination with the blue highlight background. My suggestion is once again to make it work with general contrast. Exclude hue and saturation form the context and watch the button only in luminance difference. In this situation rgb 1.0.0 equals 0.0.1 and 0.1.0. This also makes clear why imo the current interfaces always work since white is always at higher values than any other rgb option as long as you choose at maximum 2 over 3 channels. Anyway seems to me that this change, even if largely debated, still stays controversial in its application. So until something very new pops up on the table, i feel like we are turning in rounds here. This seems to me the best option amongst the other: https://projects.blender.org/blender/blender/pulls/105574#issuecomment-1095335 ![image](/attachments/e50d8469-9276-487c-97a9-052704da1bce)
Author
Member

@nathanvegdahl - ...luminance perception...have a harder time distinguishing equal blue/green mixes from gray, for example

Yes, I think the best (or the only good) part of this PR is the change of the off-state icon to an unfilled circle. And that the on-state filled circle is a larger than now. Currently there just is no change in the icon between states.

@icappiello - This seems to me the best option amongst the other

That is accepting this PR except with the change of the default color of the icon to white. That way by default it would look like that but users (and theme designers) could choose otherwise.

> @nathanvegdahl - ...luminance perception...have a harder time distinguishing equal blue/green mixes from gray, for example Yes, I think the best (or the only good) part of this PR is the change of the off-state icon to an _unfilled_ circle. And that the on-state filled circle is a larger than now. Currently there just is no change in the icon between states. > @icappiello - This seems to me the best option amongst the other That is accepting this PR except with the change of the default color of the icon to white. That way by default it would look like that but users (and theme designers) could choose otherwise.
First-time contributor

just a coincidence but I was trying to have something like that when autokey is on :
image

A small button color change is not enough for me at least and autokey , or the lack of visual clue make me lose so much time per projects ... just my 2 cents. It can be ignored :D

just a coincidence but I was trying to have something like that when autokey is on : ![image](/attachments/ce3adaf8-560e-4cb4-ad32-95955e201e03) A small button color change is not enough for me at least and autokey , or the lack of visual clue make me lose so much time per projects ... just my 2 cents. It can be ignored :D
5.1 MiB
Member

@Harley

Yes, I think the best (or the only good) part of this PR is the change of the off-state icon to an unfilled circle. And that the on-state filled circle is a larger than now. Currently there just is no change in the icon between states.

In that case, I propose that we land this with just those changes, since they're an obvious and (I think?) uncontroversial improvement. And then deciding on whether we also want a color change, and what that color should be, can be discussed separately. That way we at least get the obvious improvement now, rather than risking never getting any improvement because color is hard.

@Harley > Yes, I think the best (or the only good) part of this PR is the change of the off-state icon to an _unfilled_ circle. And that the on-state filled circle is a larger than now. Currently there just is no change in the icon between states. In that case, I propose that we land this with just those changes, since they're an obvious and (I think?) uncontroversial improvement. And then deciding on whether we also want a color change, and what that color should be, can be discussed separately. That way we at least get the obvious improvement now, rather than risking never getting any improvement because color is hard.
Harley Acheson force-pushed RecordRed from c58f7744ce to 9bc54cf0b5 2024-01-08 23:09:41 +01:00 Compare
Harley Acheson changed title from UI: Red Auto Keying Button to UI: Updated Auto Keying Button Icon 2024-01-08 23:10:51 +01:00
Harley Acheson merged commit f42071c793 into main 2024-01-08 23:17:05 +01:00
Harley Acheson deleted branch RecordRed 2024-01-08 23:17:07 +01:00
Sign in to join this conversation.
No reviewers
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No Assignees
12 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#105574
No description provided.