Fix T89515: Clicking on Favorites in File Browser will rename them
Likely uncovered by6c97c7f767
, the actual mistake would be from6942dd9f49
. The hacks to display text buttons for renaming in UI-Lists used the emboss of the text button for handling logic. It relied on the emboss `NONE` but we also introduced `NONE_OR_STATUS` with6942dd9f49
. Both values need to be treated equally for the logic of this hack to work. The change in `interface_layout.c` is actually not needed for this exact issue, but it's the correct thing to do. There may actually be more cases where `NONE` and `NONE_OR_STATUS` need to be treated equally. Something to be checked still.
This commit is contained in:
parent
90e4c8436d
commit
bf65f94f45
Notes:
blender-bot
2023-02-14 06:17:17 +01:00
Referenced by issue #88449: Blender LTS: Maintenance Task 2.93 Referenced by issue #88449, Blender LTS: Maintenance Task 2.93 Referenced by issue #89515, Clicking on Favourites will rename them
|
@ -4510,7 +4510,7 @@ static int ui_do_but_TEX(
|
|||
if (ELEM(event->type, EVT_PADENTER, EVT_RETKEY) && (!UI_but_is_utf8(but))) {
|
||||
/* pass - allow filesel, enter to execute */
|
||||
}
|
||||
else if (but->emboss == UI_EMBOSS_NONE && !event->ctrl) {
|
||||
else if (ELEM(but->emboss, UI_EMBOSS_NONE, UI_EMBOSS_NONE_OR_STATUS) && !event->ctrl) {
|
||||
/* pass */
|
||||
}
|
||||
else {
|
||||
|
|
|
@ -2355,7 +2355,7 @@ void uiItemFullR(uiLayout *layout,
|
|||
|
||||
/* Mark non-embossed textfields inside a listbox. */
|
||||
if (but && (block->flag & UI_BLOCK_LIST_ITEM) && (but->type == UI_BTYPE_TEXT) &&
|
||||
(but->emboss & UI_EMBOSS_NONE)) {
|
||||
ELEM(but->emboss, UI_EMBOSS_NONE, UI_EMBOSS_NONE_OR_STATUS)) {
|
||||
UI_but_flag_enable(but, UI_BUT_LIST_ITEM);
|
||||
}
|
||||
|
||||
|
|
|
@ -90,7 +90,8 @@ bool ui_but_is_interactive(const uiBut *but, const bool labeledit)
|
|||
if (but->flag & UI_SCROLLED) {
|
||||
return false;
|
||||
}
|
||||
if ((but->type == UI_BTYPE_TEXT) && (but->emboss == UI_EMBOSS_NONE) && !labeledit) {
|
||||
if ((but->type == UI_BTYPE_TEXT) &&
|
||||
(ELEM(but->emboss, UI_EMBOSS_NONE, UI_EMBOSS_NONE_OR_STATUS)) && !labeledit) {
|
||||
return false;
|
||||
}
|
||||
if ((but->type == UI_BTYPE_LISTROW) && labeledit) {
|
||||
|
|
Loading…
Reference in New Issue