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

Model / DisplayableObject relation should be thought carefully #2561

Closed
Ducasse opened this issue Feb 15, 2019 · 3 comments
Closed

Model / DisplayableObject relation should be thought carefully #2561

Ducasse opened this issue Feb 15, 2019 · 3 comments
Labels
Project: Graphics Athens, Bloc, Morphic, Fonts, etc. Type: Cleanup

Comments

@Ducasse
Copy link
Member

Ducasse commented Feb 15, 2019

  • Naively I was thinking that DisplayableObject should be a superclass of Model so that we can
    also directly inherit from it (some objects are not model but still want to have theme, and other)
  • DisplayableObject should probably also offer UIManager default access.
  • I'm thinking that DisplayableObject could be a subclass of Model. But may be it should not.
    In any case it should packaged in a UI package because right now Model is not depending on UI concerns. So it would be good not to introduce it.
@stale
Copy link

stale bot commented Aug 14, 2019

To limit bug bankruptcy (see https://www.joelonsoftware.com/2012/07/09/software-inventory/) this issue has been automatically marked as stale because it has not had any activity in 6 months. It will be closed in 1 month if no further activity occurs. If this issue remains important to you, please comment to reactivate the issue. Thank you for your contributions.

Joel on Software
Imagine, for a moment, that you came upon a bread factory for the first time. At first it just looks like a jumble of incomprehensible machinery with a few people buzzing around. As your eyes adjus…

@stale stale bot added the stale label Aug 14, 2019
@Ducasse
Copy link
Member Author

Ducasse commented Aug 15, 2019

This is still active. I'm waiting for Spec 20 to have have an icon manager and a theme method so that I can check all the non Model/presenter users of smalltalk ui theme smalltalk ui icons....

@stale stale bot removed the stale label Aug 15, 2019
@guillep guillep added Type: Cleanup Project: Graphics Athens, Bloc, Morphic, Fonts, etc. labels Mar 27, 2020
@guillep
Copy link
Member

guillep commented Jan 27, 2023

This is a big non punctual thing, difficult to action.
I propose to close it, maybe this should be moved to a place/wiki that stores "project ideas".

@guillep guillep closed this as completed Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Project: Graphics Athens, Bloc, Morphic, Fonts, etc. Type: Cleanup
Projects
None yet
Development

No branches or pull requests

2 participants