Pyramid icon indicating copy to clipboard operation
Pyramid copied to clipboard

Cannot compare ToTheme in the theme selector

Open labordep opened this issue 10 months ago • 6 comments

Exception : NotFound: [ :each | each model = anItem ] not found in OrderedCollection when I'm trying to edit a Toplo view into Pyramid.

image

labordep avatar Apr 06 '24 12:04 labordep

This is major because I cannot edit Toplo view with theme!

labordep avatar Apr 06 '24 12:04 labordep

@plantec says that it should be better to ask the Theme class and not the Theme instance.

labordep avatar Apr 06 '24 12:04 labordep

It is true if we considere the theme as static. For example we do not dynamicly update token values

Nyan11 avatar Apr 10 '24 07:04 Nyan11

That's why I prefer to implement "=" in Themes :)

labordep avatar Apr 10 '24 08:04 labordep

no = please. now, token properties can be added dynamically in a theme. @pierre, this is not that one should take the class. I said to store an instance in a class variable :)

plantec avatar Apr 13 '24 19:04 plantec

We can display the class name of the current Theme in the list and propose to setup another one using just class name comparison. We can reset the current Theme (in case of modifications) by a new one from the same class.

labordep avatar Apr 13 '24 19:04 labordep