Icon Alpha property also affects the thumbnails of matcaps/studio lighting... #80356

Open
opened 2020-09-02 05:06:16 +02:00 by TheRedWaxPolice · 13 comments

System Information
Operating system: Win 10

Blender Version
Broken: blender-2.91.0-68797982025c-windows64
Worked: ?

Short description of error
The "Icon Alpha" property in the Themes settings also affects the thumbnails of stuff like matcaps/studio lights....

Exact steps for others to reproduce the error
Go to: Edit -> Preferences -> Themes -> User Interface -> Styles; now try to adjust the "Icon Alpha" property while the matcaps panel is open to see the effect...
2020-08-26_03-47-24.gif

**System Information** Operating system: Win 10 **Blender Version** Broken: blender-2.91.0-68797982025c-windows64 Worked: ? **Short description of error** The "Icon Alpha" property in the Themes settings also affects the thumbnails of stuff like matcaps/studio lights.... **Exact steps for others to reproduce the error** Go to: Edit -> Preferences -> Themes -> User Interface -> Styles; now try to adjust the "Icon Alpha" property while the matcaps panel is open to see the effect... ![2020-08-26_03-47-24.gif](https://archive.blender.org/developer/F8831585/2020-08-26_03-47-24.gif)

Added subscriber: @TheRedWaxPolice

Added subscriber: @TheRedWaxPolice
Member

Added subscribers: @HooglyBoogly, @lichtwerk

Added subscribers: @HooglyBoogly, @lichtwerk
Member

Changed status from 'Needs Triage' to: 'Needs User Info'

Changed status from 'Needs Triage' to: 'Needs User Info'
Member

Well, these are icons, all preview icons are affected [material, ...].

Personally, I think the Icon Alpha property is a rare thing to tweak anyways (is this done in any theme we are shipping?), but to stay on topic: Do you mean these should be treated as thumbnails in the Filebrowser?
Pretty sure this would not be considered a bug, I am also not sure which of the preview icons one would want to exclude from being considered for Icon Alpha. (Like I said, if we do it for matcaps, we should do it for material preview icons as well).

@HooglyBoogly : got an opinion here? Tempted to close as feature request to be honest...

Well, these **are** icons, all preview icons are affected [material, ...]. Personally, I think the `Icon Alpha` property is a rare thing to tweak anyways (is this done in any theme we are shipping?), but to stay on topic: Do you mean these should be treated as thumbnails in the Filebrowser? Pretty sure this would not be considered a bug, I am also not sure which of the preview icons one would want to exclude from being considered for `Icon Alpha`. (Like I said, if we do it for matcaps, we should do it for material preview icons as well). @HooglyBoogly : got an opinion here? Tempted to close as feature request to be honest...
Contributor

Added subscriber: @Rawalanche

Added subscriber: @Rawalanche
Contributor

In #80356#1007289, @lichtwerk wrote:
Well, these are icons, all preview icons are affected [material, ...].

I don't think that's right. Those are (or technically should be) previews/thumbnails. Icons are meant to be iconic representation of certain thing, so accuracy is not so important there. Color correcting icons is fine. Previews or thumbnails on the other hand have purpose of giving you as close preview of the expected result as possible. Those spheres are there to give you as accurate representation of what your scene shading will look like after applying given matcap before you even click it, so that you can decide which one to use without several trial and error cycles.

I'd throw it into same category as for example file browser thumbnails. When you are browsing your texture library you want to see exactly how each texture looks without having to open it first. If you now have some sort of non opaque alpha, and you are previewing thumbnails on a very dark background, suddenly all textures seem darker than they should be, and you are not certain to what degree that's a product of the texture look itself, or the reduced opacity. So the only reasonable choice is to make Alpha 1.0 in such case, but that would then make having that alpha slider completely pointless, since tinting your icons with color of the UI background by reducing alpha would at the same time corrupt accuracy of thumbnails, a category which matcap previews fall into.

So I think on more abstract, UI system level, Blender should certainly differ which UI elements are meant to be icons (tool icons, panel icons, button icons, etc...) and which UI elements are meant to be thumbnails (file browser thumbnails, matcap thumbnails, HDRI thumbnails, texture thumbnails, etc...), and should exclude latter category from any degree of UI color correction.

> In #80356#1007289, @lichtwerk wrote: > Well, these **are** icons, all preview icons are affected [material, ...]. I don't think that's right. Those are (or technically should be) previews/thumbnails. Icons are meant to be iconic representation of certain thing, so accuracy is not so important there. Color correcting icons is fine. Previews or thumbnails on the other hand have purpose of giving you as close preview of the expected result as possible. Those spheres are there to give you as accurate representation of what your scene shading will look like after applying given matcap before you even click it, so that you can decide which one to use without several trial and error cycles. I'd throw it into same category as for example file browser thumbnails. When you are browsing your texture library you want to see exactly how each texture looks without having to open it first. If you now have some sort of non opaque alpha, and you are previewing thumbnails on a very dark background, suddenly all textures seem darker than they should be, and you are not certain to what degree that's a product of the texture look itself, or the reduced opacity. So the only reasonable choice is to make Alpha 1.0 in such case, but that would then make having that alpha slider completely pointless, since tinting your icons with color of the UI background by reducing alpha would at the same time corrupt accuracy of thumbnails, a category which matcap previews fall into. So I think on more abstract, UI system level, Blender should certainly differ which UI elements are meant to be icons (tool icons, panel icons, button icons, etc...) and which UI elements are meant to be thumbnails (file browser thumbnails, matcap thumbnails, HDRI thumbnails, texture thumbnails, etc...), and should exclude latter category from any degree of UI color correction.
Member

Added subscriber: @JulianEisel

Added subscriber: @JulianEisel
Member

I think we can just disable the alpha factor for previews, it doesn't make sense from what I can tell.
If there are reasons to have it, we could add a separate theme option. But icons and previews are indeed very different things on a user level.

I think we can just disable the alpha factor for previews, it doesn't make sense from what I can tell. If there are reasons to have it, we *could* add a separate theme option. But icons and previews are indeed very different things on a user level.
Member

Changed status from 'Needs User Info' to: 'Confirmed'

Changed status from 'Needs User Info' to: 'Confirmed'
Member

In #80356#1007297, @Rawalanche wrote:
Those are (or technically should be)

That would make the distinction between a bug and a request :)

