Tim Erickson

Results 395 comments of Tim Erickson

@jenlampton I think your UI suggestions are good and do make the form a bit easier to understand. The only thing I have a slight reservation about is "Page Complexity"....

@yorkshire-pudding, @cellear and I just reviewed this issue after the dev meeting. In general, we like this change but have a few small questions/suggestions: 1) In this case, we're updating...

Since the only way to test this PR is to install an OLD module and run the update process, I've included screenshots of the current pages to compare with those...

> What we need is a way to change the object name and retain the context: Backdrop CMS, Ubercart, CiviCRM, etc. Maybe this is in the form api title? This...

I certainly don't feel strongly about this. But, I can see where this is not a necessary default option (scheduling pages). In terms of discoverability, having it enabled on posts...

![image](https://github.com/user-attachments/assets/7644a9b2-b08c-4e88-bc09-17feae11d9f6) I like this idea. One thought. Should we include a `more` option with text along these lines. > Starting with version 1.30.0, Backdrop uses the installation version to [add...

Just tested this PR. It does actually add a new region to Rolph and Geary. ![image](https://github.com/user-attachments/assets/cd5fbdab-97a8-4efa-a726-3a6b6deb4e5e) While this works and solves the problem. I'm not sure it's the best solution....

@docwilmot I'm sorry, I thought that this issue was a specific or alternative fix for the problem in https://github.com/backdrop/backdrop-issues/issues/6461 As you noted, there is another fix for that problem. So,...

Maybe this will help. This "conditional" region will always show up on the layout page, even if there is nothing in it. Which is part of why this is confusing....

> I think these layouts could use a nice wide region under those narrow pillars (but above the 'bottom') to make them more useful. That's my primary motivation. OK, thanks....