UI Experiment: Operator Preferences on Status Bar #126477

Harley Acheson · 2024-08-19T03:45:05+02:00

Harley Acheson commented

2024-08-19 03:45:05 +02:00

Less a proposal and more a thing to talk about. Once applied while
navigating in viewport this shows options to change related user
preferences. The idea of making the preferences available where used.
No need for users to know what things can be changed.

While orbiting:

While zooming:

While panning:

Less a proposal and more a thing to talk about. Once applied while navigating in viewport this shows options to change related user preferences. The idea of making the preferences available where used. No need for users to know what things can be changed. --- While orbiting: ![image](/attachments/129a59fc-5798-4878-ab62-432f05210447) While zooming: ![image](/attachments/020c9091-0e85-4ec4-994b-bd5d25f2df36) While panning: ![image](/attachments/3cbe47eb-f144-42dd-99a2-e0b7367f857a)

image.png

12 KiB

image.png

9.2 KiB

image.png

6.6 KiB

👍 5

Harley Acheson added 1 commit 2024-08-19 03:45:16 +02:00

UI Experiment: Operator Preferences on Status Bar 332281dc48

Less a proposal and more a thing to talk about. Once applied, while
navigating in viewport (specifically rotation) this shows options to
change related user preferences.

Harley Acheson added this to the User Interface project 2024-08-19 03:45:51 +02:00

Harley Acheson requested review from Pablo Vazquez 2024-08-19 17:53:48 +02:00

Harley Acheson added 2 commits 2024-08-28 00:22:10 +02:00

Merge branch 'main' of projects.blender.org:blender/blender into OpPrefs 0f69232c2b

Simplified bc98c7e1cd

Pablo Vazquez commented

2024-09-12 19:13:07 +02:00

As agreed on during the latest module meeting. It'd be good to move forward with this since it exposes certain settings that many users seem to expect when coming from other software.

Another popular ones:

Auto Depth (shortcut D), which should show up during orbit, panning, and zoom:
Zoom to Position (shortcut Z), should show up during zoom. Ctrl+MMB only, I'm not sure we can update the statusbar during scroll, can we?

As agreed on during the latest module meeting. It'd be good to move forward with this since it exposes certain settings that many users seem to expect when coming from other software. Another popular ones: * `Auto Depth` (shortcut `D`), which should show up during orbit, panning, and zoom: ![auto depth](/attachments/1785c094-a35e-4e1f-ab72-cb1307c86c5d) * `Zoom to Position` (shortcut `Z`), should show up during zoom. Ctrl+MMB only, I'm not sure we can update the statusbar during scroll, can we?

image.png

43 KiB

Harley Acheson commented

2024-09-13 17:08:47 +02:00

As agreed on during the latest module meeting. It'd be good to move forward with this since it exposes certain settings that many users seem to expect when coming from other software.

Another popular ones:

Auto Depth (shortcut D), which should show up during orbit, panning, and zoom:

Zoom to Position (shortcut Z), should show up during zoom. Ctrl+MMB only, I'm not sure we can update the statusbar during scroll, can we?

One weird complication is that I was unable to differentiate between orbiting, panning, and zooming. It was all treated like one transform operation so I couldn't show the correct options at the correct times. I'll take another look in a bit though.

> As agreed on during the latest module meeting. It'd be good to move forward with this since it exposes certain settings that many users seem to expect when coming from other software. > > Another popular ones: > > * `Auto Depth` (shortcut `D`), which should show up during orbit, panning, and zoom: > ![auto depth](/attachments/1785c094-a35e-4e1f-ab72-cb1307c86c5d) > > * `Zoom to Position` (shortcut `Z`), should show up during zoom. Ctrl+MMB only, I'm not sure we can update the statusbar during scroll, can we? One weird complication is that I was unable to differentiate between orbiting, panning, and zooming. It was all treated like one transform operation so I couldn't show the correct options at the correct times. I'll take another look in a bit though.

Harley Acheson referenced this pull request

2024-10-05 21:43:30 +02:00

Transform: Simplify and specialize event handling #128563

Germano Cavalcante referenced this pull request

2024-10-05 22:05:17 +02:00

Transform: Simplify and specialize event handling #128563

Germano Cavalcante reviewed 2024-10-06 02:48:05 +02:00

Germano Cavalcante left a comment

It seems good to reveal these half-hidden options in the status bar itself!

I was unable to differentiate between orbiting, panning, and zooming

The settings supported by the operator should be found in ViewOpsType::flag.
The way each of these configurations is initiated is in ViewOpsData::init_navigation(...)

It seems good to reveal these half-hidden options in the status bar itself! > I was unable to differentiate between orbiting, panning, and zooming The settings supported by the operator should be found in `ViewOpsType::flag`. The way each of these configurations is initiated is in `ViewOpsData::init_navigation(...)`