But I totally agree with the reasoning, that is why I mentioned thumbnails in the filebrowser as well, it doesnt really make sense to act on the previews.
(it is not the only dark corner in blender that doesnt make sense, but still technically is not a bug though...)

In #80356#1007306, @JulianEisel wrote:
I think we can just disable the alpha factor for previews

Reading that as marking it a TODO, feel free to change to anything else if needed.

> In #80356#1007297, @Rawalanche wrote: > Those are (or technically should be) That would make the distinction between a bug and a request :) But I totally agree with the reasoning, that is why I mentioned thumbnails in the filebrowser as well, it doesnt really **make sense** to act on the previews. (it is not the only dark corner in blender that doesnt make sense, but still technically is not a bug though...) > In #80356#1007306, @JulianEisel wrote: > I think we can just disable the alpha factor for previews Reading that as marking it a TODO, feel free to change to anything else if needed.

Added subscriber: @jenkm

Added subscriber: @jenkm

Do we even need this option? This was for color icons ("to reduce contrast"), now the color of icons respects the theme settings.

Do we even need this option? This was for color icons ("to reduce contrast"), now the color of icons respects the theme settings.
Member

Removing it would be fine with me. I also don't see a need and initially thought it was added during 2.8. But seems it's an old option and probably redundant by now.

Removing it would be fine with me. I also don't see a need and initially thought it was added during 2.8. But seems it's an old option and probably redundant by now.
Philipp Oeser removed the
Interest
User Interface
label 2023-02-10 09:24:08 +01:00
Sign in to join this conversation.
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 project
No Assignees
5 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#80356
No description provided.