-
Notifications
You must be signed in to change notification settings - Fork 40
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
Cleanup: Merge BlGridLayoutImpl into BlGridLayout #573
Comments
Out of curiosity, I looked for the history. The original sources should be smalltalkhub. I looked in their archives and there are 2 candidates: Maybe @plantec knows |
Same happens between BlFlowLayoutMeasurer and BlFlowLayout. |
ahh, ok. I think Glenn did it. |
This comment in linear layout lead me to Android (https://developer.android.com/reference/android/view/View.MeasureSpec) Bloc/src/Bloc-Layout/BlLinearLayoutMeasurer.class.st Lines 262 to 265 in a5fe2d3
|
Additional info (weird): Both BlGridLayoutImpl and BlGridLayout are subclasses of BlLayout |
BlGridLayoutImpl is only referenced from BlGridLayout's initialize, and then it's used by forwarding BlGridLayout's API to BlGridLayoutImpl.
This may be a reminiscence of a port from another library, as the Impl suffix is not common in Pharo.
The text was updated successfully, but these errors were encountered: