search

sebshader / pdnext

Color-themed Pure data with different inlets & outlets for messages and signals
Other
45 stars 3 forks source link

pd canvas size bug #2

Closed 60-hz closed 3 years ago

60-hz commented 3 years ago

Step to reproduce:

1 - create an object [pd sub] 2 - close it 3 - click in action mode to open it 4 - The patch size is very small Same happens with abstractions

Also There is many other little ui bug with ceammc, It might be related to tcl/tk 8.6.11 which is too new for for pd 0.51-4

OSX 10.15.4

Capture d’écran 2021-03-14 à 23 36 00

sebshader commented 3 years ago

@60-hz odd, I found this bug awhile ago but thought I fixed it, and can't reproduce it on osx 10.14.6.. are you sure you're running the right pd-next version? here is the pr it was supposed to be fixed in: https://github.com/pure-data/pure-data/pull/1236 it was merged into pd in 6589fed0f0ea9b4a20f2df05deb815f83c6fbe93

I'm not sure how to go about figuring this out since my osx version is very similar.. hmm.. I suppose the most I can do is to suggest looking at (right click on app) show package contents > Contents > Resources > tcl > pd_bindings.tcl and make sure that the lines 350 and 351 are as in the "files changed" section of the commit:

set width [winfo width $mytoplevel]
set height [winfo height $mytoplevel]

also if you don't mind confirming Pd>about pd says Pd 0.51.4 at the top in the app with this problem, just to make sure..

as for the ceammc bug, that has to be either fixed in ceammc or tk 8.6 since it's a bug with the ttk "alt" theme.. I may file a bug report myself later compiling pd-next to use tk 8.6.10 should work still though. I just think 8.6.11 is just more stable in general when it comes to things like scrollbars, event loops, and menus on modern macos.

60-hz commented 3 years ago

Oh, I checked and you are right, I think that I copied some old .tcl files from an old pd and forgot about it. Sorry for the noise!