source/blender/editors/space_view3d/view3d_navigate.cc Outdated

						
				@ -529,0 +540,4 @@

				      if (event->type == EVT_PKEY) {

				        vod->viewops_flag ^= VIEWOPS_FLAG_PERSP_ENSURE;

				        U.uiflag ^= USER_AUTOPERSP;

				        U.runtime.is_dirty = true;

Germano Cavalcante commented

2024-10-06 02:34:03 +02:00

Changes to vod->viewops_flag here seem to be OK, but I think changes to U.uiflag should only be made upon operator confirmation.
If the user cancels navigation, U.uiflag would not need to be reverted in that case.

Changes to `vod->viewops_flag` here seem to be OK, but I think changes to `U.uiflag` should only be made upon operator confirmation. If the user cancels navigation, `U.uiflag` would not need to be reverted in that case.

Harley marked this conversation as resolved

source/blender/editors/space_view3d/view3d_navigate.cc Outdated

						
				@ -598,0 +620,4 @@

				  status.opmodal(IFACE_("Move"), op->type, VIEWROT_MODAL_SWITCH_MOVE);

				  status.opmodal(IFACE_("Rotate"), op->type, VIEWROT_MODAL_SWITCH_ROTATE);

				  if (vod->nav_type == &ViewOpsType_rotate) {

Germano Cavalcante commented

2024-10-06 02:40:13 +02:00

To check if the operator supports any of these settings, you could check the vod->nav_type->flag and the flags VIEWOPS_FLAG_ORBIT_SELECT and VIEWOPS_FLAG_PERSP_ENSURE.

To check if the operator supports any of these settings, you could check the `vod->nav_type->flag` and the flags `VIEWOPS_FLAG_ORBIT_SELECT` and `VIEWOPS_FLAG_PERSP_ENSURE`.

Harley marked this conversation as resolved

Harley Acheson added 2 commits 2024-10-06 19:51:37 +02:00

Merge branch 'main' of projects.blender.org:blender/blender into OpPrefs 96cf52a494

Changes based on suggestions from Germano 018fb5f15f

Harley Acheson commented

2024-10-06 19:53:50 +02:00

The settings supported by the operator should be found in ViewOpsType::flag.

Oh, that is the trick. I didn't actually notice that vod->nav_type->flag was different than vod->viewops_flag for some reason. Makes perfect sense and works now.

Now to ponder it and see if it is a useful idea...

> The settings supported by the operator should be found in `ViewOpsType::flag`. Oh, that is the trick. I didn't actually notice that `vod->nav_type->flag` was different than `vod->viewops_flag` for some reason. Makes perfect sense and works now. Now to ponder it and see if it is a useful idea...

Harley Acheson commented

2024-10-06 20:03:43 +02:00

@pablovazquez

With help from @mano-wii this might be working as intended. Seems to work. It is a little awkward to change these options while doing an operation that requires ctrl or shift, but doable. And probably not something that is changed often since now buried in Preferences.

Although these settings changes affect the NEXT use of the operation, that doesn't seem to be an issue and feels like expected behavior. Or I could be wrong and this would result in bug reports about it not having immediate effect.

@pablovazquez With help from @mano-wii this might be working as intended. Seems to work. It is a little awkward to change these options while doing an operation that requires ctrl or shift, but doable. And probably not something that is changed often since now buried in Preferences. Although these settings changes affect the NEXT use of the operation, that doesn't seem to be an issue and feels like expected behavior. Or I could be wrong and this would result in bug reports about it not having immediate effect.

Harley Acheson added 1 commit 2024-10-06 21:57:41 +02:00

Adding more options f42e5676e1

Harley Acheson commented

2024-10-06 21:59:06 +02:00

@pablovazquez

I added even more options, very likely more than we'd want, like turntable/trackball, invert, horizontal zoom, etc. Just so this can help us in judging this idea and how far we'd go with it.

@pablovazquez I added even more options, **very likely more than we'd want**, like turntable/trackball, invert, horizontal zoom, etc. Just so this can help us in judging this idea and how far we'd go with it.

Germano Cavalcante commented

2024-10-06 22:08:25 +02:00

I remembered that some other options that affect navigation are in the Sidebar.
They are the Lock options and override the options in the preferences:

I find this a bit confusing, as these options are similar, mutually exclusive and spread out in different places.

I made a mockup (using Paint) of something that I think could be a better and more visible for navigation operations:

I remembered that some other options that affect navigation are in the Sidebar. They are the Lock options and override the options in the preferences: <img width="358" alt="image" src="attachments/08ff3d40-445d-4cc4-a315-17d7fafa630e"> I find this a bit confusing, as these options are similar, mutually exclusive and spread out in different places. I made a mockup (using Paint) of something that I think could be a better and more visible for navigation operations: <img width="241" alt="image" src="attachments/fa366400-2b48-4a9c-ab74-f2ee91ced759">

image.png

30 KiB

image.png

49 KiB

👍 1

Harley Acheson added 2 commits 2024-10-08 23:29:36 +02:00

Merge branch 'main' of projects.blender.org:blender/blender into OpPrefs 96b2659590

Simplify a bit. Separate these out as "Preferences" 4f26bcf2a4

Harley Acheson commented

2024-10-08 23:58:01 +02:00

@JulianEisel - I added a separating bar and "Preferences: " (along with the Preferences icon) to see how this feels.

It still feels okay. Without the icon I found it hard to see the differences in the sections well.

@JulianEisel - I added a separating bar and "Preferences: " (along with the Preferences icon) to see how this feels. It still feels okay. Without the icon I found it hard to see the differences in the sections well.

Julian Eisel commented

2024-10-09 17:51:38 +02:00

I think this latest change makes things quite clear now visually. I don't have a strong opinion about the feature itself, but if we add it, I think version is quite fine.

Harley Acheson referenced this pull request

2024-10-10 01:17:40 +02:00

UI: Move Preference Auto Close Pairs to Text Editor #128818

Harley Acheson referenced this pull request

2024-11-17 23:35:41 +01:00

Changing Preferences Outside of the Preferences Editor #130425

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions.