Regression: Cursor arrow shows up after fill operation execution #102650

New Issue

Pratik Borhade · 2022-11-21T07:21:23+01:00

Pratik Borhade commented

2022-11-21 07:21:23 +01:00

System Information
Operating system: Windows-10-10.0.22000-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 3050 Laptop GPU/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.78

Blender Version
Broken: 3.5
Worked: 3.3

Change in cursor behavior came with bf85074148

Short description of error
Cursor arrow appears after performing fill operations. It'll disappear again when cursor is moved over other area
I guess only circular point with red outline is expected when tool is selected

Exact steps for others to reproduce the error

Open uploaded file
Select fill tool (notice cursor)
Attempt to fill closed region
After filling the closed area, default cursor arrow appears
Move cursor over other area (cursor arrow is gone now)
#102650.blend

System Information Operating system: Windows-10-10.0.22000-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 3050 Laptop GPU/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.78 **Blender Version** Broken: 3.5 Worked: 3.3 Change in cursor behavior came with bf85074148 **Short description of error** Cursor arrow appears after performing fill operations. It'll disappear again when cursor is moved over other area I guess only circular point with red outline is expected when tool is selected **Exact steps for others to reproduce the error** - Open uploaded file - Select fill tool (notice cursor) - Attempt to fill closed region - After filling the closed area, default cursor arrow appears - Move cursor over other area (cursor arrow is gone now) [#102650.blend](https://archive.blender.org/developer/F13954465/T102650.blend)

Pratik Borhade commented

2022-11-21 07:21:23 +01:00

Added subscriber: @PratikPB2123

Omar Emara commented

2022-11-21 08:41:32 +01:00

Added subscriber: @OmarEmaraDev

Omar Emara commented

2022-11-21 08:41:32 +01:00

Can't reproduce on Linux X11 myself.

Philipp Oeser commented

2022-11-21 09:32:49 +01:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2022-11-21 09:32:49 +01:00

Changed status from 'Needs Triage' to: 'Confirmed'

Philipp Oeser commented

2022-11-21 09:32:49 +01:00

Can confirm, will check

Philipp Oeser commented

2022-11-21 09:46:07 +01:00

This seems to be a two-step regression:

fill tool now requires two clicks for the fill
cursor changes to default (as reported here)

This seems to be a two-step regression: - fill tool now requires **two clicks** for the fill - cursor changes to default (as reported here)

Antonio Vazquez commented

2022-11-21 09:48:55 +01:00

Added subscriber: @antoniov

Antonio Vazquez commented

2022-11-21 09:48:55 +01:00

In #102650#1448914, @lichtwerk wrote:
This seems to be a two-step regression:

fill tool now requires two clicks for the fill

cursor changes to default (as reported here)

No, you don't need two clicks... the reason is you have enabled the help lines, so the first click enable the help lines and with the wheel mouse you can change the size. The second click is the confirmation.

> In #102650#1448914, @lichtwerk wrote: > This seems to be a two-step regression: > - fill tool now requires **two clicks** for the fill > - cursor changes to default (as reported here) No, you don't need two clicks... the reason is you have enabled the help lines, so the first click enable the help lines and with the wheel mouse you can change the size. The second click is the confirmation.

Antonio Vazquez commented

2022-11-21 09:58:53 +01:00

2022-11-21 09-58-19.mp4

