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

FullBuild.xlam disaligned with srcWindow.cls #65

Closed
6DiegoDiego9 opened this issue Jan 29, 2023 · 6 comments
Closed

FullBuild.xlam disaligned with srcWindow.cls #65

6DiegoDiego9 opened this issue Jan 29, 2023 · 6 comments

Comments

@6DiegoDiego9
Copy link
Contributor

In FullBuild.xlam I get an error on the line "RectClient = rect" because RectClient is read-only (true, since it has no letter proc.)

Public Property Let RectByType(Optional iClWnd As EWndRectType, ByVal rect As Variant)
  Select Case iClWnd
    Case EWndRectType.RectTypeClient
      RectClient = rect
    Case EWndRectType.RectTypeWindow
      RectWindow = rect
  End Select
End Property

However, src\stdWindow.cls has "RectWindow = rect" in its place.

I'm new to your repository. Should I always take the src files as the latest version?

@sancarn
Copy link
Owner

sancarn commented Jan 29, 2023

because RectClient is read-only (true, since it has no letter proc.)

Ah yes, sorry, that is a WIP feature which I've since actually just added an error in place

I'm new to your repository. Should I always take the src files as the latest version?

Before raising an issue it would likely be good to try downloading the repo again yeah. I'm making fixes and changes all the time. Unfortunately I don't have a means of updating the library classes on demand (yet) :)

@6DiegoDiego9
Copy link
Contributor Author

I see, however even after I replaced all the classes of FullBuild.xlam with the src*.cls, I still get errors:

image

image

I corrected(?) by removing the parenthesis and adding an "s".

@sancarn
Copy link
Owner

sancarn commented Jan 30, 2023

Thanks I'll go through and fix the compile errors. I haven't touched stdError in a long time.

Hope you find the library useful. Typically I don't add all modules into my projects, as they add a lot of clutter. Very few modules rely on eachother being around. So I'd only import the ones you need into your own projects :)

@6DiegoDiego9
Copy link
Contributor Author

Thanks for the fixes 👍
I'm just giving a look at everything for the first time, to evaluate hands-on what to keep and discard.
What if I'll want to contribute with some work on some selected libraries? if you work on them offline for days, it would be risky...

@sancarn
Copy link
Owner

sancarn commented Jan 30, 2023

if you work on them offline for days, it would be risky

Whenever I make a change usually I commit straight away, although sometimes features are in planning for a while. Git sorts most of the conflicts out and I will likely have to help you through your first Pull Request, but that's fine and something I'm happy to do so long as it fits in with the strategy of the project :). Might be worth raising an issue with any ideas you want to implement first or taking something from the existing roadmap if any peak your fancy :)

What if I'll want to contribute with some work on some selected libraries

If you do want to make a change or an addition:

  1. Make a branch of the repository.
  2. Make any changes / additions you want to make to your own branch.
  3. Create a "Pull Request"
  4. We'll resolve any conflicts together
  5. And if all goes well I'll approve it into the library.

If you have any specific

@sancarn
Copy link
Owner

sancarn commented Jan 31, 2023

Closing this as issues fixed

@sancarn sancarn closed this as completed Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants