Making Responsive Apps Easier with this Package

[meredydd]

This is a really cool package!

However, I would like to plead with you not to do this:

@divyeshlakhotia

This package also makes it easier to modify the css of a component directly using python.

self.label_1.set('padding-top','10px', important = True)

Putting extra things onto the Component class creates new things that will break when we next update that class! (Or worse, it might break the Components in a way that people then blame on us!) And a name like set is pretty generic – who knows what we might want to put there?

I strongly, strongly advise an approach like I described here, using a function imported from your dependency rather than monkey-patching the Component class:

HTML id attribute for components

Playwright sounds like they have the right approach! An attribute is much less disruptive than requiring an ID. As a general point of style, “monkey-patching” like some of the examples in this thread (ie adding extra properties and things to system classes to do what you want) is something the Python community discourages. It’s brittle, because it could break (or cause mysterious breakages in your code) when we update the inner workings of the Component class. It’s also harder for readers of yo…