Sietse Brouwer SietseB
  • Joined on 2023-02-08
Sietse Brouwer commented on pull request blender/blender#123982 2024-07-02 12:43:01 +02:00
macOS: Colored Titlebar and WM Client-Side Decorations API - GSoC 2024

Is this one used?

Sietse Brouwer commented on pull request blender/blender#123982 2024-07-02 12:43:00 +02:00
macOS: Colored Titlebar and WM Client-Side Decorations API - GSoC 2024

The abbreviation CSD is a bit cryptic, I think. In the context of this PR I understand it, but when I would come across WM_window_set_use_csd(win, true) in the wild, I wouldn't know what it meant.

Sietse Brouwer commented on pull request blender/blender#123982 2024-07-02 12:42:59 +02:00
macOS: Colored Titlebar and WM Client-Side Decorations API - GSoC 2024

Silly nitpick, but the style guide tells us to end comments with a full stop.

Sietse Brouwer commented on pull request blender/blender#123982 2024-07-02 12:42:58 +02:00
macOS: Colored Titlebar and WM Client-Side Decorations API - GSoC 2024

Second nitpick: better full variable names like backgroundColor than tbBackgroundCol. The 'title bar' part is implied by the function.

Sietse Brouwer commented on pull request blender/blender#123443 2024-06-23 13:25:46 +02:00
UI: Remember position and size of Preferences and Render window

@Brainzman Like you said on the chat, for macOS, letting AppKit do the job of restoring window position sounds like a good line of inquiry. It could be a matter of:

  • Adding some GHOST [function…
Sietse Brouwer pushed to ui-store-render-pref-window-position at SietseB/blender 2024-06-23 12:56:11 +02:00
9a194a137f Cleanup after merge with main
f08019da1a Merge branch 'main' into ui-store-render-pref-window-position
b32f3ae538 Merge branch 'blender-v4.2-release'
1ba06956e7 Extensions: support platform & version from URL parameters
8f95c2ce9f Merge branch 'blender-v4.2-release'
Compare 280 commits »
Sietse Brouwer commented on pull request blender/blender#123443 2024-06-21 00:19:37 +02:00
UI: Remember position and size of Preferences and Render window

Thanks for testing, Jonas! It would be great if you could look into this – when it fits into your GSoC project, that is, of course.

I did some Stackoverflow digging and now I understand the…

Sietse Brouwer pushed to ui-store-render-pref-window-position at SietseB/blender 2024-06-20 19:06:13 +02:00
20d488fb41 Prevent incorrect window size in multi-monitor setup with different dpi's
Sietse Brouwer created pull request blender/blender#123443 2024-06-19 17:44:39 +02:00
UI: Remember position and size of Preferences and Render window
Sietse Brouwer created branch ui-store-render-pref-window-position in SietseB/blender 2024-06-19 17:02:16 +02:00
Sietse Brouwer pushed to ui-store-render-pref-window-position at SietseB/blender 2024-06-19 17:02:16 +02:00
72dc3dfbcd Remember position of Preferences and Render window
eb731c036b Refactor: Anim filtering, avoid passing bDopesheet *ads parameter
f1fa7e180b Remove outdated comment
033a3f09b6 Merge branch 'blender-v4.2-release'
bc7cb481da Fix #123339: Blender fails to build with older versions of wayland
Compare 10 commits »
Sietse Brouwer commented on pull request blender/blender#120869 2024-06-10 11:17:22 +02:00
GPv3: Use multi frame falloff in transform tools

When multi frame editing isn't enabled, the frame_falloff is set to 1.0 (so no falloff).

I've improved the comments intransform_mode_curveshrinkfatten.cc a bit, because there it's more…

5eecdc9a76 Code review changes and resolved merge conflicts
e45b3617ea Merge branch 'main' into gpv3-transform-tools-multiframe-falloff
cf3323fe32 Merge branch 'blender-v4.2-release'
f59f5fd83f Fix #122901: Geonodes modifier- setting an imput as string crashes blender.
c4c3812894 Merge branch 'blender-v4.2-release'
Compare 1855 commits »
Sietse Brouwer pushed to gpv3-weight-gradient-tool at SietseB/blender 2024-06-09 21:00:03 +02:00
6461d1e340 Adapt PR to OB_MODE_WEIGHT_GPENCIL_LEGACY, give Gradient tool a brush of its own
17fbdec2c7 Merge branch 'main' into gpv3-weight-gradient-tool
cf3323fe32 Merge branch 'blender-v4.2-release'
f59f5fd83f Fix #122901: Geonodes modifier- setting an imput as string crashes blender.
c4c3812894 Merge branch 'blender-v4.2-release'
Compare 2174 commits »
Sietse Brouwer commented on pull request blender/blender#121407 2024-06-05 11:38:58 +02:00
WIP: GPv3: RNA for frame, drawing, stroke and point

That's right 👍 And the array is filled with a back pointer to Drawing because that information isn't available in PointerRNA ptr (because Drawing isn't an ID type).

Sietse Brouwer commented on issue blender/blender#122677 2024-06-05 11:31:05 +02:00
GPv3: High-level Python API

@ChengduLittleA That one is put on hold for now. It needs a design task first, agreement on the design and then (adjusting) the implementation – not enough time in 4.3 to get all those steps…

Sietse Brouwer commented on pull request blender/blender#121407 2024-06-05 11:16:55 +02:00
WIP: GPv3: RNA for frame, drawing, stroke and point

The & makes the difference here: this function is returning a pointer indicating the nth element in the StrokeSlice array. In rna_StrokeSlice_index_get this pointer is used to calculate the…

Sietse Brouwer commented on issue blender/blender#122677 2024-06-04 00:14:28 +02:00
GPv3: High-level Python API

So, if I understand it correctly, the proposal is what's already implemented in [#121407: WIP: GPv3: RNA for frame, drawing, stroke and point](blender/blender#121

bf8226d541 Cleanup after merge with main
9a14cd67cf Merge branch 'main' into gpv3-python-api-drawing-with-strokes-and-points
3a2ea7828e Fix #122599: Windows Registration/Unregistration Endless Loop
5e6760d554 Fix #122625: List of XForm ops needs to be stored at class level
a0360be760 Fix #122517: Ensure depsgraph is updated before disabling subdiv
Compare 1238 commits »
Sietse Brouwer closed pull request blender/blender#111133 2024-06-03 15:07:20 +02:00
GPencil: Gradient weight paint tool