-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Pattern Overrides: No opt-out mechanism exists for named overrides #59812
Comments
My idea to solve this would to implement this feature - Add option for fully locking a block Within the pattern blocks could be completely locked and this would be respected in instances of the pattern. The option could also be surfaced as part of the naming process so that a user could name the block and also opt-out as part of one step. Potentially in the future, this could be more granular and allow locking/unlocking individual attributes, but right now that seems like a longer term goal. |
I still hold the opinion that the default state of whether something is overwritable should be false. Even when named. The main interface for renaming a block is the list view. Not the advanced inspector controls. To me, this does not really solve the full UX issue. From my point of view, it is a bad experience to have a side-effect on a feature as common as renaming. Like I've said before. Users are so used to the fact that they can rename files, layers, elements, etc. in other pieces of software / operating systems. And that action should not have a side-effect. I'm fine with using the name as the underlying API for us to determine what overrides are connected to instead of a unique ID (even though I still think that fixing a typo in a name should not mean that all instances lose the connection because it is really hard for users to understand why that happened) (Originally shared this in #60066 but @kevin940726 shared it would be best to share here also :)) |
I don't yet personally have a full enough picture of the consequences one way or the other, but will say that if named overrides require opt-in, then it could potentially allow us to show a modal warning when renaming. |
Closing as since #60234 this feature requires opt-in by default. |
Problem
Since the PR 'Use block naming for marking blocks as overridable in patterns' was merged, all a user has to do to mark a block within a pattern as overridable is to provide a block name—either using List View, or the input in the 'Advanced' inspector controls.
Currently there's no way to name a block for organizational purposes, and not have it be overridable, which is a problem.
Let's discuss potential solutions to this problem in the comments.
The text was updated successfully, but these errors were encountered: