Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

request: include depth argument for 'isvisible' message in [pdcontrol] #803

Open
porres opened this issue Dec 2, 2019 · 3 comments · May be fixed by #828
Open

request: include depth argument for 'isvisible' message in [pdcontrol] #803

porres opened this issue Dec 2, 2019 · 3 comments · May be fixed by #828
Labels

Comments

@porres
Copy link
Contributor

@porres porres commented Dec 2, 2019

a depth argument in the 'isvisible' message can query if parent window is visible or not, in a similar fashion to 'dir/args' messages.

@umlaeute umlaeute added the feature label Dec 4, 2019
@millerpuckette

This comment has been minimized.

Copy link
Contributor

@millerpuckette millerpuckette commented Dec 5, 2019

Adding features is how software inflates and eventually dies of bloat. One wants a valid use case before just adding stuff. And even then, if there's already a way to do the same thing, one should think twice.

@porres

This comment has been minimized.

Copy link
Contributor Author

@porres porres commented Dec 5, 2019

yeah, like I said in the pd-list, I had a use case for it which I didn't need in the end, and now I don't even know of a use case for this as it is :) so if no one really needs this (as I expect anyway), I'll just close this.

ps. would still like to know a use case for this just by sheer curiosity ;)

@porres

This comment has been minimized.

Copy link
Contributor Author

@porres porres commented Dec 6, 2019

Well, like I said in the Pd list, I can see how this is useful now. And then the depth argument would follow the same syntax and idea of the 'dir' and 'args' message, because it's really useful when used inside abstractions. So I'll keep the request and maybe give it a go sometime soon.

porres added a commit to porres/pure-data that referenced this issue Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.