[2022-11-21 09-58-19.mp4](https://archive.blender.org/developer/F13955213/2022-11-21_09-58-19.mp4)

Philipp Oeser commented

2022-11-21 10:04:45 +01:00

In #102650#1448916, @antoniov wrote:
No, you don't need two clicks... the reason is you have enabled the help lines, so the first click enable the help lines and with the wheel mouse you can change the size. The second click is the confirmation.

Ah, we are talking https://docs.blender.org/manual/en/3.5/grease_pencil/modes/draw/tools/fill.html#automatic-gap-closure
(sorry, wasnt aware of this new feature)
It is a bit redundant for already closed shapes like in this report, but will make myself familiar with the gap filling

> In #102650#1448916, @antoniov wrote: > No, you don't need two clicks... the reason is you have enabled the help lines, so the first click enable the help lines and with the wheel mouse you can change the size. The second click is the confirmation. Ah, we are talking https://docs.blender.org/manual/en/3.5/grease_pencil/modes/draw/tools/fill.html#automatic-gap-closure (sorry, wasnt aware of this new feature) It is a bit redundant for already closed shapes like in this report, but will make myself familiar with the gap filling

Philipp Oeser commented

2022-11-21 10:05:16 +01:00

thx for the video!

Philipp Oeser commented

2022-11-21 10:29:19 +01:00

As for the cursor change in behavior: this came with bf85074148

As for the cursor change in behavior: this came with bf85074148

Philipp Oeser commented

2022-11-21 10:36:11 +01:00

@antoniov: seems like the call to WM_cursor_set(CTX_wm_window(C), WM_CURSOR_DEFAULT); is redundant?

@antoniov: seems like the call to `WM_cursor_set(CTX_wm_window(C), WM_CURSOR_DEFAULT);` is redundant?

Antonio Vazquez commented

2022-11-21 16:29:53 +01:00

In #102650#1448945, @lichtwerk wrote:
@antoniov: seems like the call to WM_cursor_set(CTX_wm_window(C), WM_CURSOR_DEFAULT); is redundant?

I don't think so except I miss something. When you do the first click, the cursor changes to a pencil cursor, but if you press MMB to change size of extend lines, the cursor chages while you move... as soon you release MMB the cursor back to pencil cursor.

> In #102650#1448945, @lichtwerk wrote: > @antoniov: seems like the call to `WM_cursor_set(CTX_wm_window(C), WM_CURSOR_DEFAULT);` is redundant? I don't think so except I miss something. When you do the first click, the cursor changes to a pencil cursor, but if you press `MMB` to change size of extend lines, the cursor chages while you move... as soon you release `MMB` the cursor back to pencil cursor.

Pratik Borhade commented

2022-11-22 15:07:42 +01:00

Can we raise the priority since this is a regression? May not be a top priority issue.

Antonio Vazquez commented

2022-11-22 15:21:37 +01:00

Really I don't see the bug. If you enable visual aids you must use two clicks because the first is to start and the second to confirm...during this time the cursor must be the pencil, when you confirm, the cursor is reseted. If you disable visual aids is only one click and the cursor is reset perfectly also.

Really I don't see the bug. If you enable `visual aids` you must use two clicks because the first is to start and the second to confirm...during this time the cursor must be the pencil, when you confirm, the cursor is reseted. If you disable `visual aids` is only one click and the cursor is reset perfectly also.

Philipp Oeser commented

2022-11-22 15:27:54 +01:00

It should probably not be WM_CURSOR_DEFAULT though? (havent checked in depth though)

It should probably not be `WM_CURSOR_DEFAULT` though? (havent checked in depth though)

Pratik Borhade commented

2022-11-22 15:28:50 +01:00

during this time the cursor must be the pencil, when you confirm, the cursor is reseted

@antoniov ok, you mean cursor reset to default "arrow"?
But Later if I move mouse over other area (let's say properties panel), it'll remove the "default cursor arrow". Isn't this inconsistent? :)

> during this time the cursor must be the pencil, when you confirm, the cursor is reseted @antoniov ok, you mean cursor reset to default "arrow"? But Later if I move mouse over other area (let's say properties panel), it'll remove the "default cursor arrow". Isn't this inconsistent? :)

Philipp Oeser self-assigned this 2022-11-23 13:54:42 +01:00

Philipp Oeser commented

2022-11-23 13:54:42 +01:00

Got a fix

blender-admin commented

2022-11-23 14:05:53 +01:00

This issue was referenced by b0810f788c

This issue was referenced by b0810f788c9edb326300320d482502c5f5ea28e2

Philipp Oeser commented

2022-11-27 12:40:50 +01:00

Changed status from 'Confirmed' to: 'Resolved'

Philipp Oeser closed this issue

2022-11-27 12:40:50 +01:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Regression: Cursor arrow shows up after fill operation execution #102650