Maniphest T96074

Remove redundant linked & library override icons
Confirmed, NormalTO DO
Actions

Assigned To
None
Authored By
Julian Eisel (Severin)
Feb 28 2022, 3:26 PM
Tags
  • User Interface
Subscribers
Alessio Monti di Sopra (a.monti)
Julian Eisel (Severin)
Philipp Oeser (lichtwerk)
Raimund Klink (Raimund58)
Tokens
"Like" token, awarded by mont29.

Description

For some reason we have a whole bunch of redundant icons for linked data and library overrides.

These link icons are redundant (only one of each type should be kept):

  • LINKED
  • DECORATE_LINKED (unused)
  • LINK_BLEND

Same with these library override icons:

  • LIBRARY_DATA_OVERRIDE
  • DECORATE_LIBRARY_OVERRIDE (only used in custom properties UI, could be replaced easily)

Also, the RESTRICT_INSTANCED_OFF icon looks like thicker version of the linked icon. It and its RESTRICT_INSTANCED_ON counterpart are not used.

Related:

  • This looks like the link icons, but indents to indicate locked/unlocked.
    Should just use a lock instead.
  • Here the decorator is supposed to indicate a library override (it's DECORATE_OVERRIDE), but doesn't look like that at all. Can use the regular override icon instead

Related Objects

Mentioned Here
rB777a546c0c78: Outliner Restriction: Naming sanitization and icon

Event Timeline

Julian Eisel (Severin) created this task.Feb 28 2022, 3:26 PM
Julian Eisel (Severin) updated the task description.Feb 28 2022, 3:29 PM
Bastien Montagne (mont29) awarded a token.Feb 28 2022, 3:32 PM
Raimund Klink (Raimund58) added a subscriber: Raimund Klink (Raimund58).Feb 28 2022, 3:39 PM
Julian Eisel (Severin) updated the task description.Feb 28 2022, 3:42 PM
Alessio Monti di Sopra (a.monti) added a subscriber: Alessio Monti di Sopra (a.monti).Feb 28 2022, 4:50 PM

Also, the RESTRICT_INSTANCED_OFF icon looks like thicker version of the linked icon. It and its RESTRICT_INSTANCED_ON counterpart are not used.

Ah, rB777a546c0c7 seems like those were designed for before the "restrict viewport" was a thing, they never even reached 2.80.

Philipp Oeser (lichtwerk) changed the task status from Needs Triage to Confirmed.Mar 7 2022, 2:38 PM
Philipp Oeser (lichtwerk) added a subscriber: Philipp Oeser (lichtwerk).

@Julian Eisel (Severin) : I assume all newly created TODOs from your side are confirmed?

Julian Eisel (Severin) added a comment.Mar 7 2022, 3:02 PM

We currently don't really use the "Confirmed" status for development task organization in the UI module. So for TODO or design tasks it doesn't matter much, at least to me.

Philipp Oeser (lichtwerk) added a comment.Mar 7 2022, 3:04 PM
In T96074#1318160, @Julian Eisel (Severin) wrote:

We currently don't really use the "Confirmed" status for development task organization in the UI module. So for TODO or design tasks it doesn't matter much, at least to me.

OK, if it doesnt matter on your side, please confirm (to get it out of the Triaging queue)