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

Many classes cannot be overwritten #530

Open
tomaszpieczykolan opened this issue May 12, 2021 · 1 comment
Open

Many classes cannot be overwritten #530

tomaszpieczykolan opened this issue May 12, 2021 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@tomaszpieczykolan
Copy link
Contributor

First of all, thank you for making this project! I think it's awesome that we are getting closer to being able to write proper Windows apps in Swift.

I was playing around with this framework and I noticed that I wasn't able to override the ViewController class. I can see that it is marked as public and not open. Is this intentional? Many other classes are also public.

Thanks!

@compnerd
Copy link
Owner

compnerd commented May 12, 2021

Thank you :)

Yes, the classes are currently intentionally marked as public. The only thing that would be preventative for changing classes to open is that they need to be audited and fixed up. Ideally, that would also be a good time to add some test cases (which currently are sorely lacking). When the class is marked as open, methods and properties should also be properly be converted to open from public.

If you want to take on the work to audit and adjust the classes which you need, that seems like a perfectly reasonable thing (although, that is not a please only do it for the classes you need, you are welcome to audit the entire surface area if you are motivated ;-)). PRs for that would be great, though I would ask that you keep a single PR focused to a single class - we can always work through a list of them.

@compnerd compnerd added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels May 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants