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

surface methods get too much use, move into the API #53

Closed
processing-bot opened this issue Jan 19, 2020 · 4 comments
Closed

surface methods get too much use, move into the API #53

processing-bot opened this issue Jan 19, 2020 · 4 comments

Comments

@processing-bot
Copy link
Collaborator

Created by: benfry

We've been surprised at how much frame.setResizable() and frame.setTitle() and others are being used, and we should make them into real API.

The switch from frame to surface between 2.x and 3.x was bumpier as a result.

Need to clean this up and get better behaviors for all of them:

  • setSize() is erratic across renderers
  • setResizable() can we make this a one-time operation in settings()?
  • setTitle() is often used to output the fps of an app (a terrible idea, but there we are) so needs to happen at any time (or limited to settings() to prevent that usage)
@processing-bot
Copy link
Collaborator Author

Created by: knupel

Hi, @benfry what you mean by move surface.setSize() in API. If this option is choice. There will be no more possibility to use it in code ? or you mean create a specific Processing methode to call it ? like surface.setSize() became simply setSize() I hope you think about this lastoption, because like you have wrote I use a lot surface.setSize() and surface.setTitle() in my code. By the way congratulation for the time you take to keep Processing up to date with @sampottinger.

@processing-bot
Copy link
Collaborator Author

Created by: benfry

@StanLepunK No, the opposite of that. They were left in the surface class because they weren't expected to be used much. Because they get used a lot, we should make them more official than having to introduce this weird surface thing to everyday users, i.e just resize() instead of surface.setSize() or something like that.

@processing-bot
Copy link
Collaborator Author

Created by: knupel

@benfry "ouf", like will said french guy like me. But I see there is already method public void setSize(int wide, int high) in public class PSurfaceAWT extends PSurfaceNone I never use it, it don't work ? I ask because here i'm at this time I cannot use Processing :( If something must be coded, this class is it a place to do ?

@processing-bot
Copy link
Collaborator Author

Created by: github-actions[bot]

This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant